| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-05 | |||
| 11:59:23 | sean-k-mooney | stephenfin: yep git review -f work as i wanted thanks :) | |
| 11:59:25 | efried | sean-k-mooney stephenfin +1 git-review | |
| 11:59:44 | stephenfin | sean-k-mooney: two tips for you too. #1 - if you want to download a single patch with dependencies, use '-x' instead of '-d' | |
| 12:00:16 | stephenfin | (very helpful when someone has uploaded a new revision of a patch in the middle of a series, are you want to rebase the successors) | |
| 12:00:54 | stephenfin | and #2 - use '{change},{revision}' to download a previous revision | |
| 12:01:06 | stephenfin | e.g. 'git review -d 466132,1' | |
| 12:03:23 | openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: [ci] Use pseudo-random names for new resources https://review.openstack.org/479777 | |
| 12:03:44 | sean-k-mooney | efried: ya i use it for upload becase "git push gerrit HEAD:refs/change/466132" is annorying but i havent been useing the other features | |
| 12:03:57 | sean-k-mooney | stephenfin: cool that is good to know | |
| 12:15:58 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091 | |
| 12:23:53 | openstackgerrit | Vladyslav Drok proposed openstack/nova master: Implement interface attach/detach in ironic virt driver https://review.openstack.org/419975 | |
| 12:26:34 | openstackgerrit | Vladyslav Drok proposed openstack/nova master: Implement interface attach/detach in ironic virt driver https://review.openstack.org/419975 | |
| 12:39:55 | jaypipes | efried: because {1, 2, 3} is too easy to mistake for a dict. | |
| 12:40:23 | efried | jaypipes It is? | |
| 12:40:35 | jaypipes | cdent: I would moo back but suffering from allergies badly today :( | |
| 12:41:00 | cdent | bummer dood | |
| 12:41:04 | jaypipes | efried: perhaps not for you. for others it is :) | |
| 12:41:38 | efried | jaypipes Okay, good to know. | |
| 12:42:50 | alex_xu | jaypipes: cdent two fixes for AllocationCandidates.get_by_filters https://review.openstack.org/480379 and https://review.openstack.org/480130 | |
| 12:43:05 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.live_migration_force_complete notification https://review.openstack.org/480561 | |
| 12:43:35 | cdent | I've been pretty good at not paying attention for a while | |
| 12:43:59 | alex_xu | cdent: welcome back :) | |
| 12:44:07 | cdent | thanks alex_xu | |
| 12:44:31 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.live_migration_force_complete notification https://review.openstack.org/480561 | |
| 12:45:06 | alex_xu | nova api meeting is in 15 mins at #openstack-meeting-4 | |
| 12:47:31 | openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for allocations https://review.openstack.org/470933 | |
| 12:47:32 | openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP allocations https://review.openstack.org/450550 | |
| 12:47:32 | openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP usages https://review.openstack.org/450105 | |
| 12:47:33 | openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for usages https://review.openstack.org/480563 | |
| 12:51:22 | openstackgerrit | Andrey Volkov proposed openstack/nova master: PoC: Select PCI devices with distinct tag values https://review.openstack.org/448008 | |
| 13:18:20 | openstackgerrit | Silvan Kaiser proposed openstack/nova master: [WIP] Add truncate for images if prealloc is true https://review.openstack.org/480464 | |
| 13:19:41 | cdent | avolkov: can you give me a quick summary on where things are with placement api-ref? Has there been any progress on publishing for real? | |
| 13:22:11 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257 | |
| 13:23:50 | bauzas | jaypipes: I'd really love to see your insight in https://review.openstack.org/#/c/480379/2 | |
| 13:35:39 | cdent | you've infected me jaypipes, my allergies are spinning up | |
| 13:46:37 | openstack | Launchpad bug 1698010 in OpenStack Compute (nova) "neutron-based instances should not use the nova-network 'dhcp_domain' option" [High,Confirmed] | |
| 13:46:37 | stephenfin | sean-k-mooney: Any ideas on https://bugs.launchpad.net/nova/+bug/1698010 ? | |
| 13:46:41 | stephenfin | ralonsoh too ^ | |
| 13:48:02 | ralonsoh | stephenfin: let me take a look at this one | |
| 13:48:18 | mriedem | stephenfin: that is a duplicate, | |
| 13:48:21 | mriedem | just need to find the other bug | |
| 13:48:24 | stephenfin | I can get network info from the instance using 'Instance.get_network_info', but it just returns a list of 'os_vif.objects.vif.VIFBase' subclass instances | |
| 13:48:29 | stephenfin | mriedem: Oh hai | |
| 13:48:38 | mriedem | the dhcp_domain option is read in the metadata api code and that's how it gets into config drive | |
| 13:48:48 | mriedem | as noted by bnemec in the bug report it's showing up via cloud-init | |
| 13:49:08 | stephenfin | Yup :( I was trying to see where I could get the neutron equivalent from, but we don't seem to cache it anywhere | |
| 13:49:27 | stephenfin | and I don't want to call out to neutron for every instance (assuming that is even an option) | |
| 13:50:07 | mriedem | we already call to neutron for dns info when we create or attach to a port | |
| 13:51:05 | stephenfin | Hmm, so is that a viable path to pursue? | |
| 13:51:37 | openstack | Launchpad bug 1702454 in OpenStack Compute (nova) "Transforming the RequestSpec object into legacy dicts doesn't support the requested_destination field" [High,Confirmed] - Assigned to Sylvain Bauza (sylvain-bauza) | |
| 13:51:37 | bauzas | mriedem: permission to apologize, captain. https://bugs.launchpad.net/nova/+bug/1702454 | |
| 13:52:18 | bauzas | mriedem: tl;dr: evacuations providing a target don't verify that target since newton | |
| 13:53:04 | mriedem | i thought the bug said that was only for legacy specs or whatever? | |
| 13:53:27 | mriedem | someone from suse was just fixing bugs in stable related to evacuate with a targeted host, so something must work | |
| 13:54:17 | bauzas | mriedem: see the description | |
| 13:54:39 | bauzas | mriedem: that's because we turn into dicts only for evacuate | |
| 13:54:41 | mriedem | stephenfin: could we just make dhcp_domain default to None? that would change how https://github.com/openstack/nova/blob/0b9bacb/nova/api/metadata/base.py#L546-L549 behaves | |
| 13:55:00 | bauzas | mriedem: and tbh, the bug was reported internally on our Newton branch | |
| 13:55:35 | bauzas | so I doubt it's fixed upstream but just master because of https://review.openstack.org/#/c/469037/ | |
| 13:55:42 | bauzas | that's where tech debt kills us | |
| 13:56:10 | bauzas | and I wouldn't like to backport https://review.openstack.org/#/c/469037/ | |
| 13:56:20 | bauzas | rather fixing the problem with the transformating | |
| 13:56:58 | stephenfin | mriedem: It would change it, but I don't think it would deliver the desired behavior. bnemec did that, and cloudinit added a default 'localdomain' domain instead of what neutron had configured ('nemebean.com') | |
| 13:58:23 | openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | |
| 14:05:44 | dansmith | mriedem: are we going to go ahead with this? https://review.openstack.org/#/c/468227/2 | |
| 14:07:37 | mriedem | dansmith: related to bauzas' evacuate in newton clusterfunnery above? | |
| 14:07:53 | mriedem | is there a rhosp newton evacuate red alert going off this morning? | |
| 14:08:20 | bauzas | not really | |
| 14:08:24 | bauzas | AFAIK | |
| 14:08:24 | dansmith | mriedem: is it related to that? I don't think so | |
| 14:08:34 | bauzas | just some internal BZ triage I did | |
| 14:08:44 | dansmith | I don't know about bauzas' thing so I assume unrelated | |
| 14:08:52 | mriedem | dansmith: but yes we should get that in - the evacuate stuff with forced target was broken since newton i think | |
| 14:08:54 | bauzas | yup, unrelated | |
| 14:09:01 | dansmith | mriedem: yeah it's definitely broken | |
| 14:09:01 | bauzas | mriedem: not forced | |
| 14:09:07 | dansmith | mriedem: okay then I'll review | |
| 14:09:09 | bauzas | mriedem: just 'passed' | |
| 14:09:25 | bauzas | dansmith: mriedem: I need to write a change, should be quick | |
| 14:09:47 | openstack | Launchpad bug 1702454 in OpenStack Compute (nova) "Transforming the RequestSpec object into legacy dicts doesn't support the requested_destination field" [High,Confirmed] - Assigned to Sylvain Bauza (sylvain-bauza) | |
| 14:09:47 | bauzas | dansmith: context is https://bugs.launchpad.net/nova/+bug/1702454 | |
| 14:09:57 | mriedem | ok i guess i don't know how the evacuate api handles passing a host and forcing a host differently | |
| 14:10:15 | bauzas | mriedem: that's simple | |
| 14:10:47 | bauzas | since 2.29 (or 2.30, don't remember which one), if you pass a target to the API, then we verify that target by the scheduler | |
| 14:11:08 | bauzas | unless you pass a specific parameter to the API, namely 'force' | |
| 14:11:37 | bauzas | in that case, we fully bypass the scheduler call and we directly cast the compute service with the hostname | |
| 14:11:42 | sean-k-mooney | stephenfin: without designate im not sure how you can have instance use teh neutron dns_domain unless we add it to vif_binding details in some way. | |
| 14:13:39 | mriedem | ok there was something else we found when investigating that other evacuate bug, | |
| 14:14:03 | stephenfin | sean-k-mooney: We are able to fetch that from neutron though, right? https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L1131-L1133 | |
| 14:14:09 | mriedem | was we can pass force=None here https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/evacuate.py#L120 | |
| 14:14:34 | mriedem | but the compute api assumes it's a boolean https://github.com/openstack/nova/blob/master/nova/compute/api.py#L4170 | |
| 14:14:42 | mriedem | but since it can be None that's not quite right | |
| 14:15:11 | openstack | bug 1658070 in OpenStack Compute (nova) newton "Failed SR_IOV evacuation with host" [High,In progress] https://launchpad.net/bugs/1658070 - Assigned to Matt Riedemann (mriedem) | |
| 14:15:11 | mriedem | anyway, we didn't fix that as part of bug 1658070 since the claim issue was fixed in the compute | |
| 14:15:22 | mriedem | i have to take maya to daycare | |
| 14:15:26 | mriedem | bbiab | |
| 14:15:35 | sean-k-mooney | stephenfin: you can but it would be good to include this in the os-vif network object so that you can jsut read it from there. that would require neutron to populated it in the vif binding details | |
| 14:16:43 | stephenfin | sean-k-mooney: Yeah, that's a good idea. I'll take a look and see how awkward it is | |
| 14:17:54 | sean-k-mooney | stephenfin: well this is what you would need to extend in os-vif https://github.com/openstack/os-vif/blob/master/os_vif/objects/network.py#L28-L39 | |
| 14:18:59 | sean-k-mooney | stephenfin: this is the neutron extention that provides access to the dns name https://github.com/openstack/neutron-lib/blob/master/neutron_lib/api/definitions/dns_domain_ports.py | |