| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-28 | |||
| 12:17:55 | lucasagomes | hahah apparently so :-P | |
| 12:17:56 | vdrok | pretty good :) | |
| 12:18:04 | dtantsur | yes, it's going to end with ironic merging back into nova-baremetal :D | |
| 12:18:33 | hrw | hi | |
| 12:19:09 | openstackgerrit | Alex Szarka proposed openstack/nova master: Add method for verify multiple versioned notifications https://review.openstack.org/465526 | |
| 12:20:42 | hrw | sorry to bother but could someone take a look at https://review.openstack.org/#/c/489951/ and help us get aarch64 VMs booting out of the box without having to *always* set hw_firmware_type=uefi property? | |
| 12:32:02 | openstackgerrit | Alex Szarka proposed openstack/nova master: Add method for verify multiple versioned notifications https://review.openstack.org/465526 | |
| 12:32:23 | openstackgerrit | Alex Szarka proposed openstack/nova master: Refactor init_instance:resume_guests_state https://review.openstack.org/349582 | |
| 12:35:21 | openstackgerrit | Alex Szarka proposed openstack/nova master: Reduce code complexity - manager.py https://review.openstack.org/359868 | |
| 12:35:50 | openstackgerrit | Alex Szarka proposed openstack/nova master: Reduce code complexity - linux_net.py https://review.openstack.org/359871 | |
| 12:42:13 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Add functional resize tests using shared storage https://review.openstack.org/490733 | |
| 12:42:20 | sdague | mikal: looks like your patch series needs some rebasing | |
| 12:48:15 | efried | jaypipes I want to start putting these ideas into a blueprint to socialize over the next couple weeks before PTG. Work for you if I use https://blueprints.launchpad.net/nova/+spec/devices-as-resources or did you have another already started? | |
| 12:49:07 | stephenfin | jaypipes, sdague: Could we get this in today? https://review.openstack.org/#/c/479869/ | |
| 12:50:52 | sdague | stephenfin: +2 | |
| 12:54:20 | stephenfin | ta | |
| 12:55:00 | hrw | efried: good point with setting hw_firmware_type=uefi only if unset | |
| 12:55:10 | efried | hrw :) | |
| 12:55:16 | hrw | efried: in case someone one day invent other way for aarch64 | |
| 12:55:20 | efried | Right. | |
| 12:55:42 | efried | I'm generally not a fan of implementing-for-future, but in this case it's just as easy and arguably more correct. | |
| 12:56:20 | efried | hrw Also I suspect set-if-None will satisfy your other -1er. | |
| 12:56:30 | hrw | efried: and for us whatever gets accepted as long as it will work ootb | |
| 12:56:47 | efried | Suresure. | |
| 12:57:03 | efried | hrw Is there a bug associated with this? | |
| 12:57:25 | hrw | efried: would have to check as I have no idea. | |
| 12:57:45 | hrw | efried: noticed that it still is in queue when looked at my list of patches to/in review | |
| 12:57:59 | efried | I have no skin in the game; just seemed like it might. If you want to backport it to stable/pike, you'll need one anyway. | |
| 12:58:21 | hrw | https://bugs.launchpad.net/nova will be right place? | |
| 12:58:25 | openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Remove custom comparison methods https://review.openstack.org/472285 | |
| 12:58:27 | efried | Yes. Where did this behavior come into existence? Sounds like some time after mitaka? | |
| 12:58:41 | hrw | efried: yes, Mitaka added uefi support for aarch64 | |
| 12:58:48 | hrw | before it was kernel+initrd only | |
| 13:00:34 | openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: De-duplicate some graphics tests https://review.openstack.org/497969 | |
| 13:07:38 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Account for shared storage in the report client https://review.openstack.org/491098 | |
| 13:12:52 | efried | sdague Quick +W on https://review.openstack.org/#/c/475810/ and https://review.openstack.org/#/c/476159/ ? (https://review.openstack.org/#/c/476204/ already primed) | |
| 13:14:02 | jaypipes | stephenfin: done | |
| 13:14:10 | jaypipes | efried: sorry, been in 3 hour meeting... | |
| 13:14:23 | jaypipes | efried: yeah, go for it. that's as good a name as any :) | |
| 13:14:26 | hrw | ok, mail to Kevin sent | |
| 13:14:36 | efried | jaypipes Roger wilco. | |
| 13:14:38 | jaypipes | efried: and yeah, I appreciate you collating info on that | |
| 13:14:47 | efried | But of course | |
| 13:14:49 | jaypipes | efried: feel free to get an etherpad started too if you want | |
| 13:15:10 | efried | jaypipes You mean split out from the main PTG one? | |
| 13:15:31 | efried | Can do | |
| 13:15:57 | jaypipes | efried: yup. I presume we'll have a session on that | |
| 13:16:06 | jaypipes | efried: and a separate etherpad would be good to start on | |
| 13:18:43 | openstackgerrit | Merged openstack/os-traits master: doc: Switch from oslosphinx to openstackdocstheme https://review.openstack.org/479869 | |
| 13:19:20 | openstackgerrit | Merged openstack/nova master: Refactor libvirt.utils.execute() away. https://review.openstack.org/489816 | |
| 13:25:36 | openstackgerrit | Merged openstack/os-traits master: doc: Remove cruft from conf.py https://review.openstack.org/480090 | |
| 13:26:20 | sdague | efried: looking | |
| 13:27:13 | sdague | efried: done | |
| 13:27:18 | efried | sdague Thanks! | |
| 13:30:08 | openstackgerrit | Eric Fried proposed openstack/nova master: Update PCI passthrough doc for moved options https://review.openstack.org/498461 | |
| 13:30:47 | edleafe | Scheduler subteam meeting in 30 minutes in #openstack-meeting-alt | |
| 13:30:49 | efried | stephenfin ^ split out from https://review.openstack.org/#/c/493701/ as requested, and filed against a bug (https://bugs.launchpad.net/nova/+bug/1713502) | |
| 13:30:50 | openstack | Launchpad bug 1713502 in OpenStack Compute (nova) "PCI passthrough documentation needs updating since options moved to [pci] section" [Undecided,In progress] - Assigned to Eric Fried (efried) | |
| 13:31:18 | efried | Not sure who can answer whether that should be backported to at least pike | |
| 13:31:28 | efried | (Backport to ocata would be a non-cherry-pick, cause docs all moved) | |
| 13:32:28 | stephenfin | efried: Yeah, iirc they didn't version the configuration guide so there's nothing to backport to for Ocata | |
| 13:32:49 | efried | okay. The deprecated options still work, so it's not crucial. | |
| 13:33:04 | stephenfin | I think doc patches should go back to Pike though. Maybe sdague would have some thoughts on the matter? | |
| 13:33:11 | stephenfin | Oh, so it would actually | |
| 13:33:52 | sdague | stephenfin: which doc patches? | |
| 13:34:03 | stephenfin | sdague: https://review.openstack.org/#/c/498461/1 | |
| 13:34:17 | openstackgerrit | Eric Fried proposed openstack/nova master: [Trivial] docstrings, typos, minor refactoring https://review.openstack.org/493701 | |
| 13:34:47 | efried | stephenfin Rebased ^ on top | |
| 13:35:09 | sdague | yeh, admin guide stuff should go back as it's all published per release | |
| 13:35:38 | stephenfin | efried: Odd. I expected the +2 to disappear. Apparently not | |
| 13:36:15 | efried | stephenfin When gerrit recognizes an auto rebase, it carries code reviews forward. | |
| 13:36:23 | efried | Which is *almost* always okay :) | |
| 13:38:49 | efried | stephenfin sdague https://review.openstack.org/#/c/498463/ is the pike cherry-pick. So should I go make the change in ocata too? | |
| 13:40:06 | sdague | efried: no, the admin guide is not in ocata | |
| 13:40:20 | efried | sdague Okay, that's why I can't find it :) | |
| 13:40:24 | sdague | yep | |
| 13:40:26 | efried | That doc must have existed somewhere? | |
| 13:40:30 | efried | Just in spec form mebbe? | |
| 13:40:34 | sdague | on the openstack manuals side | |
| 13:41:07 | sdague | only in pike did the manuals go in tree for nova | |
| 13:42:12 | efried | sdague Right; so my question is whether I should go find and update that manual wherever it lived in ocata | |
| 13:44:15 | stephenfin | efried: I don't think they versioned the admin guide either, so there's nothing to backport to for Ocata | |
| 13:44:26 | sdague | yeh, I'm not sure I would bother | |
| 13:44:30 | efried | cool cool. | |
| 13:58:47 | mriedem | gmann: i'm not sure https://review.openstack.org/#/c/490722/ warrants a microversion bump | |
| 13:58:56 | mriedem | should probably be discussed in the nova-api meeting this week | |
| 14:00:01 | cdent | mriedem: i rebased your shared providers functional test again. same deal as last time: leaving the failing tests with TODOs. I did make a couple of tweaks to report client in the patch above it to get a few more things passing | |
| 14:01:53 | mriedem | cdent: that was pretty incomplete from what i remember | |
| 14:02:11 | mriedem | i know i need to abandon the wip for the fixes in the report client | |
| 14:02:26 | cdent | It was but there were a couple of simple syntax thingies that move it forward a bit | |
| 14:21:56 | sdague | mriedem: https://review.openstack.org/#/c/496515 is probably worthy of backport | |
| 14:22:31 | sdague | I push for pike, what do you think about ocata as well? | |
| 14:23:27 | kashyap | sdague: Randomly chiming in, it sounds like backport-worthy, given the description | |
| 14:24:20 | mriedem | seems pretty low probability | |
| 14:24:46 | mriedem | wouldn't this be someone creating 2 instances with the same name on separate requests ~the same time? | |
| 14:24:56 | sdague | mriedem: no | |
| 14:25:11 | sdague | there is no name uniqueness requirement | |
| 14:25:24 | sdague | or, it's behind a config option, off by default | |
| 14:25:41 | sdague | so, it is 2 different requests, but they can be at any time | |
| 14:26:26 | mriedem | as long as you didn't delete the other first? | |