| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-11 | |||
| 15:37:19 | bauzas | figleaf: dtantsur: cdent: appreciated, both your warm comments and your help during that very difficult release | |
| 15:37:28 | bauzas | see ya | |
| 15:38:20 | bauzas_afk | figleaf: uh, actually, thanks for the reminder, will remove my -W now | |
| 15:42:50 | figleaf | bauzas_afk: thx | |
| 15:49:37 | openstackgerrit | Merged openstack/nova master: doc: add another up-call caveat for cells v2 for xenapi aggregates https://review.openstack.org/493006 | |
| 15:50:22 | openstackgerrit | Merged openstack/nova master: doc: add superconductor up-call caveat for cross_az_attach=False https://review.openstack.org/493007 | |
| 15:51:12 | mriedem | dtantsur: to stable/pike? | |
| 15:51:24 | dtantsur | mriedem: yeah | |
| 15:51:56 | mriedem | um, idk | |
| 15:52:01 | mriedem | didn't realize that merged | |
| 15:52:56 | dtantsur | the problem is that these filters will be unusable with ironic in Queens | |
| 15:53:07 | dtantsur | so deprecating them in Pike is IMO fair | |
| 15:53:32 | dtantsur | I've proposed the backport in https://review.openstack.org/#/c/493052/ and will leave it up to you | |
| 15:54:37 | mriedem | dtantsur: i see in the ironic release notes that resource_class can't be changed on an active node https://docs.openstack.org/releasenotes/ironic/unreleased.html | |
| 15:54:43 | mriedem | but don't see anything saying resource_class is required | |
| 15:55:08 | dtantsur | mriedem: it's on the way: https://review.openstack.org/491773 | |
| 15:55:12 | mriedem | and wouldn't setting a resource_class be required for compute scheduling? or maybe that's just a requirement on the nova side, and not for standalone ironic | |
| 15:55:16 | mriedem | ah | |
| 16:01:35 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Update api doc with latest updates in api framework https://review.openstack.org/493041 | |
| 16:04:02 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: test shelve and shelve offload with placement https://review.openstack.org/493062 | |
| 16:06:20 | gibi | cdent: bug for the shelve offload problem: https://bugs.launchpad.net/nova/+bug/1710249 | |
| 16:06:21 | openstack | Launchpad bug 1710249 in OpenStack Compute (nova) "nova doesn't clean up the resources after shelve offload" [Undecided,New] | |
| 16:06:40 | gibi | cdent: also pushed a regression test for it https://review.openstack.org/#/c/493062/ | |
| 16:07:54 | gibi | cdent: just regular caffeine :) | |
| 16:08:21 | gibi | but now it is time to leave for the weekend. See you on Monday | |
| 16:27:37 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add ``HostPortProfileInfo`` class https://review.openstack.org/441590 | |
| 16:28:45 | mriedem | sdague: i'm going to grind through stable branch reviews today before i'm out for a week, can you hit this series? https://review.openstack.org/#/q/topic:bug/1686116+branch:stable/ocata | |
| 16:32:58 | mriedem | another stable/ocata one that needs another stable core to push through https://review.openstack.org/#/c/475902/ | |
| 16:32:59 | mriedem | claudiub: ^? | |
| 16:35:32 | mriedem | another one that needs final +2 https://review.openstack.org/#/q/topic:bug/1607714+branch:stable/ocata | |
| 16:43:37 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Skip test_rebuild_server_in_error_state for cells v1 https://review.openstack.org/493076 | |
| 16:50:09 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Update api doc with latest updates in api framework https://review.openstack.org/493041 | |
| 16:52:44 | mriedem | cdent: fun catch on that one https://review.openstack.org/#/c/492477/ | |
| 16:52:59 | mriedem | would be a reasonable thing if they changed it into a policy rule | |
| 16:53:04 | mriedem | and did it on master | |
| 16:53:26 | cdent | mriedem: I’m guess they just don’t know the process | |
| 16:53:39 | mriedem | yeah probably not | |
| 16:54:04 | mriedem | https://docs.openstack.org/project-team-guide/stable-branches.html#appropriate-fixes huzzah | |
| 16:54:07 | mriedem | i'll link that in there | |
| 16:54:20 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add __str__ method to Host* objects https://review.openstack.org/493082 | |
| 17:10:13 | gmann | mriedem: sdague alex_xu can you check this. API doc updated with new routing things -https://review.openstack.org/#/c/493041/ | |
| 17:10:56 | mriedem | i think oomichi said he wanted to review that | |
| 17:11:02 | mriedem | as my proxy | |
| 17:11:08 | gmann | after this i will merge api_v2.rst into api.rst | |
| 18:52:38 | annabelleB | hey Nova team—Anne Bertucio from the Foundation here. We’re trying to confirm some features that the Community Roadmap team had highlighted for Pike. Did the allowing multiple cells v2 feature land in Pike? | |
| 19:02:06 | mriedem | annabelleB: i need to get back to your email don't i :) | |
| 19:02:41 | mriedem | annabelleB: the prelude to the pike rc1 release notes has the major stuff https://docs.openstack.org/releasenotes/nova/pike.html | |
| 19:02:48 | mriedem | multi-cell support landed but with caveats | |
| 19:03:00 | mriedem | documented here https://docs.openstack.org/nova/latest/user/cellsv2_layout.html#caveats-of-a-multi-cell-deployment | |
| 19:03:47 | annabelleB | mriedem: hehe, there was a lot of excitement around this, so wanted to make sure I got it in :) | |
| 19:03:56 | annabelleB | mriedem: perfect; I’ll read that! thank you! | |
| 19:04:13 | mriedem | placement is a more confusing and less well documented story probably | |
| 19:34:52 | sdague | mriedem: I'm going to drop off before too long, anything else you need me to look at or do before then? | |
| 19:36:35 | mriedem | i've just been going through stable/newton now | |
| 19:36:42 | mriedem | there are several ready for +W | |
| 19:36:51 | mriedem | https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/newton | |
| 19:42:07 | sdague | mriedem: https://review.openstack.org/#/c/487536/ doesn't really hurt anything to bring it back | |
| 19:42:28 | mriedem | it's specifically for another change that needs to be put on top of it | |
| 19:42:34 | mriedem | which still needs work in ocata | |
| 19:42:47 | mriedem | i didn't -1 it, just no reason to merge it yet | |
| 19:43:18 | sdague | yeh, it just seems fine to merge it as it doesn't hurt anything, and just makes other fixes potentially easier to pull back | |
| 19:45:23 | sdague | https://review.openstack.org/#/c/470347 and stack seems pretty extensive for newton | |
| 19:45:43 | mriedem | it wasn't 6 months ago | |
| 19:45:55 | mriedem | well, | |
| 19:46:02 | mriedem | so that's for re-enabling a tempest test | |
| 19:46:09 | sdague | jun 2 | |
| 19:46:11 | mriedem | https://review.openstack.org/#/c/349011/ | |
| 19:46:20 | sdague | ok | |
| 19:46:22 | mriedem | yeah, i started backporting that to newton once it was fixed in ocata and passing https://review.openstack.org/#/c/349011/ | |
| 19:46:28 | mriedem | so we can't do https://review.openstack.org/#/c/349011/ w/o it being fixed in all branches | |
| 19:47:09 | sdague | ah | |
| 19:47:47 | sdague | ok, I guess you sold me | |
| 19:47:54 | mriedem | yeah, been working on that since i was at ibm public cloud :) | |
| 19:47:59 | mriedem | which was last summer | |
| 19:49:24 | sdague | heh | |
| 20:18:17 | mriedem | vdrok: do we have a good ci run for ironic + resource classes with https://review.openstack.org/#/c/492964/ ? | |
| 20:21:44 | vdrok | mriedem: not sure if dtantsur|afk made a test patch with depends on. Can do in ~an hour when I get to keyboard | |
| 20:27:00 | mriedem | i think it's probably https://review.openstack.org/#/c/476968/ ? | |
| 20:30:38 | openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: Add more logging + hook devstack https://review.openstack.org/492957 | |
| 20:30:46 | mriedem | vdrok: fyi i think ^ will hit the entire thing | |
| 20:31:02 | mriedem | the flavor migration + dtantsur|afk's inventory fix + the ironic+devstack changes to use resource classes | |
| 20:32:18 | vdrok | mriedem: yes, but the actual flavor migration won't happen, as we can only see it in grenade I think | |
| 20:32:38 | mriedem | ah because the node doesn't have the resource_class set to migrate | |
| 20:33:25 | vdrok | mriedem: Ive done this one I1ac1b8501cbed901941e7c89fc55c5b9bea0cb4f but it does not seem to work yet | |
| 20:33:30 | oomichi | toabctl: commented on https://review.openstack.org/#/c/398308 | |
| 20:34:28 | vdrok | mriedem: with that, we'd have one instance created by Nova in resource create phase, and we'd update it's resource class | |
| 20:34:51 | vdrok | *Its node's class | |
| 20:35:22 | mriedem | ok i've approved the flavor migration patch anyway | |
| 20:35:29 | mriedem | i think you guys have shown we're good there | |
| 20:35:39 | vdrok | Thanks! | |
| 20:35:45 | mriedem | the remaining issue was the scheduling piece with the custom resource class missing, which is dtantsur|afk's patch | |
| 20:36:17 | figleaf | \o/ | |
| 20:42:58 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle addition of new nodes/instances in ironic flavor migration https://review.openstack.org/487954 | |
| 20:43:14 | mriedem | whoa wt | |
| 20:43:25 | mriedem | oh gdi | |
| 20:43:28 | mriedem | we don't have git review setup yet | |
| 20:43:34 | mriedem | on stable/pike | |
| 20:45:51 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle addition of new nodes/instances in ironic flavor migration https://review.openstack.org/487954 | |
| 20:49:34 | mriedem | ok here we go https://review.openstack.org/#/c/493227/ | |
| 20:54:38 | sdague | mriedem: I approved the patch, it just hasn't landed yet | |