| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-31 | |||
| 10:37:41 | openstackgerrit | Matthew Booth proposed openstack/nova master: Fix scope of errors_out_migration in resize_instance https://review.openstack.org/487495 | |
| 10:37:41 | openstackgerrit | Matthew Booth proposed openstack/nova master: Automatically revert resize which fails on destination https://review.openstack.org/462521 | |
| 10:37:42 | openstackgerrit | Matthew Booth proposed openstack/nova master: Fix scope of errors_out_migration in finish_resize https://review.openstack.org/487515 | |
| 10:37:42 | openstackgerrit | Matthew Booth proposed openstack/nova master: Ensure errors_out_migration errors out migration https://review.openstack.org/479802 | |
| 10:54:59 | openstackgerrit | Rawan Herzallah proposed openstack/nova master: Adding NVMEoF for libvirt driver https://review.openstack.org/482640 | |
| 11:03:45 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: explain payload inheritance in notification devref https://review.openstack.org/453667 | |
| 11:05:54 | redondo-mk | Hi. I have a question regarding packstack's AIO deployment of Liberty. In my fresh deployment, instances are successfully created, end up being in "running" state but they are not reachable by icmp (security group properly). Instance have empty console log file. I've tried cirros and ubuntu. Would anyone have any hint/info to direct me towards steps how to get my instances to work properly and not just | |
| 11:05:54 | redondo-mk | pretend that they work. In Newton I don't have this issue, don't know about Mitaka but the thing is that I need Liberty. | |
| 11:35:21 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test resize with placement api https://review.openstack.org/487958 | |
| 11:36:01 | gibi_ | cdent: hi! I think I was able to make the fake virt driver work with periodic tasks enabled in ^^ but still some asserts fail | |
| 11:36:22 | gibi_ | cdent: I will try to rebase the test top of https://review.openstack.org/#/c/488510/ to see if that helps | |
| 11:37:30 | cdent | gibi_: cool. I'm playing around with some gabbi+tempest tests to see if I can learn anything from that side, but I'm not very close yet | |
| 11:39:15 | cdent | more coffee required | |
| 11:39:32 | gibi_ | coffee is a good idea, always :) | |
| 11:41:34 | openstackgerrit | Sean Dague proposed openstack/nova master: Better handling of Neutron BadRequest/ Vendor specific exceptions https://review.openstack.org/278788 | |
| 11:52:42 | openstackgerrit | Michael Still proposed openstack/nova master: Move execs of touch to privsep. https://review.openstack.org/489190 | |
| 12:18:22 | sdague | mikal: you still awake? | |
| 12:18:36 | sdague | https://review.openstack.org/#/c/486831 just a comment before approve | |
| 12:50:46 | openstackgerrit | Alex Xu proposed openstack/nova master: placement: filtering the resource provider id when delete trait association https://review.openstack.org/489205 | |
| 12:50:47 | openstackgerrit | Alex Xu proposed openstack/nova master: placement: The common method to filter resource providers by traits https://review.openstack.org/489206 | |
| 13:04:40 | sdague | alex_xu: any more extensions remove patches need review? | |
| 13:19:30 | yasemin | hi, i build multi node devstack stable/ocata , but controller node recognize compute node. Can you help me | |
| 13:32:50 | sdague | yasemin: did you discover_hosts after the compute came up? | |
| 13:33:38 | sdague | nova-manage cell_v2 discover_hosts --verbose | |
| 13:35:48 | openstackgerrit | Merged openstack/python-novaclient master: doc: Remove Makefile https://review.openstack.org/480096 | |
| 13:40:04 | edleafe- | Scheduler subteam meeting in 20 minutes in #openstack-meeting-alt | |
| 13:43:31 | yasemin | <sdague> i run "nova-manage --debug cell_v2 discover_hosts" command, but it is not working | |
| 13:43:32 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test resize with placement api https://review.openstack.org/487958 | |
| 13:47:51 | bauzas | mmm, I have a thought if someone is kinda seasoned with pbr | |
| 13:48:27 | bauzas | say we ask operators to add a new entrypoint in setup.cfg, how can nova be redeployed by using it if you use a downstream package ? | |
| 13:49:12 | bauzas | of course, we all know we just need to either "develop" or "install" either by pip or setup.py but the question remains for other packages | |
| 13:49:29 | bauzas | a reasonable answer can be "meh, we don't care, it's a package-specific discussion" | |
| 13:51:56 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Accept any scheduler driver entrypoint https://review.openstack.org/484828 | |
| 13:52:04 | bauzas | context being ^ | |
| 13:52:36 | gibi_ | cdent: with Jay's https://review.openstack.org/#/c/488510 and resource_tracker on drugs I can reproduce problems with resize in a stable way with https://review.openstack.org/#/c/487958 | |
| 13:53:03 | openstackgerrit | Merged openstack/python-novaclient master: Update reno for stable/pike https://review.openstack.org/488799 | |
| 13:53:04 | cdent | gibi_: excellent, looking | |
| 13:53:26 | gibi_ | cdent: it seems resource clean up is missing after both confirm and revert | |
| 13:54:27 | cdent | nice work | |
| 13:55:25 | cdent | good timing too, we can noodle about in the scheduler meeting | |
| 13:57:54 | openstackgerrit | Merged openstack/nova master: hyperv: stop serial console workers while deleting vm files https://review.openstack.org/387831 | |
| 13:59:36 | edleafe | cdent: gibi_: just added it to my local agenda | |
| 14:00:11 | gibi_ | edleafe: cool, I will join to the meeting | |
| 14:00:35 | edleafe | Scheduler meeting starting now! | |
| 14:04:09 | openstackgerrit | Merged openstack/nova master: Stop using mox stubs in test_console_auth_tokens.py https://review.openstack.org/477374 | |
| 14:04:39 | openstackgerrit | Merged openstack/nova master: Add oslo_concurrency=INFO to default log levels for nova-manage https://review.openstack.org/487179 | |
| 14:04:45 | bauzas | mriedem: looks a pretty critic bug to me https://bugs.launchpad.net/nova/+bug/1707319 | |
| 14:04:45 | openstack | Launchpad bug 1707319 in OpenStack Compute (nova) "Security group doesn't apply to existing port" [Undecided,In progress] - Assigned to hongbin (hongbin034) | |
| 14:04:57 | bauzas | mriedem: critical* even | |
| 14:05:26 | bauzas | mriedem: I'm trying to get when we regressed (if we regressed) | |
| 14:05:49 | mriedem | hongbin pinged me about it on saturday, i haven't looked | |
| 14:07:30 | bauzas | mriedem: the problem is when you pass an existing NIC | |
| 14:07:40 | bauzas | mriedem: we totally forget to check the SGs if so | |
| 14:07:55 | bauzas | mriedem: we only do that when you ask nova to create a port | |
| 14:15:39 | bauzas | mriedem: thinking out loud, are security groups a nova thing or a neutron thing ? | |
| 14:15:50 | bauzas | mriedem: looks like we proxied that to neutron, nope ? | |
| 14:17:36 | mriedem | we proxy when creating the port | |
| 14:19:06 | bauzas | mriedem: nevermind, got my answer https://docs.openstack.org/security-guide/networking/services-security-best-practices.html | |
| 14:19:11 | bauzas | mriedem: will invalid the bug | |
| 14:20:54 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test resize with placement api https://review.openstack.org/487958 | |
| 14:21:27 | gibi_ | jaypipes: just added more printout to see the whole resource state when the test fails. The commit message no links to test results with that extra printouts . ^^ | |
| 14:22:27 | mriedem | bauzas: can you clarify? | |
| 14:22:44 | mriedem | https://docs.openstack.org/security-guide/networking/services-security-best-practices.html#security-groups just says nova proxies security group information to neutron | |
| 14:22:57 | kashyap | smcginnis: Hey, thanks for looking out for Jenkins for this -- https://review.openstack.org/#/c/489198. Or do you do it by a script? :-) | |
| 14:23:33 | bauzas | mriedem: I just made a comment | |
| 14:23:34 | smcginnis | kashyap: Hah, nope. I took a quick look and saw it had a timeout failure. | |
| 14:24:06 | gibi_ | jaypipes: s/no/now/ | |
| 14:24:13 | kashyap | smcginnis: Ah, thank you. | |
| 14:24:24 | smcginnis | kashyap: No problem. ;) | |
| 14:24:25 | bauzas | mriedem: tl;dr: since the port was created beforehand directly to Neutron, you should define the SG policies at that moment | |
| 14:25:02 | jaypipes | gibi_: ty sir | |
| 14:26:03 | mriedem | bauzas: did you check the code? are you sure we don't attempt to update the port? | |
| 14:26:04 | jaypipes | dansmith, cdent, gibi_: I'm wondering if we have any tests that use the libvirt driver and not the fake virt driver for resize/move operations? | |
| 14:26:15 | mriedem | because i know we update the port, i'm just not sure what we put in the body for the PUT | |
| 14:26:29 | dansmith | jaypipes: eh? don't we actually do moves in tempest? | |
| 14:26:37 | cdent | there are some moves in tempest | |
| 14:26:54 | mriedem | yes we have resize, cold migration, shelve/unshelve and live migration in tempest | |
| 14:27:14 | jaypipes | do we see the same issue with placement not being properly updated in those tempest tests? | |
| 14:27:14 | bauzas | mriedem: nope, we don't update the port | |
| 14:27:14 | mriedem | resize is tested single node in single node jobs, otherwise those are all tested on multinode jobs | |
| 14:27:16 | dansmith | maybe jaypipes means unit/functional tests? | |
| 14:27:21 | bauzas | mriedem: I checked the code | |
| 14:27:25 | bauzas | mriedem: we just ignore that | |
| 14:27:40 | gibi_ | jaypipes: notification sample test are doing resize with FakeDriver | |
| 14:27:41 | bauzas | mriedem: and that's at least existing from stable/newton | |
| 14:27:50 | bauzas | it's not a regression at all | |
| 14:27:51 | dansmith | jaypipes: I was looking at your patch and I think we probably need a debug statement when we do the remove so we know it's happening | |
| 14:28:07 | dansmith | i.e. so we can check it in the tempest run logs | |
| 14:28:08 | jaypipes | dansmith: totes | |
| 14:28:17 | bauzas | mriedem: so we *could* update the port | |
| 14:28:33 | gibi_ | jaypipes: also the server_group test has VM moves like migrate and evacuate | |
| 14:28:38 | bauzas | mriedem: but that's just two roundtrips IMHO | |
| 14:28:59 | gibi_ | jaypipes: e.g. https://github.com/openstack/nova/blob/master/nova/tests/functional/test_server_group.py#L367 | |
| 14:29:04 | cdent | jaypipes, dansmith, gibi_: what we don't have in those tests is any confirmation of the state of allocations | |
| 14:29:11 | bauzas | mriedem: since nova has SGs per instances, while neutron has per ports, it's a waste of HTTP calls IMHO | |
| 14:29:17 | cdent | and since we're probably not running at capacity it doesn't become an issue | |
| 14:30:08 | gibi_ | cdent: yes, I think the only test where we assert on capacity is the new one we are troubleshooting now | |
| 14:30:23 | dansmith | jaypipes: I don't see any "expected to find" logs in the multinode run, which would mean we were bailing out of that un-doubling | |
| 14:31:03 | mriedem | gibi_: i think cdent is talking about tempest dsvm runs | |
| 14:31:09 | mriedem | where we aren't using fake drivers | |