| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-11 | |||
| 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 | |
| 20:55:00 | sdague | https://review.openstack.org/#/c/492980/ | |
| 20:56:55 | mriedem | yup | |
| 20:56:59 | mriedem | i backported on top of that series | |
| 20:59:57 | mriedem | ok i've sent the todo list to dansmith publicly in the ML | |
| 21:00:07 | mriedem | i will now retire with a bag of chips to my basement to prepare for vacation | |
| 21:04:35 | sdague | heh | |
| 21:04:57 | sdague | mriedem: go find yourself a saspirila | |
| 21:05:11 | mriedem | i do have a bottle of fancy root beer in the fridge | |
| 21:13:11 | mriedem | melwitt: can you hit the two bottom changes here? https://review.openstack.org/#/c/450105/ - just small placement api-ref changes, then we'll have the job running | |
| 21:13:30 | melwitt | mriedem: sure thing | |
| 21:13:34 | mriedem | thanks | |
| 21:33:34 | openstackgerrit | Eric Fried proposed openstack/nova master: nova.utils.get_ksa_adapter() https://review.openstack.org/488137 | |
| 21:33:34 | openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa adapter for placement conf & requests https://review.openstack.org/492247 | |
| 21:48:46 | mtreinish | fried_rice: I pushed a new rev on: https://review.openstack.org/#/c/492651/3 which now has a depends on for: https://review.openstack.org/#/c/493231/2 | |
| 21:49:00 | mtreinish | if you wanted to give it a try | |
| 21:49:04 | fried_rice | mtreinish Ack, looking... | |
| 21:49:17 | fried_rice | mtreinish I have a nice stable (mod_wsgi) stack at the moment, so... | |
| 21:51:31 | mtreinish | fried_rice: hmm, well if you don't want to rock the boat. I'm still testing things locally | |
| 21:51:42 | fried_rice | rgr | |
| 21:51:58 | fried_rice | mtreinish Storing the whole image in memory... doesn't sound like it'll fly. | |
| 21:52:21 | mtreinish | it definitely works when I talk to the uwsgi server directly, but it looks like the apache config still needs some tweaking because I'm getting a 502 going through the proxy | |
| 21:52:40 | fried_rice | mtreinish Here's where it would be nice to be able to use a fifo, where glance can suck it down in parallel with the send. | |
| 21:53:31 | fried_rice | mtreinish swhat we do in the powervm driver when populating a boot disk *from* glance. | |
| 21:53:42 | mtreinish | yeah, I'm still trying to figure out the best way to handle that. I was just basing that middleware on what I found in the eventlet code | |
| 21:54:08 | mtreinish | and I still ahven't figured out how eventlet is doing it | |
| 21:54:59 | mtreinish | fried_rice: the trick here is to make the data look like what webob is expecting after calling uwsgi.chunked_read() but not putting it all in memory | |
| 21:55:32 | fried_rice | mtreinish Can you point me to the source for chunked_read? | |
| 21:55:57 | mtreinish | sure, one sec | |
| 21:56:20 | fried_rice | Could it be this? https://github.com/unbit/uwsgi/blob/master/core/chunked.c#L86 | |
| 21:57:17 | mtreinish | yeah that's the inner c function | |
| 21:57:22 | mtreinish | the python interface is defined here: https://github.com/unbit/uwsgi/blob/master/plugins/python/uwsgi_pymodule.c#L1185-L1200 | |