Posted	Nick	Remark
#openstack-nova - 2018-01-25
00:11:17	jaypipes	efried: the client should just know it's an *atomically* increasing integer that, if it changed from the client's previous view of the generation, should trigger a re-read of the expected state.
00:11:26	jaypipes	(and now I have to leave, I'm sorry :(
00:11:40	efried	jaypipes Have a good one.
00:11:43	Spazmotic	see ya jay
00:11:55	Spazmotic	Just waking up and drinking my coffee and getting ready for hte gym.. whew
00:32:15	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081
00:54:41	mnaser	mriedem: awesome stuff about osc-placement :D
01:46:33	oomichi	takashin: hi, still isn't https://review.openstack.org/#/c/533134/ favorite?
01:47:38	takashin	oomichi: Hi. There are redirects. So the modification is not necessary.
01:48:33	oomichi	takashin: do you see my comment?
01:49:59	takashin	oomichi: Yes. but my patch was not just replacing http with https.
01:50:11	Yikun	gmann, https://review.openstack.org/#/c/534721/, hi, I saw your inline comments, so, should we add 2 config like project_network_cidr[1 & 2] to keep this? or some other idea?
01:50:43	oomichi	takashin: then, why did you change it even if there was a redirecting?
01:54:14	openstackgerrit	Matt Riedemann proposed openstack/nova master: Fix bug case by none token context https://review.openstack.org/522112
01:54:15	mriedem	efried: can you tell me if the release note i threw in here sounds ok? ^
01:54:39	efried	...
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 ?

Earlier   Later