Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-08-11
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: Use ksa adapter for placement conf & requests https://review.openstack.org/492247
21:33:34 openstackgerrit Eric Fried proposed openstack/nova master: nova.utils.get_ksa_adapter() https://review.openstack.org/488137
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 - Implement new attach Cinder flow https://review.openstack.org/330285
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 - Tweak connection_info translation for the new Cinder attach/detach API https://review.openstack.org/493324
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 - Tweak connection_info translation for the new Cinder attach/detach API https://review.openstack.org/493324
18:32:38 openstackgerrit Ildiko Vancsa proposed openstack/nova master: WIP - Implement new attach Cinder flow https://review.openstack.org/330285
#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: Tweak connection_info translation for the new Cinder attach/detach API https://review.openstack.org/493324
18:40:25 openstackgerrit Ildiko Vancsa proposed openstack/nova master: WIP - Implement new attach Cinder flow https://review.openstack.org/330285
#openstack-nova - 2017-08-14
00:05:51 openstackgerrit Michael Still proposed openstack/nova master: Avoid chowning console logs in libvirt https://review.openstack.org/472229
00:05:51 openstackgerrit Michael Still proposed openstack/nova master: First attempt at adding a privsep user to nova itself. https://review.openstack.org/459166
00:05:52 openstackgerrit Michael Still proposed openstack/nova master: Move execs of touch to privsep. https://review.openstack.org/489190
00:05:52 openstackgerrit Michael Still proposed openstack/nova master: Move libvirts dmcrypt support to privsep. https://review.openstack.org/490737
00:05:53 openstackgerrit Michael Still proposed openstack/nova master: Move execs of tee to privsep. https://review.openstack.org/489438
00:05:55 openstackgerrit Michael Still proposed openstack/nova master: Don't shell out to mkdir, use ensure_tree() https://review.openstack.org/492326
00:05:56 openstackgerrit Michael Still proposed openstack/nova master: Move libvirt usages of chown to privsep. https://review.openstack.org/471972
00:05:57 openstackgerrit Michael Still proposed openstack/nova master: WIP: Read from console ptys using privsep. https://review.openstack.org/489486
00:05:57 openstackgerrit Michael Still proposed openstack/nova master: Move ploop commands to privsep. https://review.openstack.org/492325
00:05:58 openstackgerrit Michael Still proposed openstack/nova master: Refactor libvirt.utils.execute() away. https://review.openstack.org/489816
01:41:11 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955
05:00:45 openstackgerrit Thomas Bechtold proposed openstack/nova master: Handle deleted instances when refreshing the info_cache https://review.openstack.org/398308
05:00:53 openstackgerrit Thomas Bechtold proposed openstack/nova master: Handle deleted instances when refreshing the info_cache https://review.openstack.org/398308

Earlier   Later