Posted	Nick	Remark
#openstack-nova - 2018-01-25
01:55:19	takashin	oomichi: My patch was changing 'pike' to 'queens'. Replaceing http with https was done at the same time, but it is not worth doing it separately.
01:56:49	efried	mriedem LGTM, +1
01:58:02	oomichi	takashin: I am not sure why it is not worth doing it *separately*
01:58:16	efried	mriedem Course, you coulda addressed my nit, but meh.
01:58:53	efried	takashin Glad to catch you online. I've been wanting to tell you how much I appreciate your super-thorough code reviews.
01:59:37	efried	takashin ありがとうございました ;)
02:00:17	jose-phi_	hi any idea
02:00:28	takashin	efied: You are welcome.
02:00:36	jose-phi_	of why when i set data-port=br-ext:eno1 this interface got dead "eno1"
02:00:49	jose-phi_	and this is my data network so i lost all connectivity to the compute node
02:01:13	takashin	oomichi: it is not worth doing it alone.
02:02:08	takashin	efried: You are welcome.
02:02:52	openstackgerrit	Eric Fried proposed openstack/nova master: set_{aggregates|traits}_for_provider: tolerate set https://review.openstack.org/536625
02:02:52	openstackgerrit	Eric Fried proposed openstack/nova master: New-style _set_inventory_for_provider https://review.openstack.org/537648
02:02:53	openstackgerrit	Eric Fried proposed openstack/nova master: SchedulerReportClient.update_from_provider_tree https://review.openstack.org/533821
02:02:53	openstackgerrit	Eric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker https://review.openstack.org/520246
02:02:54	openstackgerrit	Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260
02:02:54	openstackgerrit	Eric Fried proposed openstack/nova master: Move refresh time from report client to prov tree https://review.openstack.org/535517
02:03:23	gmann	mriedem: let me rebase them, not all are required. i will do it today and let you know
02:05:10	oomichi	takashin: I just don't want to see similar patches in the future even after dropping the patch
02:05:21	efried	d'oh
02:05:30	openstackgerrit	Eric Fried proposed openstack/nova master: ProviderTree.get_provider_uuids: Top-down ordering https://review.openstack.org/536624
02:05:31	openstackgerrit	Eric Fried proposed openstack/nova master: set_{aggregates|traits}_for_provider: tolerate set https://review.openstack.org/536625
02:05:31	openstackgerrit	Eric Fried proposed openstack/nova master: New-style _set_inventory_for_provider https://review.openstack.org/537648
02:05:32	openstackgerrit	Eric Fried proposed openstack/nova master: SchedulerReportClient.update_from_provider_tree https://review.openstack.org/533821
02:05:32	openstackgerrit	Eric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker https://review.openstack.org/520246
02:05:33	openstackgerrit	Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260
02:05:33	openstackgerrit	Eric Fried proposed openstack/nova master: Move refresh time from report client to prov tree https://review.openstack.org/535517
02:06:17	takashin	oomichi: okay. I agree with the change.
02:10:49	takashin	oomichi: I voted +1 for the patch.
02:11:13	oomichi	takashin: thanks :-)
02:25:16	openstackgerrit	Takashi NATSUME proposed openstack/python-novaclient master: Add CLI to show instance usage audit logs https://review.openstack.org/409413
03:09:20	alex_xu	mriedem: thanks, will cleanup in a follow up
03:12:33	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Merge server create schema for availability zone extension https://review.openstack.org/451331
03:13:27	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Merge server create for availability zone extension https://review.openstack.org/502574
03:14:08	openstackgerrit	Jackie Truong proposed openstack/nova master: Reduce complexity of _from_db_object https://review.openstack.org/537728
03:16:20	alex_xu	mriedem: Kevin_Zheng, it would be great we add some testcase ensure the substring format can pass the regex format in the followup patch, since our API schema value the ip filter input with regex format. https://review.openstack.org/525505
03:17:37	alex_xu	mriedem: Kevin_Zheng and, I'm curious how neutron side handle the different of 'like' in the different db
03:20:47	Kevin_Zheng	alex_xu, sure
03:32:52	Yikun	alex_xu, https://review.openstack.org/#/c/521683/21/neutron/plugins/ml2/plugin.py@1862, here is 'like'
03:34:31	openstackgerrit	Jackie Truong proposed openstack/nova master: Reduce complexity of _from_db_object https://review.openstack.org/537728
03:37:50	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Fix missing marker functions https://review.openstack.org/514579
04:04:42	Spazmotic	Morning
04:30:22	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers.inc https://review.openstack.org/528201
04:45:09	openstackgerrit	Jackie Truong proposed openstack/nova master: Add trusted_certs object https://review.openstack.org/489408
04:46:46	openstackgerrit	Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949
04:51:04	openstackgerrit	Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
04:57:51	openstackgerrit	Brooks Kaminski proposed openstack/nova master: XenAPI/Stops the migration of volume backed VHDS https://review.openstack.org/533168
05:18:10	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Fix missing 'if_notifications_enabled' decorator https://review.openstack.org/537745
05:27:01	openstackgerrit	Brooks Kaminski proposed openstack/nova master: XenAPI/Stops the migration of volume backed VHDS https://review.openstack.org/533168
05:27:31	Spazmotic	My first rebase.. kind of scary
05:27:43	Spazmotic	things are puckered
05:56:21	alex_xu	Yikun: ah, thanks
06:48:59	openstackgerrit	Merged openstack/nova master: add whereto for testing redirect rules https://review.openstack.org/516385
07:46:40	Spazmotic	jianghuaw: addressed the nits by Bob and and rebased if you get a chance on Zuul confirms it :)
07:46:47	Spazmotic	once* zuul
07:49:02	jianghuaw	Spazmotic, thanks for the update. It looks good to me:-)
07:49:07	Spazmotic	Thanks so much sir!
07:49:25	Spazmotic	hunting around for my next project
07:54:28	jianghuaw	Spazmotic, you may have interesting on this patch: https://review.openstack.org/#/c/532118/
07:54:59	jianghuaw	Huan created it for getting advices.
07:55:09	jianghuaw	it's to fix a migration issue with xenapi.
07:55:31	Spazmotic	sure will take a look
07:55:44	jianghuaw	Spazmotic, thanks.
07:57:49	Spazmotic	Interesting. We do volume based LM quite a bit but we get around this with --block-migrate for iSCSI
07:59:23	jianghuaw	Spazmotic, iirc the issue happens only in the XS7.0 and forwarding releases.
08:00:01	Spazmotic	That's definately going to be a problem i'll need to test out
08:00:47	jianghuaw	or maybe 7.1 => . I put 7.1 in the log.
08:01:08	jianghuaw	But anyway it will be issue for latest XS releases.
08:01:26	jianghuaw	feel free to put comments in it.
08:01:54	jianghuaw	If you interest, feel free to take it over:-)
08:02:06	Spazmotic	I'll certainly look at it heavily, appeals to me greatly
08:02:55	jianghuaw	Thanks.
08:08:20	Spazmotic	Do you know if Huan has refactored it at all recently? Mentions they wanted to tweak the vmops method around, but i'm sure time was tight for you guys
08:14:57	jianghuaw	Spazmotic, not yet I'm sure.
08:15:25	Spazmotic	Alrighty cool.. I've got some 7.1s in an env, i'll run some tests over the next few days and get familiar with these maps, havn't had to use them before
08:15:34	Spazmotic	Thanks for bringing to my attention is actually more important than you may know
08:16:03	jianghuaw	cool.
08:29:29	openstackgerrit	Hironori Shiina proposed openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487
08:35:32	openstackgerrit	Maciej Jozefczyk proposed openstack/nova master: Do not normalize allocation_ratios https://review.openstack.org/532924
08:38:29	maciejjozefczyk	Hello
08:49:32	gmann	mriedem: updated those. 2 are ready to go and 1 i need more work on test which i will tomorrow - https://review.openstack.org/#/q/topic:bp/api-extensions-merge-queens+status:open
08:50:30	gmann	mriedem: there are other work also needed on this BP which we will continue on continued Rocky BP. if you want to move current patches to new one, it should be ok
08:52:47	gmann	mriedem: or we defer the queens one to Rocky ?
09:05:03	kashyap	It's middle of the night for Matt, so will take a few more hours for him to come online.
09:24:44	Spazmotic	Aargh Zuul why you hate me
09:27:24	Roamer`	Spazmotic, if it's any help, I doubt it's personal - it's in Zuul's nature to be very, very paranoid and suspicious of everyone, you just have to catch one of its rare good moods... :)
09:30:32	openstackgerrit	Zengzhi Wang proposed openstack/nova stable/pike: Make eventlet hub use a monotonic clock https://review.openstack.org/537811
09:57:45	hrw	morning
09:58:40	hrw	I am looking at nova's code. it adds graphics if it is needed. but I did not see where it adds usb host for usbtablet
09:58:55	hrw	does it assume that usb host is already present in VM configuration?
09:59:54	Roamer`	OK, so the StorPool volume attachment driver finally passed the Zuul check pipeline (after the Cinder backup bug was fixed)... I'm pretty sure mriedem intends to +W it once he wakes up and sees it's passed the tests, but the gate being what it is, I'd be really obliged if somebody else could +W it earlier :) Of course, "let's just wait for him" is a very reasonable option.
10:00:12	Roamer`	oof, and that would be https://review.openstack.org/#/c/140733/19 of course
10:03:54	kevinz	alex_xu: Could you review https://review.openstack.org/#/c/494156/ if you have time? Stephen already give +2 :-)
10:05:37	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Transform instance.exists notification https://review.openstack.org/403660
10:08:12	alex_xu	kevinz: got it
10:11:12	hrw	tracking how nova gets guest components suxx
10:13:20	stephenfin	mriedem, melwitt: Yeah, sorry about https://review.openstack.org/#/c/523958/ I was the approver and that's my bad. Won't happen again
10:18:35	hrw	ok. nova assumes that usb host is present
10:19:51	kashyap	stephenfin: mriedem: As I've said in the past, I don't quite agree with that line of thinking. Happy to also elaborate why in Dublin.

Earlier   Later