Earlier  
Posted Nick Remark
#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 stephenfin you'd be using the embedded flavor, presumably?
12:50:25 sean-k-mooney haveing them in the xml output would mean i dont have to ask them
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: objects: Remove 'BandwidthUsage', 'BandwidthUsageList' https://review.opendev.org/c/openstack/nova/+/759114
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: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: tests: Merge 'test_hypervisor_status' into 'test_hypervisors' https://review.opendev.org/c/openstack/nova/+/764039
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: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
20:48:44 sean-k-mooney https://zuul.seanmooney.info/t/openstack/build/2ab2af1df163459e9b88aecce13a59d3/log/controller/logs/screen-n-cpu.txt#1961
20:48:53 gmann k moving to qa
20:49:27 sean-k-mooney same upstream
20:49:29 sean-k-mooney https://zuul.opendev.org/t/openstack/build/c6152ded026647dbb76c9d95721832f2/log/controller/logs/screen-n-cpu.txt#1946
20:49:37 sean-k-mooney so i guess it sse not sse3
20:49:55 sean-k-mooney oh
20:49:58 sean-k-mooney its ssse3
20:50:02 artom sean-k-mooney, huh, it doesn't directly explain the devstack CA thing
20:50:09 artom But definitely needs fixing
20:50:23 sean-k-mooney right but that is why n-cpu did not start
20:50:32 artom gmann, ^^ I'll push a new version, we'll see if the CA thing as well
20:50:37 artom It would indeed
20:50:45 sean-k-mooney ill swap to qa
#openstack-nova - 2020-11-25
00:57:11 openstackgerrit norman shen proposed openstack/nova-specs master: Adds spec for instance live resize https://review.opendev.org/c/openstack/nova-specs/+/141219
04:16:36 openstackgerrit Wenping Song proposed openstack/nova-specs master: Correct some spell errors https://review.opendev.org/c/openstack/nova-specs/+/764117
05:09:51 openstackgerrit Merged openstack/os-vif master: Fix hacking min version to 3.0.1 https://review.opendev.org/c/openstack/os-vif/+/755168
07:45:16 openstackgerrit wushiming proposed openstack/os-vif master: Remove the unused coding style modules https://review.opendev.org/c/openstack/os-vif/+/760491
10:18:38 openstackgerrit Merged openstack/nova-specs master: Correct some spell errors https://review.opendev.org/c/openstack/nova-specs/+/764117

Earlier   Later