Posted	Nick	Remark
#openstack-nova - 2019-12-18
17:28:08	openstackgerrit	Matt Riedemann proposed openstack/nova master: Use COMPUTE_SAME_HOST_COLD_MIGRATE trait during migrate https://review.opendev.org/695220
17:44:17	openstackgerrit	Merged openstack/nova master: Zuul v3: use devstack-plugin-nfs-tempest-full https://review.opendev.org/699516
17:44:23	openstackgerrit	Merged openstack/nova master: Add troubleshooting doc about rebuilding the placement db https://review.opendev.org/698517
17:44:30	openstackgerrit	Merged openstack/nova master: Remove dead code from MigrationTask._execute https://review.opendev.org/697699
18:01:36	mriedem	stephenfin: reviewed https://review.opendev.org/#/c/679502/
18:03:34	stephenfin	thanks
18:21:22	openstackgerrit	Luigi Toscano proposed openstack/nova stable/train: Zuul v3: use devstack-plugin-nfs-tempest-full https://review.opendev.org/699753
18:28:06	openstackgerrit	Merged openstack/nova master: Optimization for nova-api _checks_for_create_and_rebuild https://review.opendev.org/698125
18:36:43	KeithMnemonic1	I know the holidays are coming up, but i appreciate any more reviews to help this get merged before the year ends https://review.opendev.org/691045
19:57:59	mriedem	melwitt: if you're around the bottom 4 nova-net removal patches are pretty easy, starts at https://review.opendev.org/#/c/696510/
21:17:36	gmann	stephenfin: did not get your comment on this - https://review.opendev.org/#/c/657698/14
22:07:47	mriedem	get yer gate bugs hyeah https://bugs.launchpad.net/nova/+bug/1856902
22:07:47	openstack	Launchpad bug 1856902 in OpenStack Compute (nova) "nova.tests.functional.libvirt.test_vpmem.VPMEMTests.test_create_servers_with_vpmem fails intermittently" [Medium,Confirmed]
#openstack-nova - 2019-12-19
00:06:41	openstackgerrit	Merged openstack/nova master: Revert cross-cell resize from the API https://review.opendev.org/638048
02:27:36	luyao	artom, stephenfin, sean-k-mooney: spec 'support live migration with vpmem' updated. https://review.opendev.org/695863
02:28:24	artom	lyarwood, ack, hopefully I can get to it before my holiday PTO on friday
02:29:07	luyao	artom: thanks
04:38:07	openstackgerrit	Merged openstack/nova master: Remove now invalid cells v1 comments from conductor code https://review.opendev.org/697670
06:07:30	openstackgerrit	Brin Zhang proposed openstack/nova-specs master: Add action event fault details https://review.opendev.org/699669
06:58:54	openstackgerrit	Merged openstack/nova stable/rocky: Add --dry-run option to heal_allocations CLI https://review.opendev.org/698525
06:58:59	openstackgerrit	Merged openstack/nova stable/rocky: Add --instance option to heal_allocations https://review.opendev.org/698529
06:59:05	openstackgerrit	Merged openstack/nova stable/rocky: Add BFV wrinkle to TestNovaManagePlacementHealAllocations https://review.opendev.org/698531
07:15:59	openstackgerrit	Merged openstack/nova stable/train: Zuul v3: use devstack-plugin-nfs-tempest-full https://review.opendev.org/699753
07:41:17	openstackgerrit	Luigi Toscano proposed openstack/nova stable/stein: Zuul v3: use devstack-plugin-nfs-tempest-full https://review.opendev.org/699924
12:17:13	stephenfin	dtantsur: Wanna give me core on x/wsme so I can go merge a load of my own stuff? :) Maintainers seem a bit thin on the ground
13:15:25	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Support live migration with qos ports https://review.opendev.org/695905
13:15:26	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Func test for qos live migration reschedule https://review.opendev.org/699015
13:15:26	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Move common test method up to base class https://review.opendev.org/699064
13:15:27	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Func test for abort and cancel live migration https://review.opendev.org/699065
13:15:27	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Enable live migration with qos ports https://review.opendev.org/699066
13:15:28	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Fix get_request_group_mapping doc https://review.opendev.org/699979
13:20:32	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Support live migration with qos ports https://review.opendev.org/695905
13:20:33	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Fix get_request_group_mapping doc https://review.opendev.org/699979
13:20:33	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Func test for qos live migration reschedule https://review.opendev.org/699015
13:20:34	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Move common test method up to base class https://review.opendev.org/699064
13:20:34	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Func test for abort and cancel live migration https://review.opendev.org/699065
13:20:35	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Enable live migration with qos ports https://review.opendev.org/699066
13:44:57	shilpasd	zzzeek: hi
14:22:26	mriedem	relatively simple docs bug if someone wants it https://bugs.launchpad.net/nova/+bug/1857013
14:22:26	openstack	Launchpad bug 1857013 in OpenStack Compute (nova) "Configure live migrations in nova - timeout section for kvm doesn't mention live_migration_timeout_action" [Low,New]
14:25:44	openstackgerrit	Matt Riedemann proposed openstack/nova stable/stein: Zuul v3: use devstack-plugin-nfs-tempest-full https://review.opendev.org/699924
14:27:08	tosky	mriedem: oh, do you nova people require a cherry pick footer for each item in the chain?
14:27:23	mriedem	tosky: preferred
14:27:29	mriedem	to see that they are going in order
14:27:38	mriedem	since it was a clean cherry-pick i just fixed it using the gerrit ui
14:27:45	tosky	yep, noticed
14:28:01	mriedem	alex_xu: this functional test race failure is spiking http://status.openstack.org/elastic-recheck/#1856902
14:28:09	mriedem	i'm inclined to skip that bug until someone can post a fix
14:28:18	mriedem	stephenfin: ^ any thoughts on that?
14:36:12	shilpasd	mriedem:hi
14:36:34	mriedem	oh is see the bug
14:37:21	shilpasd	mriedem: for https://review.opendev.org/#/c/694462/3/nova/db/sqlalchemy/api.py@2010, i have made changes suggested, but its returning list of tuples
14:37:47	mriedem	stephenfin: i think the issue is the delete server helper in the libvirt functional tests is no longer waiting for the server to be gone
14:37:56	mriedem	probably due to the big refactoring going on
14:38:59	mriedem	shilpasd: ok, i guess you'd have to debug it
14:39:26	mriedem	stephenfin: or this was just always racy and we're hitting fast nodes
14:42:49	shilpasd	mriedem: can you please check 'http://paste.openstack.org/show/78779' lines 40 and 41
14:43:19	openstackgerrit	Matt Riedemann proposed openstack/nova master: Fix race in test_create_servers_with_vpmem https://review.opendev.org/699997
14:43:21	mriedem	stephenfin: melwitt: ^
14:43:21	shilpasd	mriedem: i need to iterate again to have list of uuids
14:43:46	mriedem	that paste doesn't work
14:45:54	shilpasd	mriedem: means its wrong approach?
14:46:09	mriedem	i mean your paste url is broken
14:47:18	shilpasd	http://paste.openstack.org/show/787793/
14:47:28	shilpasd	corrected one
14:48:24	mriedem	i don't know if filter(and_(...)) works the same as filter_by chained together, i generally have to play with sqlalchemy orm stuff when doing things like this
14:48:28	mriedem	which is what i meant by debugging
14:49:38	shilpasd	okay, will do that and check
14:50:02	shilpasd	tried filter_by but how 'in_' parameter works with filter_by
14:50:20	shilpasd	models.BlockDeviceMapping.instance_uuid.in_(instance_uuids)
14:57:04	mriedem	shilpasd: if i knew the answer off the top of my head i'd tell you
14:57:08	mriedem	you're going to have to play with it
14:57:11	mriedem	as would i
14:59:47	shilpasd	mriedem: yes, will do it
15:04:50	mriedem	shilpasd: this works for me http://paste.openstack.org/show/787794/
15:07:17	shilpasd	mriedem: thanks will add, this is more better that what i had
15:07:22	shilpasd	will check this
15:22:38	shilpasd	mriedem: it failing if we have list items more than 1, will look further how to resolve this
15:23:23	shilpasd	mriedem: and regarding deletion of TC 'https://review.opendev.org/#/c/694462/3/nova/tests/unit/db/test_db_api.py@993'
15:24:06	shilpasd	i have removed since observed that deleted instance are not considered in 'os-simple-tenant-usage'
15:24:09	shilpasd	so removed
15:24:44	tosky	mriedem: re the devstack-plugin-nfs job backport: I think it should work flawlessly up to pike, but I'm a bit unsure what to do with ocata
15:25:11	tosky	because devstack/ocata lacks most of the infrastructure for zuul v3 jobs (mainly roles)
15:26:06	tosky	would it be fine to copy the legacy job in-tree? After cleaning up the legacy job also in cinder, I'd like to remove it from project-config
15:29:48	stephenfin	mriedem: Skip it
15:30:20	stephenfin	I'm in a meeting rn but I can work on switching that test to use events instead of polling status, since that should be less racey
15:31:09	mriedem	stephenfin: i already have a fix posted https://review.opendev.org/699997
15:31:44	mriedem	tosky: i would just remove it from queens, pike and ocata
15:32:01	mriedem	it's a non-voting experimental queue job so it doesn't get noticed anyway, and those branches are in extended maintenance mode
15:32:27	tosky	mriedem: oh, ok, thanks
15:32:43	mriedem	shilpasd: you shouldn't remove db api test code just because of how one caller is going to use this
15:33:08	mriedem	shilpasd: you have to stop thinking about the db api layer for this as something solely for the simple tenant usage stuff, i.e. not using the active_by_window_joined method for testing it
15:33:08	stephenfin	I wonder why that suddenly changed. None of my changes touched that file :/
15:33:13	stephenfin	Except for 7ae1a1091326292b02d879ae5410d5e7e78d3997 but that's unrelated
15:33:21	mriedem	stephenfin: maybe just hitting faster nodes
15:33:27	stephenfin	Could be
15:33:30	mriedem	when it fails it's on some rax nodes
15:33:38	mriedem	fewer noisy neighbors right now
15:33:44	stephenfin	+W'd anyway
15:33:46	stephenfin	nice find

Earlier   Later