Earlier  
Posted Nick Remark
#openstack-nova - 2020-01-15
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
14:58:36 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Make the security group API a module https://review.opendev.org/696746
14:58:36 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove remaining nova-network quotas https://review.opendev.org/697152
14:58:37 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'FloatingIP' object https://review.opendev.org/697153
14:58:37 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'Network' object https://review.opendev.org/697154
14:58:38 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'SecurityGroup' object https://review.opendev.org/697155
14:58:38 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove now unnecessary nova-net workaround https://review.opendev.org/702440
14:58:39 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'nova.image.api' module https://review.opendev.org/702451
14:58:39 openstackgerrit Stephen Finucane proposed openstack/nova master: WIP: nova-net: Remove unused nova-network objects https://review.opendev.org/697156

Earlier   Later