Posted	Nick	Remark
#openstack-nova - 2020-01-15
01:41:29	brinzhang	Sundar: creating releasenote guide:https://docs.openstack.org/reno/latest/user/usage.html#creating-new-release-notes
01:55:38	openstackgerrit	Merged openstack/nova master: Print help if nova-manage subcommand is not specified https://review.opendev.org/694806
01:55:45	openstackgerrit	Merged openstack/nova master: nova-net: Copy shared utils from nova-net module https://review.opendev.org/696515
01:55:51	openstackgerrit	Merged openstack/nova master: nova-net: Remove final references to nova-network https://review.opendev.org/696516
01:55:58	openstackgerrit	Merged openstack/nova master: nova-net: Remove 'MetadataManager' https://review.opendev.org/696748
02:12:19	openstackgerrit	Merged openstack/nova master: nova-net: Add TODOs to remove security group-related objects https://review.opendev.org/696517
02:12:26	openstackgerrit	Merged openstack/nova master: nova-net: Remove dependency on nova-net from fake cache https://review.opendev.org/696749
02:12:33	openstackgerrit	Merged openstack/nova master: nova-net: Kill it https://review.opendev.org/696518
02:19:06	openstackgerrit	Merged openstack/nova stable/pike: Skip checking of target_dev for vhostuser https://review.opendev.org/702231
02:25:15	openstackgerrit	Merged openstack/nova stable/pike: cleanup evacuated instances not on hypervisor https://review.opendev.org/687912
03:30:21	openstackgerrit	Merged openstack/nova stable/pike: Move restart_compute_service to a common place https://review.opendev.org/699488
03:30:26	openstackgerrit	Merged openstack/nova stable/pike: Add functional test for resize crash compute restart revert https://review.opendev.org/687913
03:30:31	openstackgerrit	Merged openstack/nova stable/pike: Prevent init_host test to interfere with other tests https://review.opendev.org/687916
03:30:38	openstackgerrit	Merged openstack/nova stable/pike: Functional reproduce for bug 1833581 https://review.opendev.org/687917
03:30:38	openstack	bug 1833581 in OpenStack Compute (nova) train "instance stuck in BUILD state if nova-compute is restarted" [Low,Fix committed] https://launchpad.net/bugs/1833581 - Assigned to Balazs Gibizer (balazs-gibizer)
04:34:15	openstackgerrit	Merged openstack/nova master: pre-commit: Use Python 3 to run checks https://review.opendev.org/702453
06:25:07	openstackgerrit	Dustin Cowles proposed openstack/nova master: Provider Config File: Function to further validate and retrieve configs https://review.opendev.org/676029
06:25:07	openstackgerrit	Dustin Cowles proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree https://review.opendev.org/676522
06:25:08	openstackgerrit	Dustin Cowles proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs https://review.opendev.org/693460
09:14:28	openstackgerrit	Arnaud Morin proposed openstack/nova master: Force refresh of instance info cache on event https://review.opendev.org/702394
09:22:24	openstackgerrit	Riccardo Pittau proposed openstack/nova master: Fix typos for update_available_resource reference https://review.opendev.org/702602
10:00:10	openstackgerrit	waleed mousa proposed openstack/os-vif master: [Follow Up] OVS DPDK port representors support https://review.opendev.org/665965
10:29:09	openstackgerrit	Stephen Finucane proposed openstack/nova stable/train: Print help if nova-manage subcommand is not specified https://review.opendev.org/702637
10:29:40	brinzhang	efried, alex_xu: https://review.opendev.org/#/c/699669/ this spec was +2 by gibi, and sean-k-mooney +1, could you please review this, determinate if it does to go.
10:31:09	brinzhang	stephenfin: could you please review this bug fix https://review.opendev.org/#/c/580271/, it is ready.
10:31:50	openstackgerrit	Stephen Finucane proposed openstack/nova master: functional: Add unified '_build_server' helper function https://review.opendev.org/697537
10:31:50	openstackgerrit	Stephen Finucane proposed openstack/nova master: functional: Add unified '_(build|create)_flavor' helper functions https://review.opendev.org/697538
10:31:51	openstackgerrit	Stephen Finucane proposed openstack/nova master: functional: Add '_delete_server' to 'InstanceHelperMixin' https://review.opendev.org/697539
10:31:51	openstackgerrit	Stephen Finucane proposed openstack/nova master: WIP: functional: Add '_create_server' helper https://review.opendev.org/697540
10:32:31	stephenfin	brinzhang: sure
10:32:53	brinzhang	stephenfin: cool, thanks ^^
10:44:05	luyao	stephenfin: Do you have time look at spec 'support live migration with vpmem' today? https://review.opendev.org/#/c/695863'
10:44:16	stephenfin	yup, will do (y)
10:44:39	luyao	stephenfin: thanks. :D
10:54:56	openstackgerrit	Stephen Finucane proposed openstack/nova master: nova-net: Remove unused exceptions https://review.opendev.org/697149
10:54:56	openstackgerrit	Stephen Finucane proposed openstack/nova master: nova-net: Remove unnecessary 'neutronv2' prefixes https://review.opendev.org/696776
10:54:57	openstackgerrit	Stephen Finucane proposed openstack/nova master: nova-net: Remove layer of indirection in 'nova.network' https://review.opendev.org/696745
10:54:57	openstackgerrit	Stephen Finucane proposed openstack/nova master: nova-net: Make the security group API a module https://review.opendev.org/696746
10:54:58	openstackgerrit	Stephen Finucane proposed openstack/nova master: nova-net: Remove remaining nova-network quotas https://review.opendev.org/697152
10:54:58	openstackgerrit	Stephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'FloatingIP' object https://review.opendev.org/697153
10:54:59	openstackgerrit	Stephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'Network' object https://review.opendev.org/697154
10:54:59	openstackgerrit	Stephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'SecurityGroup' object https://review.opendev.org/697155
10:55:00	openstackgerrit	Stephen Finucane proposed openstack/nova master: nova-net: Remove now unnecessary nova-net workaround https://review.opendev.org/702440
10:55:00	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove 'nova.image.api' module https://review.opendev.org/702451
10:55:01	openstackgerrit	Stephen Finucane proposed openstack/nova master: WIP: nova-net: Remove unused nova-network objects https://review.opendev.org/697156
10:58:19	openstackgerrit	Stephen Finucane proposed openstack/nova master: nova-net: Remove layer of indirection in 'nova.network' https://review.opendev.org/696745
10:58:20	openstackgerrit	Stephen Finucane proposed openstack/nova master: nova-net: Make the security group API a module https://review.opendev.org/696746
10:58:20	openstackgerrit	Stephen Finucane proposed openstack/nova master: nova-net: Remove remaining nova-network quotas https://review.opendev.org/697152
10:58:21	openstackgerrit	Stephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'FloatingIP' object https://review.opendev.org/697153
10:58:21	openstackgerrit	Stephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'Network' object https://review.opendev.org/697154
10:58:22	openstackgerrit	Stephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'SecurityGroup' object https://review.opendev.org/697155
10:58:22	openstackgerrit	Stephen Finucane proposed openstack/nova master: nova-net: Remove now unnecessary nova-net workaround https://review.opendev.org/702440
10:58:23	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove 'nova.image.api' module https://review.opendev.org/702451
10:58:23	openstackgerrit	Stephen Finucane proposed openstack/nova master: WIP: nova-net: Remove unused nova-network objects https://review.opendev.org/697156
11:08:33	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove 'nova.image.api' module https://review.opendev.org/702451
11:08:33	openstackgerrit	Stephen Finucane proposed openstack/nova master: WIP: nova-net: Remove unused nova-network objects https://review.opendev.org/697156
11:09:11	stephenfin	gibi: Could you have a look at https://review.opendev.org/#/c/695733/, when you have a chance?
11:24:56	brinzhang	stephenfin: about https://review.opendev.org/#/c/580271/6/nova/compute/api.py@5258 what do you mean, add "aggregate.updated_at = timeutils.utcnow()" to that api? https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/aggregates.py#L179
11:25:19	brinzhang	stephenfin: under line179
11:25:56	stephenfin	brinzhang: No, I'm referring to the 'Aggregate.remove_host' method in 'nova.objects.aggregate'
11:26:30	stephenfin	brinzhang: So in here https://github.com/openstack/nova/blob/master/nova/objects/aggregate.py#L394-L401
11:26:39	gibi	stephenfin: ack, added to my queue
11:26:45	stephenfin	ta
11:27:51	kashyap	Potentially stupid question, what's invalid about this syntax here: https://paste.centos.org/view/5d8c9d97
11:28:35	stephenfin	kashyap: it's only invalid on Python 2
11:28:48	stephenfin	you need to use python3 now
11:30:13	stephenfin	kashyap: Look at https://github.com/openstack/nova/blob/master/nova/compute/utils.py#L1374-L1375 then look at https://www.python.org/dev/peps/pep-3102/
11:31:21	kashyap	stephenfin: Ah, darn, when I modified my single test-run I forgot to pass py36
11:31:28	kashyap	Thanks for the references :-)
11:31:52	stephenfin	np
11:32:46	brinzhang	stephenfin: I will check, thanks
11:54:19	openstackgerrit	Kashyap Chamarthy proposed openstack/nova master: libvirt: Add a default Virtio RNG device to guests https://review.opendev.org/616603
12:07:27	openstackgerrit	Merged openstack/os-traits master: Add COMPUTE_RESCUE_BFV trait https://review.opendev.org/694033
12:24:43	openstackgerrit	Brin Zhang proposed openstack/nova master: Populate 'updated_at' when aggregate is updated https://review.opendev.org/580271
12:26:47	brinzhang	stephenfin: I removed the duplicate handle in compute api. In the object.aggregate was done it, updated.
14:03:57	kashyap	sean-k-mooney: Hey, finally got around to this RNG thing, mind having a gander when you can - https://review.opendev.org/#/c/616603/
14:16:07	Sundar	dansmith, efried: I am here if there are any questions on https://review.opendev.org/#/c/631244/ or related stuff.
14:37:10	dansmith	Sundar: I just got up, so I haven't seen, but I'm assuming you pushed a new rev?
14:38:02	Sundar	I published comments, will push patches after you agree.
14:40:05	Sundar	dansmith: Thanks, no rush
14:40:33	Sundar	brinzhang: Thanks for the release notes pointers
14:41:17	stephenfin	kashyap: I hit that
14:41:47	kashyap	stephenfin: Thank you
14:43:07	kashyap	Okay, read your remarks, will respond there.
14:47:55	sean-k-mooney	kashyap: sure ill take a look
14:48:02	kashyap	Thanks
14:48:39	sean-k-mooney	Sundar: when you push the new version be sure to rebase all the patches in the chain against master
14:49:26	Sundar	Yes, already done internally
14:49:31	sean-k-mooney	Sundar: i deployed https://review.opendev.org/#/c/699554/2 yesterday but it does not work because it is missing depencies
14:51:01	Sundar	sean-k-mooney: I think you mean the root_required patch, which has been merged now?
14:51:20	sean-k-mooney	yes
14:52:00	sean-k-mooney	on the cyborg side i notice the device v2 api has merged on monday/tuseday
14:52:41	sean-k-mooney	are the arq v2 https://review.opendev.org/#/c/696089/ and deployable v2 https://review.opendev.org/#/c/698190/ changes required for this work?
14:54:07	sean-k-mooney	i deployed with cyborg master which had the device v2 changes when i was testing
14:54:27	Sundar	sean-k-mooney: Yes, I am updating the patch sets with these dependencies
14:55:33	Sundar	sean-k-mooney: Yes, device v2 API merged, but that should be tangential to Nova. The main thing that Nova patches needs is https://review.opendev.org/698846
14:55:42	Sundar	*need
14:58:00	sean-k-mooney	ok well i was trying to use the cyborg placemtn client so haveing the api to validate the existing of the fake device and compare with placment was useful
14:58:02	Sundar	sean-k-mooney: You don;t need https://review.opendev.org/#/c/698190/. Looking at the other one
14:58:35	openstackgerrit	Stephen Finucane proposed openstack/nova master: nova-net: Remove layer of indirection in 'nova.network' https://review.opendev.org/696745

Earlier   Later