Posted	Nick	Remark
#openstack-nova - 2017-07-24
23:30:11	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042
23:38:14	openstackgerrit	Ken'ichi Ohmichi proposed openstack/nova master: Remove the useless fake ExtensionManager from API unittests https://review.openstack.org/486416
#openstack-nova - 2017-07-25
00:40:42	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Stop using mox stubs in cast_as_call.py https://review.openstack.org/477730
00:55:45	openstackgerrit	Merged openstack/nova master: Add tags to instance.create Notification https://review.openstack.org/459493
01:02:29	openstackgerrit	Hironori Shiina proposed openstack/nova master: Ironic: Support boot from Cinder volume https://review.openstack.org/215385
01:04:37	openstackgerrit	Merged openstack/nova master: Allow wrapping of closures https://review.openstack.org/479801
01:05:57	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Flush RC_CACHE after each gabbit sequence https://review.openstack.org/486805
01:08:21	cdent	dansmith: if you’re still around, this is probably a better fix for that 201 204 problem: https://review.openstack.org/#/c/486805/
01:08:30	cdent	(I couldn’t sleep, got bored, read mail…)
01:08:31	alex_xu	oomichi: appreciate if you can take a look at https://review.openstack.org/#/c/486414/, that patch is the goal of pike
01:15:19	mriedem	cdent: i'll drop mine once yours passes functional tests
01:15:30	cdent	mriedem: roger that
01:32:22	openstackgerrit	Merged openstack/nova master: Migrate Ironic Flavors https://review.openstack.org/484949
02:07:58	openstackgerrit	Merged openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946
02:28:54	openstackgerrit	Chris Dent proposed openstack/nova master: WIP: Using wsgi-intercept in OSAPIFixture https://review.openstack.org/486825
02:29:48	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Use wsgi_intercept in PlacementFixture https://review.openstack.org/486237
02:30:26	openstackgerrit	Chris Dent proposed openstack/nova master: WIP: Using wsgi-intercept in OSAPIFixture https://review.openstack.org/486825
02:40:57	openstackgerrit	Chris Dent proposed openstack/nova master: WIP: Optimize MiniDNS for fewer syscalls https://review.openstack.org/486829
02:41:44	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Update the placement deployment instructions https://review.openstack.org/469048
02:44:43	openstackgerrit	Tony Breeds proposed openstack/nova master: Avoid chowning console logs in libvirt https://review.openstack.org/472229
02:44:44	openstackgerrit	Tony Breeds proposed openstack/nova master: First attempt at adding a privsep user to nova itself. https://review.openstack.org/459166
02:44:44	openstackgerrit	Tony Breeds proposed openstack/nova master: Move the last_bytes util method to libvirt https://review.openstack.org/472228
02:44:45	openstackgerrit	Tony Breeds proposed openstack/nova master: Do not import nova.conf into nova/exception.py https://review.openstack.org/485907
02:44:45	openstackgerrit	Tony Breeds proposed openstack/nova master: Move libvirt usages of chown to privsep. https://review.openstack.org/471972
02:44:46	openstackgerrit	Tony Breeds proposed openstack/nova master: rootwrap.d cleanup mislabeled files https://review.openstack.org/486831
02:56:15	openstackgerrit	Chris Dent proposed openstack/nova master: WIP: Using wsgi-intercept in OSAPIFixture https://review.openstack.org/486825
03:01:20	openstackgerrit	Hironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector https://review.openstack.org/468353
03:01:47	openstackgerrit	Chris Dent proposed openstack/nova master: WIP: Optimize MiniDNS for fewer syscalls https://review.openstack.org/486829
03:30:24	TheJulia	hshiina: looks like https://review.openstack.org/215385 is already in merge conflict again :(
03:58:13	openstackgerrit	Julia Kreger proposed openstack/nova master: Ironic: Support boot from Cinder volume https://review.openstack.org/215385
03:58:23	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Wrong href link returned when providing non-existed version in GET version API https://review.openstack.org/486850
04:17:24	openstackgerrit	Ngo Quoc Cuong proposed openstack/nova master: Remove translation of log messages https://review.openstack.org/466995
04:47:19	openstackgerrit	Merged openstack/nova master: request_log addition for running under uwsgi https://review.openstack.org/485602
05:17:22	openstackgerrit	sudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/481007
06:17:28	openstackgerrit	Tony Breeds proposed openstack/nova master: Avoid chowning console logs in libvirt https://review.openstack.org/472229
06:17:28	openstackgerrit	Tony Breeds proposed openstack/nova master: First attempt at adding a privsep user to nova itself. https://review.openstack.org/459166
06:17:29	openstackgerrit	Tony Breeds proposed openstack/nova master: Move the last_bytes util method to libvirt https://review.openstack.org/472228
06:17:29	openstackgerrit	Tony Breeds proposed openstack/nova master: Do not import nova.conf into nova/exception.py https://review.openstack.org/485907
06:17:30	openstackgerrit	Tony Breeds proposed openstack/nova master: Move libvirt usages of chown to privsep. https://review.openstack.org/471972
06:17:30	openstackgerrit	Tony Breeds proposed openstack/nova master: rootwrap.d cleanup mislabeled files https://review.openstack.org/486831
06:20:56	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Wrong href link returned when providing non-existed version in GET version API https://review.openstack.org/486850
06:30:38	trinaths	hi, I'm using nova (mitaka) compute. when I spawn a VM, it will auto shutoff after 10 minutes. and at qemu logs, it was the error: 2017-07-25T00:52:49.334691Z qemu-system-aarch64: Failed initializing vhost-user memory map, consider using -object memory-backend-file share=on
06:30:38	trinaths	qemu-system-aarch64: /root/qemu/hw/virtio/vhost.c:1186: vhost_virtqueue_mask: Assertion `vdev->use_guest_notifier_mask' failed. Please help me in resolving this issue.
06:33:45	openstackgerrit	Tony Breeds proposed openstack/nova master: Avoid chowning console logs in libvirt https://review.openstack.org/472229
06:37:06	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time https://review.openstack.org/465954
06:54:24	openstackgerrit	Tony Breeds proposed openstack/nova master: Avoid chowning console logs in libvirt https://review.openstack.org/472229
06:54:25	openstackgerrit	Tony Breeds proposed openstack/nova master: First attempt at adding a privsep user to nova itself. https://review.openstack.org/459166
06:54:25	openstackgerrit	Tony Breeds proposed openstack/nova master: Move the last_bytes util method to libvirt https://review.openstack.org/472228
06:54:26	openstackgerrit	Tony Breeds proposed openstack/nova master: Do not import nova.conf into nova/exception.py https://review.openstack.org/485907
06:54:26	openstackgerrit	Tony Breeds proposed openstack/nova master: Move libvirt usages of chown to privsep. https://review.openstack.org/471972
06:54:27	openstackgerrit	Tony Breeds proposed openstack/nova master: rootwrap.d cleanup mislabeled files https://review.openstack.org/486831
07:47:18	openstackgerrit	Hironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector https://review.openstack.org/468353
08:03:22	openstackgerrit	Rawan Herzallah proposed openstack/nova master: Adding NVMEoF for libvirt driver https://review.openstack.org/482640
08:07:42	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Wrong href link returned when providing non-existed version in GET version API https://review.openstack.org/486850
08:15:30	jangutter	Hi, there are two patches left for netronome-smartnic-enablement. Would anyone have cycles to leave comments? 50 lines of code, 300 lines of tests, many litres of blood. https://review.openstack.org/#/c/486426/ and https://review.openstack.org/#/c/483459/
08:16:43	jangutter	For that matter, Mellanox's got a similar patch: https://review.openstack.org/#/c/398265/
08:42:46	openstackgerrit	Artem Tiumentcev proposed openstack/nova-specs master: SPEC: nova disk quota tracking https://review.openstack.org/486947
08:49:08	openstackgerrit	Tony Breeds proposed openstack/nova master: Avoid chowning console logs in libvirt https://review.openstack.org/472229
09:01:21	kashyap	mdbooth: Thanks for the review here: https://review.openstack.org/#/c/485752/
09:01:32	kashyap	mdbooth: Test - I will fix, that's also why I -1ed it myself.
09:01:55	kashyap	mdbooth: But the core change itself is working; the downstream bug reporter has tested the fix and confirmed that they don't see the perf regression anymore with this fix
09:18:24	mdbooth	kashyap: Yeah, I said it looked good :)
09:18:43	mdbooth	Except for the bits which didn't, but you can sort those easily enough.
09:18:57	kashyap	mdbooth: Sure, did notice your remark. :-)
09:19:17	kashyap	mdbooth: The bits which didn't are - (a) missing tests; (b) adding in the config line that accidentally removed
09:19:21	kashyap	Anything else you noticed?
09:19:42	mdbooth	Nope.
09:20:05	mdbooth	The code structure is kinda ugly, but sahid already noted that
09:20:22	openstackgerrit	Alexandru Muresan proposed openstack/nova master: Ensure that vif ports are unplugged during cold migration https://review.openstack.org/486955
09:22:38	kashyap	mdbooth: There's a comment from Sahid? I don't even see it, /me refreshes the URL
09:22:49	mdbooth	Not a review comment
09:22:49	kashyap	Maybe you mean in the code
09:22:53	kashyap	Yeah, noted.
09:22:57	mdbooth	A code comment from the distant past
09:23:56	kashyap	Yep
09:25:33	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Add tags fields for server list detail/show api-ref https://review.openstack.org/465842
09:31:47	openstackgerrit	Rikimaru Honjo proposed openstack/nova master: compute: Sweep duplicated bdm records https://review.openstack.org/290793
09:32:31	openstackgerrit	Béla Vancsics proposed openstack/nova master: Transform instance.resize_confirm notification https://review.openstack.org/482557
09:36:02	bauzas	gabor_antal: gibi: about https://review.openstack.org/#/c/467514/7/nova/compute/api.py@4977 honestly my only concern was that I think "import" is probably not a great RESTful action
09:37:10	bauzas	gabor_antal: gibi: cdent reminded me previously that a PUT can create or update with a body so in theory there should only be one notification of creation
09:37:39	bauzas	but anyway, seems I'm nitpicking
09:45:44	bauzas	gibi: btw. any notifications patches I could help to land before FF ?
10:18:30	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/os-vif master: Migration from ``ip`` commands to ``pyroute2`` https://review.openstack.org/484386
10:29:33	gibi	bauzas: hi! thanks for the feedback. I don't have direct user feedback so I cannot say for sure that we need import in the notification but it feels useful for me
10:29:43	bauzas	cools
10:31:14	gibi	bauzas: for patches to help in, I see on patch that needs a second +2 https://review.openstack.org/#/c/480984/
10:31:51	gibi	bauzas: besides that we have a list of patches that looks good from subteam perspective (and has +1) here https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/versioned-notification-transformation-pike+label:Code-Review%253E%253D%252B1+label:Verified%253E%253D1+AND+NOT+label:Verified%253C0+AND+NOT+label:Code-Review%253C0
10:34:12	openstackgerrit	Gábor Antal proposed openstack/nova master: Transform HostAPI-set_enabled notifications https://review.openstack.org/482622
10:34:15	gibi	bauzas: any help is appreciated
10:38:28	bauzas	gibi: going out for lunch but will review those afterwards
10:40:50	openstackgerrit	Merged openstack/nova master: Remove check_detach https://review.openstack.org/446671
10:43:18	gibi	bauzas: enjoy for lunch, I just brought mine in the meantime
10:49:05	gibi	s/for/your/
10:50:24	openstackgerrit	Alexandra Settle proposed openstack/nova master: doc: Rework index page per new sections https://review.openstack.org/478485
11:03:40	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
11:09:27	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
11:14:50	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042
11:21:53	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
11:25:36	ildikov	stephenfin: hi
11:26:57	ildikov	stephenfin: I just wanted to ask whether you got a chance to check stvnoyes's answers here: https://review.openstack.org/#/c/463987/ to see whether they address your concerns?

Earlier   Later