Posted	Nick	Remark
#openstack-nova - 2019-07-12
15:42:37	openstackgerrit	Stephen Finucane proposed openstack/nova master: Integrate 'pre-commit' https://review.opendev.org/665518
15:44:01	openstack	Launchpad bug 1829896 in OpenStack Compute (nova) "libvirt: "Instance failed to shutdown in 60 seconds." in the gate" [Undecided,New]
15:44:01	mriedem	if someone is looking for something to investigate/debug next week which should help speed up tempest run times for anything that requires stopping a server https://bugs.launchpad.net/nova/+bug/1829896 (like shelve, stop/start, etc)
15:46:26	mriedem	"While performing stop, rescue and shelve, rebuild operations"
15:46:36	mriedem	yikes at 60 seconds a pop that's a lot of wasted time in the gate
15:53:20	mriedem	here is a devstack change to work around that https://review.opendev.org/670591
15:53:33	mriedem	which is probably something we should have done in the first place
15:55:59	mriedem	gmann: ^ probably of interest to you in speeding up gate times
16:05:42	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Device tags: don't pass pf_interface=True to get_mac_by_pci_address https://review.opendev.org/670593
16:23:19	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: Convert nova-next to a zuul v3 job https://review.opendev.org/670196
16:24:01	mriedem	efried: just fyi you can probably remove https://review.opendev.org/#/q/topic:bp/add-host-and-hypervisor-hostname-flag-to-create-server+(status:open+OR+status:merged) from the runway slot since the only thing not yet merged is the docs patch (but it's approved) and the osc change which is dependent on the novaclient release
16:24:23	efried	mriedem: ack, thanks
16:25:12	mriedem	with that i think i'm out, have fun
17:25:52	openstackgerrit	Merged openstack/nova master: Revert resize: wait for events according to hybrid plug https://review.opendev.org/667177
17:27:57	artom	\o/
17:42:54	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Pass migration to finish_revert_migration() https://review.opendev.org/668631
17:42:54	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Add functional test for resize crash compute restart revert https://review.opendev.org/670393
17:42:55	openstackgerrit	Artom Lifshitz proposed openstack/nova master: [DNM] testing bug/1813789 revert resize events https://review.opendev.org/664442
19:38:48	spatel	sean-k-mooney: yt?
19:50:09	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Multiple API cleanup changes https://review.opendev.org/666889
20:01:11	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Multiple API cleanup changes https://review.opendev.org/666889
20:03:44	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Multiple API cleanup changes https://review.opendev.org/666889
20:20:07	openstackgerrit	Artom Lifshitz proposed openstack/nova stable/stein: Revert resize: wait for events according to hybrid plug https://review.opendev.org/670645
20:22:09	openstackgerrit	Artom Lifshitz proposed openstack/nova stable/rocky: Revert resize: wait for events according to hybrid plug https://review.opendev.org/670648
20:23:10	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Multiple API cleanup changes https://review.opendev.org/666889
20:47:33	openstackgerrit	François Palin proposed openstack/nova master: Call volume detach even if terminate_connection fails https://review.opendev.org/669674
23:16:41	openstackgerrit	Artom Lifshitz proposed openstack/nova stable/stein: Revert resize: wait for events according to hybrid plug https://review.opendev.org/670645
23:24:01	openstackgerrit	Artom Lifshitz proposed openstack/nova stable/rocky: Revert resize: wait for events according to hybrid plug https://review.opendev.org/670648
#openstack-nova - 2019-07-13
00:34:11	openstackgerrit	Merged openstack/nova master: Remove needs:* todo from deprecated APIs api-ref https://review.opendev.org/668234
00:44:24	openstackgerrit	Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for node.list https://review.opendev.org/656027
00:44:25	openstackgerrit	Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for setting instance id https://review.opendev.org/659690
00:44:25	openstackgerrit	Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for validating instance and node https://review.opendev.org/656028
00:44:26	openstackgerrit	Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for getting network metadata from node https://review.opendev.org/670213
00:44:26	openstackgerrit	Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for add/remove instance info from node https://review.opendev.org/659691
04:43:57	openstackgerrit	Merged openstack/nova master: Fix no propagation of nova context request_id https://review.opendev.org/662715
04:49:48	openstackgerrit	Brin Zhang proposed openstack/nova master: Specify availability_zone to unshelve https://review.opendev.org/663851
11:41:31	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Multiple API cleanup changes https://review.opendev.org/666889
12:15:47	openstackgerrit	Takashi NATSUME proposed openstack/nova stable/stein: Fix no propagation of nova context request_id https://review.opendev.org/670694
#openstack-nova - 2019-07-14
02:59:27	openstackgerrit	Takashi NATSUME proposed openstack/nova stable/stein: Fix no propagation of nova context request_id https://review.opendev.org/670694
07:37:52	openstackgerrit	Brin Zhang proposed openstack/nova master: Specify availability_zone to unshelve https://review.opendev.org/663851
14:28:34	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Pass migration to finish_revert_migration() https://review.opendev.org/668631
14:28:34	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Add functional test for resize crash compute restart revert https://review.opendev.org/670393
14:28:35	openstackgerrit	Artom Lifshitz proposed openstack/nova master: [DNM] testing bug/1813789 revert resize events https://review.opendev.org/664442
#openstack-nova - 2019-07-15
01:26:17	openstackgerrit	Arthur Dayne proposed openstack/python-novaclient stable/rocky: Add Python 3 Train unit tests https://review.opendev.org/670749
01:27:04	openstackgerrit	ya.wang proposed openstack/nova master: Add compatibility checks for CPU mode and CPU models and extra flags https://review.opendev.org/670299
01:27:05	openstackgerrit	ya.wang proposed openstack/nova master: Add release note https://review.opendev.org/670441
01:27:05	openstackgerrit	ya.wang proposed openstack/nova master: Support report multi CPU model traits https://review.opendev.org/670300
03:19:08	openstack	Launchpad bug 1836141 in OpenStack Compute (nova) "vm resize failed due to the remains left by failed actions" [Undecided,New]
03:19:08	Spencer_Yu	https://bugs.launchpad.net/nova/+bug/1836141 Some failed actions would leave the remains in resize instance, which will lead next resize to fail.
04:37:29	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: DRY get_flavor in flavor manage tests https://review.opendev.org/668281
06:54:30	openstackgerrit	Brin Zhang proposed openstack/nova-specs master: Proposal for a safer noVNC console with password authentication https://review.opendev.org/623120
08:23:56	alex_xu	efried: bauzas need your help on the direction https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:master+topic:claim_for_instance
09:38:16	sean-k-mooney	stephenfin: do you know the history behind why we went form testr to ostestr to stestr? im wondering if we should consider regerting to testr or look at finding a non subunit based alternitive
09:40:55	sean-k-mooney	https://blog.kortar.org/?p=370
09:41:54	sean-k-mooney	found the history ^
09:42:36	stephenfin	sean-k-mooney: I was pretty sure testr was subunit-based, tbh
09:42:54	sean-k-mooney	it is
09:43:02	sean-k-mooney	that does not mean it has the same bug
09:57:10	cdent	sean-k-mooney: are you talking about the overflow bug with too much logging? or something else?
10:04:23	sean-k-mooney	cdent: ya we are now hitting it downstream
10:04:51	sean-k-mooney	RHEL8 ship newer versions of some libs then upper constratis uses for stien
10:05:11	sean-k-mooney	cdent: so we get more deprecation warnings then upstream
10:05:19	cdent	the same bug is present in testr and ostestr (which is a testr wrapper). My understanding is the issue is in testtools and/or subunit not testr or stestr
10:05:43	cdent	and it really comes down to a binary data structure being too small
10:05:54	cdent	it should be per test
10:06:05	sean-k-mooney	cdent: well yes and no
10:06:12	sean-k-mooney	it is in subunit parser
10:06:20	sean-k-mooney	but the subunit protocal
10:06:33	cdent	so finding and turning off the deprecation warnings (or simply fixing them) is probably the easiest thing
10:06:35	sean-k-mooney	allows large packets to be fragmented
10:06:47	sean-k-mooney	i think the issue is we are not fragmenting the stream
10:07:04	sean-k-mooney	cdent: ya that is how we are fixing it upstream
10:07:18	sean-k-mooney	but downstream that is more of a challange
10:07:43	sean-k-mooney	it either requires backporting things that may not be backported upstream or downstream only patches
10:13:35	sean-k-mooney	cdent: i was wondering if there was a way to redirect all deperecation warnings to a seperate file so that it does not end up in the test output
10:14:05	sean-k-mooney	we should be able to install a python logging filter i think but i havent tried
10:15:15	cdent	yeah, that does appear to be the case
10:18:43	sean-k-mooney	the trick would be using a generic enough regex to match deprecations without redirecting real errors. we would also want the gate jobs to copy the deprecation file so that we can actully track how big it is and squash them
10:22:57	sean-k-mooney	it lookse like we already filter a bunch of waring here https://github.com/openstack/nova/blob/master/nova/tests/fixtures.py#L815-L877
10:24:02	sean-k-mooney	cdent: actully https://github.com/openstack/nova/blob/master/nova/tests/fixtures.py#L820 should be makeing deprecation warning print once
10:24:10	sean-k-mooney	but maybe that is per test?
10:24:49	sean-k-mooney	i would have assumed it would be per logger but maybe we dont use the warning filter in all places we should
10:26:10	cdent	could be that it is not turned on everywhere? there are some tests that don't use the usual base?
10:27:34	sean-k-mooney	the test that is exploding downswtream is the test_instace_action functional test which runns a bunch of tests interall https://github.com/openstack/nova/blob/7279d6fa009c6e276188bcad0ad5a1832849a4f9/nova/tests/functional/notification_sample_tests/test_instance.py#L357-L377
10:28:13	sean-k-mooney	it is not direcly using the fixtre but ill look at its inheritance tree
10:28:26	sean-k-mooney	maybe we dont use it in the functional tests
10:29:37	gibi	sean-k-mooney: does not this solved the problem https://review.opendev.org/#/c/656844/ ?
10:30:56	sean-k-mooney	i can check if we have that on osp 15 which is stine but migi was saying the backport he has tried sofar do not fix it for us
10:31:28	sean-k-mooney	we dont have auto import of backports so its possible we are missing that one
10:36:17	sean-k-mooney	gibi: ya we have that and no it does not fix it for us
10:43:45	sean-k-mooney	cdent: looks like the waring filter is installed for the functional tests
10:43:54	sean-k-mooney	so i guess we are still looking too much
10:44:21	sean-k-mooney	part of the issue is i think stestr is creating a log stream per worker not per test
12:56:31	hongda	Hello everyone. Can you help to review: https://review.opendev.org/#/c/670016/ and https://review.opendev.org/#/c/669867/ ? They tried to fix live-migration failure when token expires. Thanks a lot. XD
13:00:46	sean-k-mooney	we would have to fix this on master first before https://review.opendev.org/#/c/670016/ can be applied
13:00:54	sean-k-mooney	why is that stable only by the way
13:01:54	efried	alex_xu: Talk to me
13:05:34	sean-k-mooney	hongda: im not sure we should just blindly use the admin context
13:06:10	sean-k-mooney	we could be it would proably be better to only use the admin context if the token had expired
13:07:15	sean-k-mooney	hongda: it also looks like you are reusing an old bug that was closed in 2017

Earlier   Later