Posted	Nick	Remark
#openstack-nova - 2019-12-13
21:21:05	mriedem	i think in this case the query isn't scoped to a project
21:21:07	mriedem	so we should be ok
21:21:46	mriedem	project_only=False by default and read_deleted='yes' means we don't modify the query to filter on deleted
21:21:55	mriedem	so hell we likely don't even need model_query with this
21:22:00	mriedem	just normal sqla orm
21:22:14	zzzeek	woop
21:24:09	mriedem	thanks mike
21:30:27	mriedem	re nova-network removal series, bottom 2 are pretty easy and +2ed https://review.opendev.org/#/c/697151/
21:30:48	mriedem	just removing dead code
21:38:11	gibi	mriedem: kicked the deads
21:43:46	mriedem	pulling an all-nighter!
21:47:24	openstackgerrit	Sundar Nadathur proposed openstack/os-traits master: Add a trait that compute manager can handle accelerator requests. https://review.opendev.org/698961
21:48:01	gibi	mriedem: baby sitting
21:48:34	gibi	mriedem: but not for long
21:50:35	mriedem	back to the future 3 huh
21:52:27	gibi	it worked like a charm
21:54:26	mriedem	how old?
21:55:35	gibi	7. I think he liked the first two too
21:56:19	gibi	he asked for the third so I complied
21:56:23	mriedem	heh
21:56:35	mriedem	is it time for star wars? you could keep him busy for an entire weekend
21:57:11	gibi	he has already consumed half (the better half) of star wars
21:57:38	gibi	and I'm not a star wars fan. I grew up on star trek instead
21:57:59	mriedem	then i will hold my tongue
21:59:40	gibi	but I accept that the general public does not agree with me :)
22:01:33	gibi	I think I stopped star wars when jar jar binks appeared
22:07:38	gibi	now the parents are back so I'm dropping. Have a nice evening
22:21:15	openstackgerrit	Merged openstack/nova stable/pike: Fix listing deleted servers with a marker https://review.opendev.org/690735
22:21:22	openstackgerrit	Merged openstack/nova master: Move _update_pci_request_spec_with_allocated_interface_name https://review.opendev.org/696574
22:21:37	openstackgerrit	Merged openstack/nova master: nova-net: Remove unused 'stub_out_db_network_api' https://review.opendev.org/697151
22:52:06	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add recreate test for bug 1855927 https://review.opendev.org/699045
22:52:06	openstack	bug 1855927 in OpenStack Compute (nova) "_poll_unconfirmed_resizes may not retry later if confirm_resize fails in API" [Low,New] https://launchpad.net/bugs/1855927
23:36:56	openstackgerrit	Eric Fried proposed openstack/nova master: Use Placement 1.35 (root_required) https://review.opendev.org/699050
23:37:08	efried	dansmith: ^
23:41:33	openstackgerrit	Eric Fried proposed openstack/nova master: Use reasonable name for provider mapping https://review.opendev.org/698735
#openstack-nova - 2019-12-14
01:57:16	openstackgerrit	Merged openstack/nova master: Confirm cross-cell resize from the API https://review.opendev.org/637316
01:59:03	openstackgerrit	Merged openstack/nova master: Add revert_snapshot_based_resize_at_dest compute method https://review.opendev.org/637630
01:59:10	openstackgerrit	Merged openstack/nova master: Deal with cross-cell resize in _remove_deleted_instances_allocations https://review.opendev.org/639453
01:59:20	openstackgerrit	Merged openstack/nova master: Add finish_revert_snapshot_based_resize_at_source compute method https://review.opendev.org/637647
01:59:31	openstackgerrit	Merged openstack/nova master: Add RevertResizeTask https://review.opendev.org/638046
02:06:20	openstackgerrit	Eric Fried proposed openstack/nova master: Use Placement 1.35 (root_required) https://review.opendev.org/699050
02:08:38	openstackgerrit	Eric Fried proposed openstack/nova master: Use Placement 1.35 (root_required) https://review.opendev.org/699050
04:15:19	openstackgerrit	Merged openstack/nova master: Sync ImageMetaPropsPayload fields https://review.opendev.org/698613
04:22:19	openstackgerrit	Merged openstack/nova master: Revert "(Temporarily) readd bare support for py27" https://review.opendev.org/698890
05:09:40	openstackgerrit	Merged openstack/nova master: nova-net: Remove db methods for ProviderMethod https://review.opendev.org/697150
06:55:32	openstackgerrit	Brin Zhang proposed openstack/nova master: Support re-configure the delete_on_termination in server https://review.opendev.org/693828
07:00:37	openstackgerrit	Brin Zhang proposed openstack/nova master: PoC: Support re-configure the delete_on_termination in server https://review.opendev.org/693828
10:43:24	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Support live migration with qos ports https://review.opendev.org/695905
10:43:25	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Func test for qos live migration reschedule https://review.opendev.org/699015
10:43:25	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Move common test method up to base class https://review.opendev.org/699064
10:43:26	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Func test for abort and cancel live migration https://review.opendev.org/699065
11:04:18	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Enable live migration with qos ports https://review.opendev.org/699066
21:38:15	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add cross-cell resize tests for _poll_unconfirmed_resizes https://review.opendev.org/698322
21:38:15	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: Add negative test to delete server during cross-cell resize claim https://review.opendev.org/688832
21:38:16	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: Implement reschedule logic for cross-cell resize/migrate https://review.opendev.org/696213
21:38:16	openstackgerrit	Matt Riedemann proposed openstack/nova master: Fix accumulated non-docs nits for cross-cell-resize series https://review.opendev.org/698935
#openstack-nova - 2019-12-15
07:41:36	openstackgerrit	Brin Zhang proposed openstack/nova-specs master: Allow specify user to reset password https://review.opendev.org/682302
11:12:23	openstackgerrit	Lee Yarwood proposed openstack/nova stable/train: Skip cpu comparison on AArch64 https://review.opendev.org/699115
11:12:55	openstackgerrit	Lee Yarwood proposed openstack/nova stable/stein: Skip cpu comparison on AArch64 https://review.opendev.org/699116
11:13:30	openstackgerrit	Lee Yarwood proposed openstack/nova stable/rocky: Skip cpu comparison on AArch64 https://review.opendev.org/699117
13:48:15	openstackgerrit	Merged openstack/nova stable/train: Switch to uses_virtio to enable iommu driver for AMD SEV https://review.opendev.org/698738
13:48:19	openstackgerrit	Merged openstack/nova stable/train: Also enable iommu for virtio controllers and video in libvirt https://review.opendev.org/685756
13:58:05	openstackgerrit	Merged openstack/nova stable/train: Create a controller for qga when SEV is used https://review.opendev.org/698739
13:58:09	openstackgerrit	Merged openstack/nova stable/train: Revert "docs: Highlight the current broken state of SEV" (partially) https://review.opendev.org/698746
14:00:18	openstackgerrit	Merged openstack/nova stable/train: Mask the token used to allow access to consoles https://review.opendev.org/696685
#openstack-nova - 2019-12-16
03:04:28	openstackgerrit	Brin Zhang proposed openstack/nova-specs master: Allow specify user to reset password https://review.opendev.org/682302
03:13:55	openstackgerrit	Luyao Zhong proposed openstack/nova master: address specific resources cleanup issue https://review.opendev.org/699148
03:24:33	openstackgerrit	Boxiang Zhu proposed openstack/nova master: Make evacuation respects anti-affinity rule https://review.opendev.org/649963
03:31:53	openstackgerrit	Boxiang Zhu proposed openstack/nova master: Fix live migration break group policy simultaneously https://review.opendev.org/651969
14:04:42	mriedem	gibi: i dumped some ideas on https://review.opendev.org/#/c/638047/56/nova/conductor/manager.py@1883 - do you have any preferences?
14:04:59	gibi	mriedem: on a meeting for 30 minutes but I will look
14:15:48	gibi	mriedem: responded
14:17:26	mriedem	heh and i just answered that elsewhere https://review.opendev.org/#/c/638048/56/nova/tests/functional/test_cross_cell_migrate.py@595
14:17:38	mriedem	i'll work on a patch today to fix both of those things
14:37:31	mriedem	I can also remove the cache from MigrationContext https://review.opendev.org/#/c/691991/15/nova/objects/migration_context.py@131 in the FUP patch
14:38:46	gibi	mriedem: cool, I can re-review tomorrow at some point
14:39:01	gibi	mriedem: but I'm mostly OK with the series now
14:39:28	mriedem	ack, thanks
14:55:55	mriedem	i put https://blueprints.launchpad.net/nova/+spec/support-move-ops-with-qos-ports-ussuri in a runway slot
14:56:01	mriedem	+1 week for the holiday
14:56:15	mriedem	not that people are really paying attention to runways anymore...
14:56:47	tbachman	pilots
14:57:34	mriedem	it's all auto-pilot now
14:57:56	tbachman	that’s the way I roll when developing
14:58:23	tbachman	explains all the bugs
15:06:13	gibi	mriedem: thanks
15:06:14	KeithMnemonic	good day mriedem: i did an update to fix that issue with osc --marker --deleted. Would you mind giving me your thought on if this is the right direction to go in. Thank you. https://review.opendev.org/691045
15:06:49	KeithMnemonic	it was what we discussed a few weeks ago, just for ID only when --deleted is also used
15:08:48	gibi	mriedem: will you implement the fix for https://review.opendev.org/#/c/638047/56/nova/conductor/manager.py@1883 in a separate commit top of the series? if yes then I will +A the current patch to keep the series moving
15:10:14	mriedem	gibi: yes i'm working on it now
15:10:20	gibi	mriedem: cool.
15:10:58	aarents	lyarwood: I commented out your reply if you may have a look ? https://review.opendev.org/#/c/696084
15:12:39	mriedem	KeithMnemonic: done
15:13:13	mriedem	gibi: i've also removed the cache from https://review.opendev.org/#/c/691991/ in the FUP patch at the end
15:13:20	mriedem	just haven't pushed it yet
15:14:05	gibi	mriedem: thanks. I see your comment about having the whole instance object cached. That seems even more dangerous but dansmith seemed to have a good reason for that cache https://github.com/openstack/nova/commit/7a9942c5924d7ccfe321c54d0b5e70cfb3aede57
15:14:59	mriedem	i think that's probably safer since a Migration record is created with an instance.uuid always i think
15:15:03	mriedem	so it's not going to change
15:15:15	mriedem	gibi: also on the assertion to add in https://review.opendev.org/#/c/638268/56 - can i do that in a FUP as well to keep things moving?
15:15:26	mriedem	you said you were out after today and most of the series is approved so i'd like to FUP what i can

Earlier   Later