| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-11 | |||
| 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 | |
| 23:03:31 | fried_rice | sdague / leakypipes still around? | |
| #openstack-nova - 2017-08-12 | |||
| 02:50:37 | openstackgerrit | zhiguo.li proposed openstack/nova master: +1 https://review.openstack.org/493271 | |
| 02:50:49 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Update api doc with latest updates in api framework https://review.openstack.org/493041 | |
| 05:30:14 | openstackgerrit | Merged openstack/nova master: [placement] Add api-ref for RP usages https://review.openstack.org/450105 | |
| 06:03:12 | openstackgerrit | Merged openstack/nova master: test server evacuation with placement https://review.openstack.org/492548 | |
| 06:16:46 | openstackgerrit | Merged openstack/nova master: [placement] Add api-ref for allocation_candidates https://review.openstack.org/481112 | |
| 18:29:08 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: WIP - Tweak connection_info translation for the new Cinder attach/detach API https://review.openstack.org/493324 | |
| 18:29:08 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: WIP - Add attachment_complete call to volume/cinder.py https://review.openstack.org/493323 | |
| 18:29:08 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: WIP - Implement new attach Cinder flow https://review.openstack.org/330285 | |
| 18:32:37 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: WIP - Add attachment_complete call to volume/cinder.py https://review.openstack.org/493323 | |
| 18:32:38 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: WIP - Implement new attach Cinder flow https://review.openstack.org/330285 | |
| 18:32:38 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: WIP - Tweak connection_info translation for the new Cinder attach/detach API https://review.openstack.org/493324 | |
| #openstack-nova - 2017-08-13 | |||
| 00:32:57 | openstackgerrit | Jianghua Wang proposed openstack/nova master: WIP: support os_xenapi's VDI streaming https://review.openstack.org/486475 | |
| 18:40:24 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Add attachment_complete call to volume/cinder.py https://review.openstack.org/493323 | |
| 18:40:25 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: WIP - Implement new attach Cinder flow https://review.openstack.org/330285 | |
| 18:40:25 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Tweak connection_info translation for the new Cinder attach/detach API https://review.openstack.org/493324 | |