Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-01-14
16:51:25 openstackgerrit Balazs Gibizer proposed openstack/nova master: Remove extra instance.save() calls related to qos SRIOV ports https://review.opendev.org/702261
16:51:26 openstackgerrit Balazs Gibizer proposed openstack/nova master: Use common server create function for qos func tests https://review.opendev.org/701353
16:52:21 sean-k-mooney my main two feature that were important for this cycle have already merged so the rest are less imporant. i want to finish stuff i have started before doing new stuff too so im lean to not filing another blueprint but i also know its will be trivail to do
16:53:05 openstackgerrit Balazs Gibizer proposed openstack/nova master: Enable live migration with qos ports https://review.opendev.org/699066
18:18:47 openstackgerrit Matt Riedemann proposed openstack/nova master: Use COMPUTE_SAME_HOST_COLD_MIGRATE trait during migrate https://review.opendev.org/695220
20:12:24 efried dansmith: old, simple, should be an easy +A: https://review.opendev.org/#/c/694806/
20:23:11 dustinc new behavior: since the no-op entry gets removed during validation, the $COMPUTE_NODE value gets used when merging
20:23:11 dustinc old behavior: if using uuid=$COMPUTE_NODE you could exempt individual providers from the $COMPUTE_NODE entry during merge step by adding specific no-op entries for them
20:23:11 dustinc I realized that there was an unintended change in functionality and am not sure if it is worth worrying about or not
20:23:11 dustinc efried: I was working on moving checks for no-op providers from the merge step to the validation step as you suggested here https://review.opendev.org/#/c/676029/31/nova/compute/provider_config.py@199
20:24:48 dustinc the old behavior was not specifically intended or documented, but might actually be useful to someone
20:25:36 efried dustinc: interesting, glad we caught it before we landed this rather than later when somebody *was* relying on it and we yanked the rug. We should noodle whether we want to allow that or not.
20:26:06 efried I'm going to say: for now, we should explicitly document (wherever we're going to document this thing) that no-op stanzas will be ignored completely and have no effect.
20:26:19 efried which means the new way you're doing the check will be the right way.
20:26:48 efried "ignored completely" with a warning (which I guess isn't ignored completely, but you get the idea)
20:39:05 dustinc Sounds good to me, I want to push what I have so will go that route for now
21:25:59 openstackgerrit Eric Fried proposed openstack/nova master: nova-net: Kill it https://review.opendev.org/696518
21:49:34 openstackgerrit Dustin Cowles proposed openstack/nova master: Provider Config File: YAML file loading and schema validation https://review.opendev.org/673341
21:49:35 openstackgerrit Dustin Cowles proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree https://review.opendev.org/676522
21:49:35 openstackgerrit Dustin Cowles proposed openstack/nova master: Provider Config File: Function to further validate and retrieve configs https://review.opendev.org/676029
21:49:36 openstackgerrit Dustin Cowles proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs https://review.opendev.org/693460
22:06:25 Sundar Hello all, in release notes like https://github.com/openstack/nova/blame/master/releasenotes/notes/image-precaching-d46506568fefa1ea.yaml, what is the id that gets added at the end?
22:23:20 dansmith Sundar: don't hand-create them
22:23:38 dansmith Sundar: tox -evenv reno new some-short-name-of-thing
22:24:58 Sundar dansmith: do I use that to create a file which I then 'git add'?
22:25:05 dansmith Sundar: yes
22:25:11 Sundar Thanks
22:31:17 openstackgerrit Merged openstack/nova master: Avoid spurious error logging in _get_compute_nodes_in_db https://review.opendev.org/695453
23:10:25 efried dustinc: finished reviewing the provider config series. Getting close, nice work.
23:12:49 dustinc efried: thanks, checking it out now..
23:16:48 efried aaand I'm out
#openstack-nova - 2020-01-15
01:34:51 brinzhang Sunda: just use "reno new < some-short-name-of-thing>" under the project also can create the releasenote file, that depend-on reno packages, so you should installed in your env.
01:35:56 brinzhang Sundar
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 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)
03:30:38 openstackgerrit Merged openstack/nova stable/pike: Functional reproduce for bug 1833581 https://review.opendev.org/687917
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: Functions to merge provider configs to provider tree https://review.opendev.org/676522
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: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|create)_flavor' helper functions https://review.opendev.org/697538
10:31:50 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Add unified '_build_server' helper function https://review.opendev.org/697537
10:31:51 openstackgerrit Stephen Finucane proposed openstack/nova master: WIP: functional: Add '_create_server' helper https://review.opendev.org/697540
10:31:51 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Add '_delete_server' to 'InstanceHelperMixin' https://review.opendev.org/697539
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 unnecessary 'neutronv2' prefixes https://review.opendev.org/696776
10:54:56 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove unused exceptions https://review.opendev.org/697149
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:57 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove layer of indirection in 'nova.network' https://review.opendev.org/696745
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:58 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove remaining nova-network quotas https://review.opendev.org/697152
10:54:59 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'SecurityGroup' object https://review.opendev.org/697155
10:54:59 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'Network' object https://review.opendev.org/697154
10:55:00 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'nova.image.api' module https://review.opendev.org/702451
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: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: Remove remaining nova-network quotas https://review.opendev.org/697152
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:21 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'Network' object https://review.opendev.org/697154
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:22 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove now unnecessary nova-net workaround https://review.opendev.org/702440
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:23 openstackgerrit Stephen Finucane proposed openstack/nova master: WIP: nova-net: Remove unused nova-network objects https://review.opendev.org/697156
10:58:23 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:08:33 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'nova.image.api' module https://review.opendev.org/702451
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/

Earlier   Later