Posted	Nick	Remark
#openstack-nova - 2020-11-23
16:52:56	kashyap	In any case, I'm copying my comment into this one, too: https://bugs.launchpad.net/nova/+bug/1761798
18:14:26	openstackgerrit	Merged openstack/nova master: virt: Remove 'change_instance_metadata' API https://review.opendev.org/c/openstack/nova/+/749316
18:42:34	openstackgerrit	Stephen Finucane proposed openstack/nova master: virt: Remove 'get_all_bw_counters' API https://review.opendev.org/c/openstack/nova/+/749312
18:42:34	openstackgerrit	Stephen Finucane proposed openstack/nova master: virt: Remove 'reset_network' API https://review.opendev.org/c/openstack/nova/+/749315
18:42:35	openstackgerrit	Stephen Finucane proposed openstack/nova master: db: Drop models for removed services, features https://review.opendev.org/c/openstack/nova/+/698706
18:42:35	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Remove 'BandwidthUsage', 'BandwidthUsageList' https://review.opendev.org/c/openstack/nova/+/759114
19:12:20	sean-k-mooney	stephenfin: ah those are related to xenapi removal
19:16:50	sean-k-mooney	i tought it was related to the compute_monitors config intially
19:16:57	sean-k-mooney	https://github.com/openstack/nova/blob/120d866df7ab81b666c0e43b97705a90483533bc/nova/conf/compute.py#L107-L126
19:24:02	sean-k-mooney	stephenfin: by the way we talked about the metrics weigher and weighing on things that were not compute weights in the past
19:24:06	sean-k-mooney	https://github.com/openstack/nova/blob/master/nova/compute/monitors/__init__.py#L40-L48
19:24:17	sean-k-mooney	the monitors are a stevaldor entry point
19:24:30	sean-k-mooney	and the numa memory bandwtih ones were actully plugins not in tree
19:24:57	sean-k-mooney	https://github.com/openstack/nova/blob/c250aca00ba89b6fad86bfaebb3083e9f5c55f3b/nova/compute/monitors/__init__.py#L44 the doc used to reference them years ago
19:28:43	sean-k-mooney	well it was propsoed upstream https://review.opendev.org/c/openstack/nova/+/270344
19:28:53	sean-k-mooney	but it was not merged
19:31:20	sean-k-mooney	https://review.opendev.org/c/openstack/nova-specs/+/180983 was the spec
21:21:27	sean-k-mooney	gibi: stephenfin i just realised that https://review.opendev.org/c/openstack/nova/+/653506 is still not merged but its been so long that i have lost most of the context. its still proably should be fixed but im not sure when ill get to it. hopefully soon but i tought this was merged years ago...
#openstack-nova - 2020-11-24
00:10:54	gmann	gibi: stephenfin: can you check this sample config fix- https://review.opendev.org/c/openstack/nova/+/762721
00:49:36	brinzhang	gibi, dansmith: do you have time to review the cyborg shelve/unshelve support patch, it has addressed all your patch https://review.opendev.org/c/openstack/nova/+/729563
01:38:09	openstackgerrit	DatLQ proposed openstack/nova stable/rocky: Fix unplugging VIF when migrate/resize VM https://review.opendev.org/c/openstack/nova/+/763898
01:58:13	openstackgerrit	DatLQ proposed openstack/nova stable/rocky: Fix unplugging VIF when migrate/resize VM https://review.opendev.org/c/openstack/nova/+/763898
05:51:22	openstackgerrit	Kevin Zhao proposed openstack/nova master: Support Cpu Compararion on Aarch64 Platform https://review.opendev.org/c/openstack/nova/+/763928
07:20:24	openstackgerrit	Jorhson Deng proposed openstack/nova-specs master: Add instance_state to ensure volume attachment successful https://review.opendev.org/c/openstack/nova-specs/+/758920
08:05:31	openstackgerrit	MIKI Nobuhiro proposed openstack/nova-specs master: Add IP address to libvirt guest metadata https://review.opendev.org/c/openstack/nova-specs/+/760750
08:26:25	openstackgerrit	Nobuhiro MIKI proposed openstack/nova-specs master: Add IP address to libvirt guest metadata https://review.opendev.org/c/openstack/nova-specs/+/760750
08:35:59	openstackgerrit	Jorhson Deng proposed openstack/nova-specs master: Add instance_state to ensure volume attachment successful https://review.opendev.org/c/openstack/nova-specs/+/758920
10:46:29	openstackgerrit	Xinran WANG proposed openstack/nova-specs master: SRIOV Enabled Nic Support Specification https://review.opendev.org/c/openstack/nova-specs/+/742785
11:09:34	lyarwood	gibi: morning, could I pester you for a review of https://review.opendev.org/c/openstack/nova/+/756434 - RH owner and stephenfin has already ACK'd so I didn't want to +W it without further review.
11:09:55	gibi	lyarwood: ack, looking
11:10:06	lyarwood	gibi: thanks
11:11:41	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Remove 'BandwidthUsage', 'BandwidthUsageList' https://review.opendev.org/c/openstack/nova/+/759114
11:11:42	openstackgerrit	Stephen Finucane proposed openstack/nova master: db: Drop models for removed services, features https://review.opendev.org/c/openstack/nova/+/698706
11:14:21	gibi	lyarwood: +a
11:16:11	lyarwood	gibi: cheers
11:54:32	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Make PCI claim NUMA aware during live migration https://review.opendev.org/c/openstack/nova/+/748453
12:22:21	openstackgerrit	Balazs Gibizer proposed openstack/nova stable/victoria: Warn when starting services with older than N-1 computes https://review.opendev.org/c/openstack/nova/+/761923
12:22:26	openstackgerrit	Balazs Gibizer proposed openstack/nova stable/victoria: Add upgrade check about old computes https://review.opendev.org/c/openstack/nova/+/761924
12:28:45	gibi	elod: ^^ did the squash
12:43:33	sean-k-mooney	gibi: stephenfin since the autor of https://review.opendev.org/c/openstack/nova-specs/+/760750 ignored my attempt to colabarte and have just one spec i want to get your input on "do i really need a spec"
12:44:11	sean-k-mooney	basically i was plannign to update the libvirt metadata with teh flavor extra specs and image proeprties via a specless blueprint
12:44:44	sean-k-mooney	this is an internal api used only for debugging so i kind of would prefer to just propose a patch and debate the implemeantion there
12:45:19	sean-k-mooney	i can write a spec but its not an api change so it feels kind fo heavy weight to me
12:46:42	stephenfin	I think gibi's the only person that can really answer that
12:47:05	gibi	do you remember why we asked for a spec for the first place? Now it would be strange to treate the two features differently
12:47:33	stephenfin	I think because there were a few design questions and implications for things like attach/detach
12:48:26	sean-k-mooney	i can write a spec but the only time the flavor and image will change are resize and rebuild where im regenreating the xml anyway
12:48:44	gibi	and resize while changes the flavor it does regen... ^^
12:49:09	gibi	then I'm OK with a specless bp
12:49:24	gibi	this makes the flavor part easier
12:50:03	sean-k-mooney	yep basically im sick of having to figure out what the flavor and image propertise are form customer logs
12:50:25	sean-k-mooney	haveing them in the xml output would mean i dont have to ask them
12:50:25	stephenfin	you'd be using the embedded flavor, presumably?
12:50:28	sean-k-mooney	stephenfin: yep
12:51:20	sean-k-mooney	which is also important as i can say well this vm was booted before you updated the flavor as we can see form the xml
12:53:25	sean-k-mooney	gibi: ill add it to the adgenda for the team meeting and if other want a spec ill add a small one.
12:53:54	gibi	sean-k-mooney: cool
13:37:35	Qianbiao	Hello, Nova.
13:37:56	Qianbiao	Hello, i build a customer vendor_data metadata service.
13:38:18	Qianbiao	And i can get vendor through "curl http://169.254.169.254/openstack/2018-08-27/vendor_data2.json"
13:38:43	Qianbiao	But cloud-init does not try to load this file, it seems it only loads http://169.254.169.254/openstack/2018-08-27/vendor_data.json
13:38:48	Qianbiao	what config should i update?
13:59:22	Qianbiao	and i do not see how it works, from the source code: https://github.com/canonical/cloud-init/blob/master/cloudinit/sources/helpers/openstack.py#L230-L255
13:59:35	Qianbiao	it seems vendor_data2.json is never loaded.
14:40:11	bauzas	dansmith: do you have time for a quick question about the RPC API bump ?
14:41:13	bauzas	dansmith: could I just drop https://github.com/openstack/nova/blob/master/nova/compute/rpcapi.py#L786 now (and all the nova-network methods) or should I wait until X ?
14:42:17	bauzas	dansmith: afaicu, if the operator would like to pin the RPC API to 5.x, then we would still need to support the method in the manager, but not in the rpcapi since nobody would use this method, right ?
14:42:47	bauzas	as only computes would be in Victoria but none of the conductors
15:10:50	openstackgerrit	Sylvain Bauza proposed openstack/nova master: WIP: Bump the Compute RPC API to version 6.0 https://review.opendev.org/c/openstack/nova/+/761452
15:22:36	openstackgerrit	Stephen Finucane proposed openstack/nova master: db: Drop models for removed services, features https://review.opendev.org/c/openstack/nova/+/698706
15:31:24	openstackgerrit	Stephen Finucane proposed openstack/nova master: virt: Remove 'get_all_bw_counters' API https://review.opendev.org/c/openstack/nova/+/749312
15:31:25	openstackgerrit	Stephen Finucane proposed openstack/nova master: db: Drop models for removed services, features https://review.opendev.org/c/openstack/nova/+/698706
15:31:25	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Remove 'BandwidthUsage', 'BandwidthUsageList' https://review.opendev.org/c/openstack/nova/+/759114
15:31:26	openstackgerrit	Stephen Finucane proposed openstack/nova master: db: Remove KeyPair model from main DB https://review.opendev.org/c/openstack/nova/+/763989
15:31:49	stephenfin	gibi: Resolved that pep8 issue, if you've a chance to revisit https://review.opendev.org/c/openstack/nova/+/749312
15:51:56	gibi	stephenfin: done
15:52:02	stephenfin	ty
15:52:51	gibi	I will get back to the db model drop patches later
15:53:28	stephenfin	yup, no panic on those :)
15:56:10	gibi	I'm far from panic :D
16:39:28	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Add support for virtio-based input devices https://review.opendev.org/c/openstack/nova/+/756552
17:24:33	openstackgerrit	Stephen Finucane proposed openstack/nova master: api: Drop statistics-style fields from os-hypervisors https://review.opendev.org/c/openstack/nova/+/764040
17:24:33	openstackgerrit	Stephen Finucane proposed openstack/nova master: tests: Merge 'test_hypervisor_status' into 'test_hypervisors' https://review.opendev.org/c/openstack/nova/+/764039
17:33:32	openstackgerrit	Stephen Finucane proposed openstack/nova-specs master: Update modernize-os-hypervisors-api spec https://review.opendev.org/c/openstack/nova-specs/+/763043
17:33:33	openstackgerrit	Stephen Finucane proposed openstack/nova-specs master: Add 'vcpus', 'vcpus_used' to modernize-os-hypervisors-api spec https://review.opendev.org/c/openstack/nova-specs/+/764041
18:30:59	openstackgerrit	sean mooney proposed openstack/nova master: libvirt: update live migration defaults https://review.opendev.org/c/openstack/nova/+/760050
18:32:58	sean-k-mooney	oh i need to update nova.tests.unit.virt.libvirt.test_driver.LibvirtConnTestCase.test_live_migration_monitor_completion
18:43:02	openstackgerrit	sean mooney proposed openstack/nova master: libvirt: update live migration defaults https://review.opendev.org/c/openstack/nova/+/760050
20:38:39	openstackgerrit	Daniel de Oliveira Pereira proposed openstack/nova master: Avoid changing transport_url when updating Cell0 https://review.opendev.org/c/openstack/nova/+/762433
20:42:52	artom	Is `sync-devstack-data : Push the CA certificate` failing a new thing?
20:43:17	artom	I'm hitting it on a whitebox patch, but can't for the life of me figure out how my change could possibly be causing it
20:43:25	artom	So I'm wondering if it's maybe unrelated and a known issue?
20:43:56	sean-k-mooney	im not aware of it
20:44:03	sean-k-mooney	did it fail on my job too?
20:44:24	sean-k-mooney	also #openstack-qa might know
20:46:55	artom	sean-k-mooney, https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/764042
20:47:09	artom	The controller times out waiting for the compute n-cpu to come up
20:47:24	artom	And no n-* logs on the compute, just that devstack CA error that I can see
20:48:05	gmann	artom: at least not happening on other patches, at least it running fine on some latest patches run
20:48:36	sean-k-mooney	nternal error: Unknown CPU feature sse3

Earlier   Later