| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-05 | |||
| 16:28:22 | mriedem | it used it for some other stuff too, related to the adminPass | |
| 16:29:23 | mriedem | reading the spec https://specs.openstack.org/openstack/nova-specs/specs/mitaka/implemented/optional-shared-storage-flag-in-evacuate-api.html | |
| 16:30:02 | openstackgerrit | Eric Fried proposed openstack/nova master: placement doc: Conflict caveat for DELETE APIs https://review.openstack.org/540919 | |
| 16:30:16 | efried | cdent, edleafe: ^ For the remainder of bug 1746373 | |
| 16:30:17 | openstack | bug 1746373 in OpenStack Compute (nova) "Placement APIs with missing conflict detection" [Undecided,In progress] https://launchpad.net/bugs/1746373 - Assigned to Eric Fried (efried) | |
| 16:30:30 | efried | jaypipes also ^ | |
| 16:32:03 | mriedem | dansmith: fine to just leave it in for compat | |
| 16:39:15 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Rename the 'recreate' param to rebuild_instance to 'evacuate' https://review.openstack.org/508190 | |
| 16:39:54 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Rename the 'recreate' param in rebuild_instance to 'evacuate' https://review.openstack.org/508190 | |
| 16:47:42 | hrw | stephenfin: what is docstring? | |
| 16:47:52 | openstackgerrit | Merged openstack/python-novaclient master: nova limits ERROR (Exception): Field names must be unique https://review.openstack.org/527922 | |
| 16:47:53 | openstackgerrit | Merged openstack/python-novaclient master: Implement hypervisor hostname exact pattern match https://review.openstack.org/520187 | |
| 16:49:38 | hrw | ah. """ D: | |
| 16:50:36 | dansmith | mriedem: ack | |
| 16:52:40 | mriedem | artom: https://review.openstack.org/#/c/508190/2/nova/compute/manager.py@2663 | |
| 16:53:13 | artom | mriedem, ah, doh | |
| 16:53:28 | artom | mriedem, well, my second one still kinda stands :P | |
| 16:53:33 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: Make sure that we have usable input for graphical console https://review.openstack.org/538003 | |
| 16:53:37 | hrw | stephenfin: as you wish ;D | |
| 16:53:56 | openstackgerrit | Merged openstack/python-novaclient master: Add CLI to show instance usage audit logs https://review.openstack.org/409413 | |
| 16:54:02 | artom | I'll remove the -1 though | |
| 16:57:01 | stephenfin | hrw: ++ | |
| 16:59:29 | hrw | mriedem, jaypipes, sahid: can you take another look at https://review.openstack.org/538003 patch? it is simplest of all versions and comes with extra test | |
| 17:01:06 | stephenfin | bauzas: Gentle reminder for https://review.openstack.org/#/c/530743/ | |
| 17:03:26 | mriedem | stephenfin: see the latest comment there, it introduces a bug | |
| 17:04:04 | mriedem | so a big -1 from me on the backport | |
| 17:04:30 | jaypipes | hrw: I never saw a response from you to sahid's review comments? | |
| 17:05:04 | hrw | jaypipes: rewrote whole as an answer | |
| 17:05:33 | stephenfin | mriedem: Yup, I was focussing on the latter part of the comment though: a partial fix was better than no fix | |
| 17:05:37 | jaypipes | hrw: heh, ok :) | |
| 17:05:56 | jaypipes | hrw: I don't see a followup from sahid which is why I asked.. | |
| 17:06:03 | stephenfin | Probably best to wait for the fixes' fix to be merged though | |
| 17:06:06 | hrw | jaypipes: sure | |
| 17:07:17 | hrw | jaypipes: when 'a reply' would be just message then I write it. If it is simple fix then I mark Done' | |
| 17:07:43 | hrw | jaypipes: if it is complicated then do code mostly | |
| 17:08:09 | hrw | jaypipes: and here comment was more or less 'rewrite the shit' so I did | |
| 17:11:57 | jaypipes | hrw: understood. I'd still like to have sahid's opinion on it, though, since he had the biggest objection to it.. | |
| 17:12:04 | hrw | sure | |
| 17:28:50 | hrw | I have an idea for another 'lets add some device' patch just not found yet where to plug it | |
| 17:29:36 | hrw | on pcie machines (aarch64, x86/q35) more pcie-root-port controllers can be handy if someone wants to add more disks/networks | |
| 17:34:35 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add nova-status check for ironic flavor migration https://review.openstack.org/527541 | |
| 17:34:39 | openstackgerrit | Eric Fried proposed openstack/nova master: placement doc: Conflict caveat for DELETE APIs https://review.openstack.org/540919 | |
| 17:37:47 | hrw | it is like: basic VM instance config gets created with (let's say) 4 pcie root ports (think of them as pcie slot on virtual motherboard). then we plug video card, usb host controller (and usb keyboard to it) so we have just 2 slots left. then we add RNG and memory balloon... hm. have to boot an instance and check | |
| 17:44:19 | openstackgerrit | Merged openstack/nova master: Update cells v2 layout doc caveats for Queens https://review.openstack.org/540667 | |
| 17:44:29 | openstackgerrit | Merged openstack/nova stable/ocata: libvirt: Re-initialise volumes, encryptors, and vifs on hard reboot https://review.openstack.org/531422 | |
| 17:49:28 | hrw | arm64 guest has 1 pcie slot free when booted with graphical console | |
| 17:59:54 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add late server group policy check to rebuild https://review.openstack.org/525242 | |
| 18:00:49 | gibi | mriedem: fixed the bugfix patch as well ^^ | |
| 18:01:42 | mriedem | ok | |
| 18:01:48 | mriedem | will take a look after lunch | |
| 18:02:57 | gibi | mriedem: have nice lunch | |
| 18:03:50 | gibi | mriedem: I'm no checking your idea to use wait_for_versioned_notification instead of the for loop and the sleep int he functional test, but I have to leave soon so cannot promise I finish with that today | |
| 18:04:17 | mriedem | that's fine, it can be a later cleanup | |
| 18:05:03 | gibi | mriedem: OK | |
| 18:08:53 | mriedem | cfriesen: i have an idea about the perf hit in pike, | |
| 18:09:04 | mriedem | maybe versioned notifications? | |
| 18:09:09 | mriedem | cfriesen: are you running with those on? | |
| 18:09:46 | mriedem | default is to send both versioned and unversioned notifications, | |
| 18:10:00 | mriedem | and the versioned notifications have to sometimes lazy-load fields on the instance for the notification payload | |
| 18:10:05 | mriedem | which means more round trips over rpc to the db | |
| 18:12:24 | jaypipes | hrw: reviewed. | |
| 18:15:00 | cfriesen | mriedem: will check | |
| 18:15:56 | hrw | jaypipes: reading | |
| 18:18:11 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add regression test for bug 1735407 https://review.openstack.org/526095 | |
| 18:18:12 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add late server group policy check to rebuild https://review.openstack.org/525242 | |
| 18:18:12 | openstack | bug 1735407 in OpenStack Compute (nova) "[Nova] Evacuation doesn't respect anti-affinity rules" [Medium,In progress] https://launchpad.net/bugs/1735407 - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 18:18:47 | cfriesen | mriedem: looks like explicitly unversioned. thanks for the suggestion though. | |
| 18:19:25 | gibi | mriedem: managed to remove the sleep from the functional test. | |
| 18:19:32 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: Make sure that we have usable input for graphical console https://review.openstack.org/538003 | |
| 18:19:59 | hrw | jaypipes: thanks | |
| 18:21:53 | mriedem | cfriesen: damn | |
| 18:27:56 | dansmith | mriedem: so, we have some calls like build_and_run_instances() that has basically all optional arguments, even though they're not really optional | |
| 18:28:20 | dansmith | mriedem: we're removing one in the middle of those due to deprecations, so I was thinking I'd remove the optional/kwarg-ness from them all, | |
| 18:28:30 | dansmith | but that broke a bunch of tests that use the optionalness | |
| 18:28:48 | dansmith | I kinda think we might just want to take the hit now and fix that in the process, but do you have a strong opinion either way? | |
| 18:29:12 | mriedem | i was going to say it might not be worth the trouble to refactor all of that | |
| 18:29:26 | mriedem | but it's not a strong opinion on | |
| 18:29:28 | mriedem | *no | |
| 18:29:32 | dansmith | okay | |
| 18:30:30 | dansmith | well, I guess I'll just be lazy and not do it then | |
| 18:35:56 | jaypipes | hrw: +W | |
| 18:36:03 | hrw | thx | |
| 18:36:20 | openstackgerrit | Yuki Nishiwaki proposed openstack/nova master: Check enable_new_services option in service.py https://review.openstack.org/540962 | |
| 18:44:49 | openstackgerrit | Chris Dent proposed openstack/nova-specs master: Add generation support in aggregate association https://review.openstack.org/540447 | |
| 18:58:08 | mriedem | dansmith: i'm ok with lazy, cleaning up cruft and dropping deprecated stuff is good for me | |
| 18:58:15 | mriedem | given we're only a few days out from rc1 | |
| 18:58:27 | dansmith | ack, yeah I left it alone | |
| 18:58:41 | dansmith | I'm working on all the signature changes for the other stuff in the tests right now | |
| 19:06:56 | hrw | LibvirtConfigGuestPCIeRootPortController is my next class to add. but not today. | |
| 19:18:27 | openstackgerrit | Merged openstack/nova stable/pike: Import the config drive docs from openstack-manuals https://review.openstack.org/515053 | |
| 19:18:41 | ameeda | gibi and mriedem Thank you very much, I will upload new patch soon | |
| 19:20:31 | dansmith | mriedem: I already made that cleanup change for a couple of the other calls, which I'm paying for in tests right now, but they're less invasive than build | |
| 19:21:59 | ameeda | mriedem: can you please confirm this for me ? https://review.openstack.org/#/c/540274/ its nova-manager doc | |
| 19:30:17 | sean-k-mooney | dansmith: mriedem so i found a bug at the end of last week that has been in the nova libvirt forever https://bugs.launchpad.net/nova/+bug/1747496 | |
| 19:30:18 | openstack | Launchpad bug 1747496 in OpenStack Compute (nova) "MTUs are not set for VIFs if using kernel ovs + hybrid plug = false" [Undecided,New] | |
| 19:31:16 | sean-k-mooney | dansmith: mriedem this is not new in queens but my question is the fix is simple. just set the mtu in the libvirt xml or alternitivly use os-vif which is a little more work | |
| 19:31:50 | sean-k-mooney | dansmith: mriedem will i trow a path up for rocky or is this something ye want to backport? | |
| 19:33:45 | openstackgerrit | Ameed Ashour proposed openstack/nova master: detach instance volumes when VM creation fails https://review.openstack.org/528385 | |
| 19:34:19 | sean-k-mooney | using os-vif is not backportable hence why i am leaning towords adding the libvirt element for the mtu. we can always delegate entirly to os-vif in rocky if we like but just wonderin if we want to fix this in queens? | |
| 19:36:06 | openstackgerrit | Merged openstack/nova stable/ocata: Set server status to ERROR if rebuild failed https://review.openstack.org/536904 | |