| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-25 | |||
| 18:17:14 | dansmith | I never use -y | |
| 18:17:20 | mriedem | git rebase -i HEAD~3 etc | |
| 18:17:31 | mriedem | i use git rebase -R -y quite often | |
| 18:17:31 | dansmith | because if I have messed up the rebase, I might resubmit a year's worth of changes | |
| 18:31:32 | melwitt | mriedem: I'm good with this ironic "get IP address for volume connector" change if you wanted to revisit it https://review.openstack.org/#/c/468353 | |
| 18:39:03 | mriedem | melwitt: ok | |
| 18:52:29 | hrw | hello again | |
| 18:53:00 | hrw | stephenfin: https://paste.fedoraproject.org/paste/2PNsiE0OF0VYnF~sKHdXdw is initial version of 'lets add usb host' patch | |
| 18:53:12 | hrw | stephenfin: I though it will be more complicated | |
| 18:54:53 | mriedem | melwitt: wording question in https://review.openstack.org/#/c/537642/ | |
| 18:56:01 | mriedem | edleafe: i don't think your alternate hosts change is ever going to get out of the check queue... | |
| 18:57:07 | edleafe | mriedem: maybe we need alternate CI | |
| 18:57:10 | edleafe | :) | |
| 19:01:49 | openstackgerrit | melanie witt proposed openstack/nova master: Add release note for QEMU native LUKS decryption https://review.openstack.org/537642 | |
| 19:05:06 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: [dnm] Make sure that we have USB Host controller for tablet https://review.openstack.org/538003 | |
| 19:06:25 | hrw | please take a look as I am not sure is it proper way of doing that | |
| 19:10:23 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Ironic: Get IP address for volume connector https://review.openstack.org/468353 | |
| 19:10:24 | mriedem | melwitt: check my release note update on ^ and see if that's still OK, then i'll +W | |
| 19:13:27 | melwitt | mriedem: looks good to me | |
| 19:14:27 | mriedem | want to put your +2 back on? | |
| 19:14:40 | melwitt | yep done | |
| 19:18:37 | mriedem | efried: so i should defer https://blueprints.launchpad.net/nova/+spec/granular-resource-requests to rocky right? | |
| 19:18:47 | mriedem | i think that was mentioned as a rocky thing at this point in the meeting today | |
| 19:19:26 | efried | mriedem It can't be finished in Q, because dependent on nrp alloc cands work, which probably also won't make Q. But 95% of the code is done and merged (months ago). | |
| 19:19:33 | efried | mriedem So however you want to paperwork that... | |
| 19:19:54 | mriedem | can't functionality use it yet from the API right? | |
| 19:19:57 | efried | correct | |
| 19:20:01 | mriedem | ok, then i'll defer | |
| 19:20:03 | efried | ack | |
| 19:24:53 | mriedem | mgoddard: so on the nova ironic traits stuff, | |
| 19:25:01 | mriedem | it looks like the ironic API changes are done https://review.openstack.org/#/q/topic:bug/1722194+(status:open+OR+status:merged) | |
| 19:25:18 | mriedem | and the ironicclient changes to support it are approved, but not yet released, and today is client release freeze | |
| 19:25:51 | mriedem | and nova would need to bump the minimum required version of this in global-requirements to use the new client stuff, and i think the requirements team is going to freeze the requirements repo after today too | |
| 19:26:24 | mriedem | jlvillal: are you waiting for these ironicclient changes to merge before you do the final ironicclient release for queens? ^ | |
| 19:27:02 | efried | mriedem The nova side stuff isn't all approved yet either FYI: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/ironic-driver-traits | |
| 19:27:09 | mriedem | efried: i know, | |
| 19:27:18 | efried | k :) | |
| 19:27:21 | mriedem | i'm asking because i don't want to waste my time on the nova stuff if the deps won't get merged int time | |
| 19:27:51 | mriedem | the ironic client change failed in the gate with a post failure, so it's going to have at least one recheck yet | |
| 19:28:03 | mriedem | plus a release, plus a g-r bump | |
| 19:28:13 | efried | Not happening today. | |
| 19:28:21 | efried | no chance | |
| 19:28:55 | jlvillal | mriedem, I am not sure rloo and TheJulia know more the current status than me. | |
| 19:29:08 | mriedem | ok jroll is asking in -release too | |
| 19:29:31 | TheJulia | Moving to laptop with battery power | |
| 19:31:46 | TheJulia | mriedem: thanks for the recheck | |
| 19:32:16 | TheJulia | mriedem: yes, waiting for the client changes we need to merge before cutting the release :( | |
| 19:33:27 | rloo | mriedem: the traits work will land for sure; just one client patch in gate. the rescue work has been approved but we need 2 ironic patches to land before client can land. and gate is super not-fast today as you probably know :-( | |
| 19:34:19 | mriedem | there hasn't been any review on the nova patches for rescue, so that's going to be deferred | |
| 19:35:15 | mriedem | also https://review.openstack.org/#/c/508116/ - that doesn't bump the minimum required ironic api version to 1.37 before trying to patch the node with traits | |
| 19:35:19 | mriedem | but that's required isn't it? | |
| 19:35:22 | mriedem | jroll: ^ | |
| 19:35:44 | mriedem | we'll likely need someone to work the nova patches at this point since i'm guessing mgoddard and johnthetubaguy are eating boiled meat at this point | |
| 19:36:59 | rloo | mriedem: boiled meat. yummmy. which is better, doing that or reviewing? i'll take a look at the nova patches in a few min. | |
| 19:37:08 | jroll | I can help too | |
| 19:37:35 | mriedem | well i'm assuming the mv needs to change to 1.37 since that's the ironic api version that added traits support | |
| 19:37:41 | jroll | it doesn't | |
| 19:37:44 | jroll | explaining in gerrit now | |
| 19:37:44 | mriedem | i could be wrong, there are a lot of +1s on there | |
| 19:37:46 | mriedem | ok | |
| 19:38:38 | jroll | comment is there mriedem | |
| 19:38:46 | jroll | and this is the one that needs the api version bump https://review.openstack.org/#/c/532288/9 | |
| 19:39:04 | jroll | since it's grabbing the traits from the node | |
| 19:41:19 | izombie | Trying to list servers and I'm hitting 404 and I can't understand why | |
| 19:41:24 | izombie | Any help? | |
| 19:42:31 | TheJulia | on a plus side jroll, that is the correct version :) | |
| 19:44:14 | jroll | yep! | |
| 19:45:17 | mriedem | izombie: check the nova-api logs | |
| 19:45:37 | mriedem | jroll: ah ok, thanks | |
| 19:46:44 | izombie | :) I did. Even debugged! I'm lost the route is not matched. It reaches till route middleware and then throws HTTPNotFound | |
| 19:48:58 | artom | izombie, technically I'm supposed to send you to #openstack (see /topic) | |
| 19:49:30 | artom | izombie, but, what request are you making? | |
| 19:49:30 | melwitt | mriedem: any particular reviews I can help with today? | |
| 19:49:56 | izombie | Simple nova list. And it breaks here https://github.com/openstack/nova/blob/master/nova/wsgi.py#L458 | |
| 19:50:18 | izombie | artom: thanks for not sending :P | |
| 19:50:32 | artom | izombie, well, I did, in a hint-y sort of way ;) | |
| 19:50:39 | artom | izombie, can you run the client in --debug? | |
| 19:50:53 | artom | And what are you running it against? devstack? | |
| 19:51:06 | izombie | I will try my luck there, meanwhile I thought to try my luck here as I'm out of it now | |
| 19:51:19 | mriedem | melwitt: i'm just going through this ironic driver traits series | |
| 19:51:30 | mriedem | https://review.openstack.org/#/c/508116/ | |
| 19:58:28 | izombie | guess, I won't fix it today. | |
| 19:58:50 | artom | I mean, I did ask them --debug output :/ | |
| 19:59:44 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Refactor _build_device_metadata https://review.openstack.org/533804 | |
| 19:59:45 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Consider hostdev devices when building metadata https://review.openstack.org/533805 | |
| 20:00:31 | efried | mriedem The ironic traits thing, we're trying to get it in? | |
| 20:00:59 | efried | mriedem I can probably do any last-minute changes if you didn't get another volunteer. | |
| 20:01:17 | mriedem | we'll see | |
| 20:01:25 | jroll | I'm happy to help too | |
| 20:01:25 | mriedem | i'm reviewing the changes now yes, | |
| 20:01:36 | mriedem | which requires looking at some of the provider tree stuff that already merged, like set_traits_for_provider | |
| 20:01:53 | efried | mriedem Yes, that was a prereq. Let me know if you want the exec summary on that. | |
| 20:03:03 | mriedem | i don't suppose there are any integration tests anywhere in ironic that run something that plumbs this all through | |
| 20:03:31 | mriedem | like, create a flavor with a required trait, decorate a node with it, pass the trait from the node to the resource provider in placement, etc | |
| 20:03:54 | jroll | nope | |
| 20:04:18 | jroll | not terribly hard to do other than the time crunch | |
| 20:05:06 | mriedem | you gonna tattoo a todo for yourself? | |
| 20:05:37 | jroll | I'll think about it :P | |
| 20:05:41 | mriedem | we could also have a simple functional test in-tree that has a fake virt driver that returns a fake trait, and we make sure that the RT and such creates it in placement for us | |
| 20:05:50 | jroll | I can add it to the existing job pretty easy | |