Posted	Nick	Remark
#openstack-nova - 2018-06-11
18:58:15	mriedem	just paperwork
18:58:34	mriedem	but see the storyboard thread in the ML about why people care
18:59:21	melwitt	okay, I just didn't know to/how to do it. my bad
19:01:43	mriedem	artom: re https://review.openstack.org/#/c/572790/ i don't think we need your tempest patch
19:01:47	mriedem	i'm not sure why you created it
19:01:57	mriedem	because did you see that ^ depends on https://review.openstack.org/#/c/573025/ ?
19:02:12	artom	mriedem, we don't? I mean, to actually test the "positive" case where we refuse the swap
19:02:29	mriedem	you mean negative?
19:03:07	artom	mriedem, I genuinely have no idea :/
19:03:17	artom	mriedem, the case where we assertRaises(400)
19:03:18	artom	That one.
19:03:18	mriedem	testing error conditions in tempest are called negative tests
19:03:26	artom	OK, negative then :)
19:05:17	melwitt	mriedem: okay, done. thank you for letting me know about it https://launchpad.net/nova/+milestone/rocky-2
19:05:45	melwitt	I removed the rocky-2 milestone target from https://blueprints.launchpad.net/nova/+spec/preemptible-instances also
19:08:02	mriedem	artom: ok here is your 400 http://logs.openstack.org/09/573809/2/check/nova-multiattach/ebd4e2a/job-output.txt.gz#_2018-06-09_14_49_19_637410
19:08:18	mriedem	PUT http://104.130.136.141/compute/v2.1/servers/57ba4f52-8076-45e2-b6ac-14cf789b6926/os-volume_attachments/8361a693-e7c6-4a9b-8303-c303d5fbdad5
19:08:23	mriedem	Body: {"volumeAttachment": {"volumeId": "d9636fa2-78a3-468a-b56a-9e6e2d6f14f6"}}
19:08:48	artom	mriedem, yep, which is why I posted that patch
19:08:56	artom	mriedem, you don't have to merge it if you don't have to :)
19:09:07	artom	Although the cleanup failure is interesting
19:09:11	mriedem	i'm not QA core so i can't merge it..
19:09:28	artom	Either I'm really thick and doing something wrong, or refusing to swap a volume legit makes it undetachable
19:10:32	mriedem	where is the cleanup code that is waiting for the volume to be available after the swap failure,
19:10:41	mriedem	the code is doing the right thing, it leaves the old volume attached (in-use)
19:11:04	mriedem	the trace in the tempest failure doesn't make sense to me
19:11:14	artom	Same here, I can't understand it
19:11:31	artom	I tried printing the full stacktrace manually, but got just a bunch of test runner stuff
19:11:46	artom	So I can't even pinpoint where it's from - I *think* it's the cleanup phase
19:11:59	mriedem	i'm pretty sure it's the cleanups from attach_volume
19:12:09	mriedem	# is already detached.
19:12:09	mriedem	# Ignore 404s on detach in case the server is deleted or the volume
19:12:09	mriedem	self.volumes_client, volume['id'], 'available')
19:12:09	mriedem	self.addCleanup(waiters.wait_for_volume_resource_status,
19:12:09	mriedem	# teardown.
19:12:09	mriedem	# is so we don't error out when trying to delete the volume during
19:12:09	mriedem	# On teardown detach the volume and wait for it to be available. This
19:12:10	mriedem	self.addCleanup(self._detach_volume, server, volume)
19:12:27	mriedem	so, your test can't use that from the base class
19:12:59	mriedem	oh wait
19:13:03	mriedem	hmm
19:13:03	mriedem	but it should be detaching the volume
19:14:51	mriedem	looks like n-cpu is detaching it here http://logs.openstack.org/09/573809/2/check/nova-multiattach/ebd4e2a/logs/screen-n-cpu.txt.gz#_Jun_09_14_49_16_096647
19:17:13	mriedem	this shows the volume was available right when it failed http://logs.openstack.org/09/573809/2/check/nova-multiattach/ebd4e2a/job-output.txt.gz#_2018-06-09_15_07_12_120196
19:17:26	mriedem	[tempest.common.waiters] volume 8361a693-e7c6-4a9b-8303-c303d5fbdad5 reached available after waiting for 4.377925 seconds
19:18:04	artom	Whoa? But it waits for like a minute and a half?
19:19:04	mriedem	this is the detach request http://logs.openstack.org/09/573809/2/check/nova-multiattach/ebd4e2a/job-output.txt.gz#_2018-06-09_15_07_12_102448
19:19:09	mriedem	2018-06-09 14:49:15,860 5908 INFO [tempest.lib.common.rest_client] Request (TestMultiAttachVolumeSwap:_run_cleanups): 202 DELETE http://104.130.136.141/compute/v2.1/servers/57ba4f52-8076-45e2-b6ac-14cf789b6926/os-volume_attachments/8361a693-e7c6-4a9b-8303-c303d5fbdad5 0.301s
19:19:21	mriedem	2018-06-09 14:49:19,377 5908 INFO [tempest.common.waiters] volume 8361a693-e7c6-4a9b-8303-c303d5fbdad5 reached available after waiting for 4.377925 seconds
19:19:27	mriedem	took 4 seconds to detach it
19:21:28	mriedem	yeah i have no idea where that waiter is coming from
19:21:43	mriedem	besides just the attach_volume cleanup
19:22:25	artom	I'll try to dig into it more
19:22:44	artom	Not now though, don't want to context switch from NUMA LM
19:31:16	openstackgerrit	Chris Dent proposed openstack/nova master: Do not use nova.test in placement.test_util https://review.openstack.org/574403
19:31:17	openstackgerrit	Chris Dent proposed openstack/nova master: Do not use nova.test in placement.handlers.test_aggregate https://review.openstack.org/574404
19:31:18	openstackgerrit	Chris Dent proposed openstack/nova master: Do not use nova.test in placement.test_requestlog https://review.openstack.org/574405
19:31:19	openstackgerrit	Chris Dent proposed openstack/nova master: Do not use nova.test in placement.test_fault_wrap https://review.openstack.org/574406
19:31:20	openstackgerrit	Chris Dent proposed openstack/nova master: Do not use nova.test in placement.test_handler https://review.openstack.org/574407
19:31:21	openstackgerrit	Chris Dent proposed openstack/nova master: Do not use nova.test in placement.test_microversion https://review.openstack.org/574408
19:31:22	openstackgerrit	Chris Dent proposed openstack/nova master: Do not use nova.test in placement.test_deploy https://review.openstack.org/574409
19:33:08	openstackgerrit	Zack Cornelius proposed openstack/nova master: Refactor libvirt get_memory_used_mb() https://review.openstack.org/571030
19:33:09	openstackgerrit	Zack Cornelius proposed openstack/nova master: Implement file backed memory for instances in libvirt https://review.openstack.org/567876
19:41:48	mriedem	dansmith: is dash working for you?
19:42:01	dansmith	mriedem: not at the moment but it was a few minutes ago
19:42:14	dansmith	this happens when they have zuul down and it returns a {}
19:42:16	dansmith	so I assume that's it
19:42:32	mriedem	yeah looks like from infra chatter zuul is down
19:43:10	dansmith	"sweet"
19:45:47	openstackgerrit	Matt Riedemann proposed openstack/nova master: Keep attach_mode as top-level field in _translate_attachment_ref https://review.openstack.org/574413
20:17:33	openstackgerrit	Dan Smith proposed openstack/nova master: Use oslo.messaging per-call monitoring https://review.openstack.org/566696
21:09:10	mriedem	dansmith: did you forget to git add nova/conf/rpc.py?
21:09:16	dansmith	eff
21:09:30	mriedem	heh
21:09:40	openstackgerrit	Dan Smith proposed openstack/nova master: Use oslo.messaging per-call monitoring https://review.openstack.org/566696
21:13:05	mriedem	did you think about creating a new [rpc] group?
21:16:50	openstackgerrit	Julia Kreger proposed openstack/nova master: ironic: bugfix: ensure a host is set for volume connectors https://review.openstack.org/571982
21:27:06	dansmith	mriedem: I can't move rpc_response_timeout, so it didn't seem worth the trouble
21:27:09	dansmith	and potential confusion
21:27:50	openstackgerrit	Eric Fried proposed openstack/nova master: Direct Placement API: ksa Adapter https://review.openstack.org/574470
21:31:05	openstackgerrit	Eric Fried proposed openstack/nova master: Direct Placement API: ksa Adapter https://review.openstack.org/574470
21:35:26	openstackgerrit	Merged openstack/nova master: Document the internal online_migrations function behaviors https://review.openstack.org/574268
21:44:16	openstackgerrit	Dan Smith proposed openstack/nova master: Use oslo.messaging per-call monitoring https://review.openstack.org/566696
21:44:34	openstackgerrit	Eric Fried proposed openstack/nova master: Direct Placement API: ksa Adapter https://review.openstack.org/574470
22:01:21	openstackgerrit	Chris Dent proposed openstack/nova master: Do not use nova.test in placement.test_util https://review.openstack.org/574403
22:01:22	openstackgerrit	Chris Dent proposed openstack/nova master: Do not use nova.test in placement.handlers.test_aggregate https://review.openstack.org/574404
22:01:23	openstackgerrit	Chris Dent proposed openstack/nova master: Do not use nova.test in placement.test_requestlog https://review.openstack.org/574405
22:01:24	openstackgerrit	Chris Dent proposed openstack/nova master: Do not use nova.test in placement.test_fault_wrap https://review.openstack.org/574406
22:01:25	openstackgerrit	Chris Dent proposed openstack/nova master: Do not use nova.test in placement.test_handler https://review.openstack.org/574407
22:01:26	openstackgerrit	Chris Dent proposed openstack/nova master: Do not use nova.test in placement.test_microversion https://review.openstack.org/574408
22:01:27	openstackgerrit	Chris Dent proposed openstack/nova master: Do not use nova.test in placement.test_deploy https://review.openstack.org/574409
23:15:00	openstackgerrit	Matt Riedemann proposed openstack/nova master: Time how long pre_live_migration() takes https://review.openstack.org/574481
23:15:01	openstackgerrit	Matt Riedemann proposed openstack/nova master: DNM: test heartbeat code in pre_live_migration https://review.openstack.org/574482
23:19:36	openstackgerrit	Eric Fried proposed openstack/nova-specs master: Spec: Handling Reshaped Provider Trees https://review.openstack.org/572583
23:48:38	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: Heal allocations with missing/incomplete consumer information https://review.openstack.org/574488
23:49:37	openstackgerrit	Merged openstack/nova master: placement: always create consumer records https://review.openstack.org/567678
#openstack-nova - 2018-06-12
01:40:06	naichuans	mriedem: Hi, Matt. We can set one or more hosts' SR to LVM to check it. But look it can't refect to the test result. Currently, we checked it on an independent host in our company, all the tests are passed.
01:55:02	openstackgerrit	Merged openstack/nova master: Fix typo in enable_certificate_validation config option help https://review.openstack.org/572185
02:04:31	openstackgerrit	Merged openstack/nova master: Mention running rootwrap in daemon mode if hitting vif plug timeouts https://review.openstack.org/571577
02:29:48	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Fix all invalid obj_make_compatible test case https://review.openstack.org/574240

Earlier   Later