| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-11 | |||
| 15:12:55 | mtreinish | fried_rice: well I at least figured out why I thought it worked before. The config I have up there now drops the chunked data on the floor. (when you talk to uwsgi directly instead of going through the proxy) | |
| 15:13:14 | mtreinish | so the request gets a good response, but the data isn't actually used | |
| 15:13:31 | fried_rice | mtreinish Hah, so I would get a zero-size image, or better yet, a full-sized image full of zeros? | |
| 15:13:46 | mtreinish | heh, it's a zero size image | |
| 15:14:16 | mtreinish | I think the 502s from apache are it realizes the request isn't being properly handled and bails out | |
| 15:18:34 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Update api doc with latest updates in api framework https://review.openstack.org/493041 | |
| 15:21:04 | vdrok | sdague: mriedem could you also take a look at https://review.openstack.org/492579 if you have a bit of time? This is fixing https://bugs.launchpad.net/nova/ocata/+bug/1706772 we are seeing sometimes on the ocata grenade job, as placement start triggers reload of apache, so keystone becomes inaccessible and n-cpu fails to start | |
| 15:21:05 | openstack | Launchpad bug 1706772 in OpenStack Compute (nova) ocata "InternalServerError: Internal Server Error (HTTP 500) in n-cpu logs on startup with Ironic driver" [Medium,In progress] - Assigned to Matt Riedemann (mriedem) | |
| 15:24:50 | openstackgerrit | Merged openstack/nova master: Deprecate bare metal filters https://review.openstack.org/492563 | |
| 15:25:50 | sdague | vdrok: ok, as a backport | |
| 15:26:18 | vdrok | sdague: yup, thanks! | |
| 15:27:50 | openstackgerrit | Merged openstack/nova master: Update reno for stable/pike https://review.openstack.org/492982 | |
| 15:33:26 | dtantsur | mriedem: can/should I request a backport for https://review.openstack.org/492563 now? | |
| 15:36:02 | bauzas | folks, time to hang up for a couple of weeks | |
| 15:36:28 | cdent | enjoy the time bauzas | |
| 15:36:32 | bauzas | gentle reminder, I'll be back on 31st of August | |
| 15:36:45 | figleaf | bauzas: thanks for helping with https://review.openstack.org/487954 overnight | |
| 15:36:51 | dtantsur | bauzas: enjoy! | |
| 15:36:54 | figleaf | bauzas: and have a great holiday! | |
| 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 | |