Posted	Nick	Remark
#openstack-nova - 2020-01-14
16:15:42	efried	stephenfin: +A on the precommit gizmo, sorry for the delay. And thanks for the vTPM spec approval \o/
16:30:27	sean-k-mooney	efried: is the blueprint/spec approval deadline m2?
16:32:05	sean-k-mooney	im just looking at my downstream feature backlog and i might file one more blueprint for this cycle to add viommu support to th elibvirt driver. it will just be a small change like the vpmu change but just tring to figure out when i need to decided if im going to file it
16:34:02	sean-k-mooney	i want to finish the image metadata prefiltering work first but the viommu feature is tecnically more useful to end customers
16:49:27	efried	sean-k-mooney: spec freeze 2/13 https://wiki.openstack.org/wiki/Nova/Ussuri_Release_Schedule yes. At that point I'll probably ask us to do a scrub to prioritize whatever's been Definition:Approved and drop some things off the bottom.
16:50:00	sean-k-mooney	ya i was execpting there to be a scrub
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	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: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	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	new behavior: since the no-op entry gets removed during validation, the $COMPUTE_NODE value gets used when merging
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: Function to further validate and retrieve configs https://review.opendev.org/676029
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: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	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

Earlier   Later