Posted	Nick	Remark
#openstack-nova - 2019-04-28
10:47:05	openstackgerrit	Merged openstack/os-vif stable/rocky: make kuryr-kubernetes-tempest-daemon-octavia non voting https://review.opendev.org/647500
11:41:54	openstackgerrit	chenker proposed openstack/python-novaclient master: Optimize limit option docs string description for novaclient https://review.opendev.org/643035
11:43:50	openstackgerrit	Merged openstack/nova master: Do not perform port update in case of baremetal instance. https://review.opendev.org/649345
11:45:22	openstackgerrit	chenker proposed openstack/python-novaclient master: Optimize limit option docs string description for novaclient https://review.opendev.org/643035
11:47:14	openstackgerrit	chenker proposed openstack/python-novaclient master: Optimize limit option docs string description for novaclient https://review.opendev.org/643035
11:48:47	openstackgerrit	chenker proposed openstack/python-novaclient master: Optimize limit option docs string description for novaclient https://review.opendev.org/643035
12:47:21	openstackgerrit	Merged openstack/nova-specs master: Abandon parallel-scheduler backlog spec https://review.opendev.org/649101
15:47:24	openstackgerrit	Matt Riedemann proposed openstack/nova stable/stein: Do not perform port update in case of baremetal instance. https://review.opendev.org/656176
15:53:50	openstackgerrit	Eric Fried proposed openstack/nova-specs master: Abandon instance-tasks backlog spec https://review.opendev.org/649100
15:54:14	efried	johnthetubaguy: rebase because I had four parallel changes doing the same thing to different specs ^
15:55:48	openstackgerrit	Eric Fried proposed openstack/nova-specs master: Abandon flavor-classes backlog spec https://review.opendev.org/649095
15:56:45	openstackgerrit	Eric Fried proposed openstack/nova-specs master: Abandon same-instances-scheduling backlog spec https://review.opendev.org/649102
17:07:35	openstackgerrit	Merged openstack/nova master: test_rpc: Stop f****** with global state https://review.opendev.org/655843
17:32:57	mnaser	lols ^
17:45:59	openstackgerrit	Lee Yarwood proposed openstack/nova master: libvirt: Avoid using os-brick encryptors when device_path isn't provided https://review.opendev.org/649951
18:16:50	openstackgerrit	Merged openstack/nova stable/stein: Create request spec, build request and mappings in one transaction https://review.opendev.org/656018
20:06:16	openstackgerrit	Chris Dent proposed openstack/nova master: Delete the placement code https://review.opendev.org/618215
20:22:54	openstackgerrit	Merged openstack/nova master: Make nova.compute.rpcapi.ComputeAPI.router a singleton https://review.opendev.org/649197
#openstack-nova - 2019-04-29
01:45:22	openstackgerrit	shenxindi proposed openstack/nova master: Correct spelling errors https://review.opendev.org/656220
04:21:13	openstackgerrit	Merged openstack/nova master: Allow driver to specify switch&port for faster lookup https://review.opendev.org/617695
04:52:04	openstackgerrit	Merged openstack/nova master: AZ list performance optimization: avoid double service list DB fetch https://review.opendev.org/636947
04:52:12	openstackgerrit	Merged openstack/nova master: Correct spelling errors https://review.opendev.org/656220
05:23:52	openstackgerrit	caoyuan proposed openstack/os-vif master: Replace git.openstack.org URLs with opendev.org URLs https://review.opendev.org/655068
05:24:54	openstackgerrit	caoyuan proposed openstack/os-vif master: Replace git.openstack.org URLs with opendev.org URLs https://review.opendev.org/655068
06:07:07	openstackgerrit	yuanliu proposed openstack/nova master: Fix query method for nova compute services by compute_driver types https://review.opendev.org/637083
07:48:47	kashyap`	aspiers: efried: jaypipes: On your question about "what ARE these 'ssbd', 'spec-ctrl' ..." — they are hardware-level CPU features provided by the vendor via a microcode update, and are visible via CPUID.
07:49:03	kashyap`	jaypipes: I saw your comments on the traits change, will address
07:58:11	openstackgerrit	zhufl proposed openstack/nova master: Fix invalid function name https://review.opendev.org/656262
08:10:19	kashyap	aspiers: My hairs are splitting: on that "traits" thingie, am I making sense: https://review.opendev.org/#/c/655193/2
08:35:21	kashyap	jaypipes: Also note: you noted ont the change that all the mentioned flags are "mutually exclusive" -- not true: AMD 'virt-ssb' and 'amd-ssb' are "mutually _necessary_". Noted it in the review.
08:38:05	openstackgerrit	Merged openstack/nova master: Use aggregate_add_host in nova-manage https://review.opendev.org/640080
08:47:07	kashyap	jaypipes: But yes, you make an interesting and valid point on use case of these traits (Nova vs. some "baremetal hypervisor node management system"). I'm still thinking...
09:35:32	openstackgerrit	Kashyap Chamarthy proposed openstack/os-traits master: Add CPU traits for Meltdown/Spectre mitigation https://review.opendev.org/655193
09:35:33	openstackgerrit	Kashyap Chamarthy proposed openstack/os-traits master: [Request for Comments] ADD "SVM" CPU trait to amd.py https://review.opendev.org/656276
09:36:04	kashyap	efried: jaypipes: aspiers ^ I know you're travelling, when you can, have a gander at the "Request for Comments" patch above.
09:37:41	kashyap	efried: jaypipes: aspiers: (The commit message summarizes the story.)
12:14:32	openstackgerrit	Kashyap Chamarthy proposed openstack/nova master: libvirt: Rework 'EBUSY' (SIGKILL) error handling code path https://review.opendev.org/639091
12:15:07	kashyap	lyarwood: If your fingers are itching to review something --^
12:15:11	kashyap	:-)
12:36:03	openstackgerrit	Corey Bryant proposed openstack/nova stable/stein: xenapi/agent: Change openssl error handling https://review.opendev.org/656304
12:53:07	openstackgerrit	Corey Bryant proposed openstack/nova stable/rocky: xenapi/agent: Change openssl error handling https://review.opendev.org/656307
12:53:54	openstackgerrit	Corey Bryant proposed openstack/nova stable/queens: xenapi/agent: Change openssl error handling https://review.opendev.org/656308
12:55:35	openstackgerrit	Corey Bryant proposed openstack/nova stable/rocky: xenapi/agent: Change openssl error handling https://review.opendev.org/656307
12:56:21	openstackgerrit	Corey Bryant proposed openstack/nova stable/queens: xenapi/agent: Change openssl error handling https://review.opendev.org/656308
12:59:53	openstackgerrit	Corey Bryant proposed openstack/nova stable/queens: xenapi/agent: Change openssl error handling https://review.opendev.org/656308
13:46:16	mriedem	mtreinish: are you able to tell me what i'm doing wrong with the tempest/stestr regex stuff in this job? https://review.opendev.org/#/c/655222/
13:57:05	mtreinish	mriedem: http://logs.openstack.org/22/655222/6/check/nova-multi-cell/3ab8168/job-output.txt.gz#_2019-04-26_23_55_53_956207 it looks like it's including the comments which is probably confusing things on the cli
13:57:19	mtreinish	also the quoting looks kinda off, but it might work like that
13:57:29	mtreinish	oh, he disappeared :/
13:58:19	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Revert "Wait for network-vif-plugged on resize revert" https://review.opendev.org/639396
13:58:19	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Run revert resize tests in nova-live-migration https://review.opendev.org/653498
13:58:20	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Revert resize: wait for external events in compute manager https://review.opendev.org/644881
14:49:14	openstackgerrit	Kashyap Chamarthy proposed openstack/os-traits master: Add CPU traits for Meltdown/Spectre mitigation https://review.opendev.org/655193
14:49:15	openstackgerrit	Kashyap Chamarthy proposed openstack/os-traits master: ADD "SVM" CPU trait to amd.py https://review.opendev.org/656276
16:02:45	mriedem	mtreinish: think i figured out my dilemma
16:02:50	mriedem	https://github.com/openstack/tempest/blob/master/roles/run-tempest/README.rst
16:02:56	mriedem	(?x) # Ignore comments and whitespaces
16:03:01	mriedem	i didn't take that literally
16:04:53	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add nova-multi-cell job https://review.opendev.org/655222
16:04:54	openstackgerrit	Matt Riedemann proposed openstack/nova master: Enable n-novnc in nova-multi-cell job https://review.opendev.org/655711
16:14:52	larsks	Dumb nova question: is specifying "--live ''" to the 'openstack server migrate' command the same as "setting the host parameter to null" in the corresponding api request? The docs suggest using the "nova live-migration" command in this case, rather than "openstack server migrate", but that latter *seems* to work and I want to make sure that's not an accident...
16:15:38	sean-k-mooney	that is a good question
16:15:55	sean-k-mooney	we have unfortunetly overloaded --live to do two things
16:16:11	sean-k-mooney	enable live migratin and specify the host
16:16:20	sean-k-mooney	really we should have a --host option for that
16:16:57	sean-k-mooney	i would have to check the osc code too see if we have special cased ''
16:16:59	larsks	That sounds like a really good idea.
16:17:06	larsks	(having --host separate)
16:18:06	larsks	I was wondering more if an empty string would effectively evaluate the same as null on the server side.
16:19:03	sean-k-mooney	initally at least it looks like we pass the '' through directly https://github.com/openstack/python-openstackclient/blob/master/openstackclient/compute/v2/server.py#L1468
16:20:56	sean-k-mooney	the nova client also intiall does not special case '' https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/servers.py#L436
16:23:18	larsks	Okay. I'm not familiar enough with the code to see where that ends up on the server side. I'm hoping that we'll find that 'host' is being evaluated as a boolean to determine whether its set or not, in which case it would Just Work.
16:24:11	larsks	Oh, I think I found it.
16:25:00	larsks	https://github.com/openstack/nova/blob/master/nova/compute/api.py#L4502
16:25:08	larsks	So I think that works out.
16:25:50	mtreinish	mriedem: oh, that's confusing. I wouldn't have read that correctly either
16:26:07	mriedem	i thought it was an example
16:26:20	sean-k-mooney	right so the api say the seculer will select the host if the value is none https://developer.openstack.org/api-ref/compute/?expanded=live-migrate-server-os-migratelive-action-detail
16:26:47	mriedem	larsks: the osc live migrate command *requires* a host which is bad if you're not using a newer microversion to run that host through the scheduler,
16:26:50	sean-k-mooney	but looking at that coe an empty strig would also be ignored by that if
16:26:50	mriedem	because it forces the host
16:27:16	larsks	Right.
16:27:28	mriedem	larsks: see ~L16 https://etherpad.openstack.org/p/DEN-osc-compute-api-gaps
16:27:34	mriedem	going to talk about fixing that this week
16:27:37	mriedem	there have been several attempts
16:28:01	mriedem	because since queens you can also specify a target host for cold migrate but not yet with osc
16:28:28	sean-k-mooney	i assume we are not allowed to change existing args in osc
16:28:55	sean-k-mooney	but we caould add a --live-migrate flag and --host <host> args
16:28:55	larsks	mriedem: ...and just to make sure I'm not missing something, there continues to be no "live migrate *all* instances off the specified host" command at the api level, so e.g. setting up for system maintenance still requires individually migrating each instance, right?
16:29:18	sean-k-mooney	larsks: correct
16:29:21	larsks	Thanks.
16:29:24	mriedem	larsks: not in the api or osc
16:29:28	sean-k-mooney	and there likely should not be one at teh api
16:29:33	mriedem	there is a nova cli command that does that
16:29:44	sean-k-mooney	larsks: the nova host-evecuate-live command is impleeted in the nova client
16:29:50	mriedem	larsks: http://www.danplanet.com/blog/2016/03/03/evacuate-in-nova-one-command-to-confuse-us-all/
16:30:16	larsks	Is there an osc command? Because the nova cli doesn't support clouds.yaml, which is a pain.
16:30:38	larsks	Meh, I can work around that. In any case, thank you for the pointer.
16:30:38	mriedem	no
16:30:44	sean-k-mooney	larsks: by the way the reason i said its not a good idea to have this at the api level is starting multple live migration on the same host concurrently increase the risk that they will fail or not complete

Earlier   Later