| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-04-23 | |||
| 16:38:42 | elodilles | thanks o/ | |
| 16:39:13 | elodilles | bauzas: the zed release patch: https://review.opendev.org/c/openstack/releases/+/916813 | |
| 16:41:27 | elodilles | btw, it's probably useful to release now then form newer stable branches as well. i'll create stable release patches for those | |
| 16:59:40 | elodilles | here they are: 2023.1 - https://review.opendev.org/916815 ; 2023.2 - https://review.opendev.org/916816 | |
| 17:00:10 | elodilles | i think we don't need a stable/2024.1 release yet | |
| 17:01:27 | elodilles | (this would be the content on caracal in the moment: https://paste.opendev.org/show/bq3vt56UP5WVQ9baFLUZ/ ) | |
| #openstack-nova - 2024-04-24 | |||
| 14:17:47 | artom | dansmith, sean-k-mooney, I think https://review.opendev.org/c/openstack/nova/+/915139/3 (the cpu_shared_set live migration fix) is at a point where it should be an easy-ish +2/+A - can we get that going? I've got a couple of whitebox changed that need it, which will allow us to cross a few tests off our list | |
| 14:18:21 | sean-k-mooney | ah ya i didnt get to reveiw that last week | |
| 14:18:40 | sean-k-mooney | i have 12 mins before my next meeting so ill take a look | |
| 14:21:37 | artom | Cheers :) | |
| 14:36:48 | simondodsley | is there a known prgramatic method to get the unique ID of a hypervisor? I know it is in the Nova DB but not sure which construct I can use to get that out through a python call | |
| 14:41:26 | sean-k-mooney | do you mean the compute node UUID | |
| 14:41:40 | sean-k-mooney | that is aviable to admins via the hypervior api | |
| 14:46:43 | dansmith | Uggla_: mind if I just quickly fix the truthiness nits in https://review.opendev.org/c/openstack/nova/+/915139/3/nova/virt/libvirt/migration.py and then I'll +2? | |
| 14:51:59 | dansmith | and also the commit message nit | |
| 14:52:07 | dansmith | (which is not really a nit, IMHO) | |
| 14:53:06 | artom | dansmith, mind if I proxy for him and just tell you to do it? ;) | |
| 14:53:14 | artom | (With appreciation for your review time) | |
| 14:56:44 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: scheduler: AggregateMultitenancyIsolation to support unlimited tenant https://review.opendev.org/c/openstack/nova/+/896512 | |
| 14:59:27 | opendevreview | Dan Smith proposed openstack/nova master: Fix: migration configuration with cpu_shared_set (object part) https://review.opendev.org/c/openstack/nova/+/903706 | |
| 14:59:28 | opendevreview | Dan Smith proposed openstack/nova master: Test live migration between hosts with differnet cpu_shared_sets https://review.opendev.org/c/openstack/nova/+/913744 | |
| 14:59:28 | opendevreview | Dan Smith proposed openstack/nova master: Fix: migration configuration with cpu_shared_set (libvirt part) https://review.opendev.org/c/openstack/nova/+/915139 | |
| 15:31:16 | dansmith | tkajinam: did you want to opine on this before I flesh it out? https://review.opendev.org/c/openstack/nova/+/915911/1 | |
| 15:32:10 | opendevreview | Max proposed openstack/nova master: feat: nova-manage db instace events cleanup https://review.opendev.org/c/openstack/nova/+/916927 | |
| 15:34:44 | tkajinam | dansmith, I'll check the series of these patches and will leave any comments (or +1 if no) | |
| 19:58:58 | opendevreview | Merged openstack/nova master: Fix: migration configuration with cpu_shared_set (object part) https://review.opendev.org/c/openstack/nova/+/903706 | |
| 20:37:39 | opendevreview | Merged openstack/nova master: Fix: migration configuration with cpu_shared_set (libvirt part) https://review.opendev.org/c/openstack/nova/+/915139 | |
| 21:07:42 | simondodsley | sean-k-mooney: sorry for the delay in responding. I need to get the compute UUID from within a cinder driver. Do you know where/how I could do that? | |
| #openstack-nova - 2024-04-25 | |||
| 09:10:45 | songwenping | sean-k-mooney: hi, is there concurrency case in tempest? | |
| 10:20:30 | sean-k-mooney | songwenping: can you be more specific? tempest can run with multiple test in parallel | |
| 10:22:31 | sean-k-mooney | simondodsley: im not entirly sure why the cinder driver woudl need the compute node uuid. we dont use that as a value we exchange between services. we use the CONF.host value which is typically the hostname but can be the fqdn | |
| 10:23:34 | sean-k-mooney | anyway to fid it you can use https://docs.openstack.org/api-ref/compute/#list-hypervisors | |
| 10:24:12 | sean-k-mooney | so /os-hypervisors?hypervisor_hostname_pattern=<host value goes here> | |
| 10:24:52 | sean-k-mooney | use microveriosn 2.53 or higher to get teh id filed as a uuid | |
| 10:25:21 | sean-k-mooney | <2.53 the id filed is a cell speific int so you should avoid that | |
| 10:27:37 | auniyal | sean-k-mooney, how to set min_microversion in temepst, this https://paste.opendev.org/show/bJLSDBLxBgj2hzBcJEiq/ in tempest.conf | |
| 10:28:01 | auniyal | but I am getting SKIPPED: The microversion range[2.11 - latest] of this test is out of the configuration range[None - None]. | |
| 10:28:29 | auniyal | I was refering this - https://docs.openstack.org/tempest/latest/sampleconf.html | |
| 10:49:30 | sean-k-mooney | auniyal: you set it in teh test class | |
| 10:49:52 | sean-k-mooney | you may also be able to do it seperately but most tests seam to do it at the class level | |
| 10:49:59 | auniyal | yeah, in test class min_microversion = '2.11' | |
| 10:50:40 | sean-k-mooney | you shoudl not need to chnage the tempest.conf | |
| 10:51:19 | auniyal | I am getting SKIPPED: The microversion range[2.11 - latest] of this test is out of the configuration range[None - None] | |
| 10:51:37 | sean-k-mooney | sure but devstack will produce a valid tempst conf for you | |
| 10:51:53 | sean-k-mooney | and you should never realy need to change the values for the micorverion range in general | |
| 10:52:10 | auniyal | ack, so change rearding microverion in tempest conf | |
| 10:52:14 | auniyal | *no change | |
| 10:52:48 | sean-k-mooney | what you have in the pastbin shoudl work | |
| 10:52:55 | auniyal | sean-k-mooney, how can I fix skipping test then | |
| 10:52:58 | sean-k-mooney | but my guess is that is nto the actul issue | |
| 10:53:01 | sean-k-mooney | can you should be the test | |
| 10:53:25 | auniyal | https://paste.opendev.org/show/bZgalcGsTREjXm08RT34/ | |
| 10:54:19 | sean-k-mooney | https://github.com/openstack/tempest/blob/1c3fc43c9dae274a19683b003f78b5f50edbc062/doc/source/microversion_testing.rst that is the docs on adding microversion test by the way | |
| 10:55:21 | sean-k-mooney | it looks like you might be missing the resocue_setup funciton un less you are gettign that form the ineritied test | |
| 10:56:46 | sean-k-mooney | its not provided directly by the parent class https://opendev.org/openstack/whitebox-tempest-plugin/src/branch/master/whitebox_tempest_plugin/api/compute/base.py#L38 | |
| 10:56:55 | sean-k-mooney | so we need to check BaseV2ComputeAdminTest | |
| 10:57:41 | sean-k-mooney | ok it should get it form here https://github.com/openstack/tempest/blob/1c3fc43c9dae274a19683b003f78b5f50edbc062/tempest/api/compute/base.py#L123 | |
| 10:58:28 | auniyal | CONF.compute.min_microversion - so None | |
| 10:58:29 | sean-k-mooney | from what i can see you have done it correctly | |
| 10:59:03 | sean-k-mooney | auniyal: its proably best to ask about this isn the QA channel by the way | |
| 10:59:15 | auniyal | sean-k-mooney, ack | |
| 10:59:21 | sean-k-mooney | that is where we gernerally discuss tempest/whitebox thigns | |
| 10:59:23 | auniyal | thanks for looking | |
| 10:59:30 | sean-k-mooney | there are more people that understand it better there | |
| 11:22:58 | songwenping_ | sean-k-mooney: use mutli threads to process the same resource. e.g. create a flavor use the same name with two thread, one thread should raise FlavorExists exception. | |
| 11:35:52 | sean-k-mooney | songwenping_: we generally dont do that in tempest, if we need to create a flavor we either generat a randome name or a uique oen for a given test | |
| 11:36:18 | sean-k-mooney | so that shoudl either never happen or test that share names need to have the @serial decorator applied | |
| 11:37:37 | songwenping_ | thanks, got it. | |
| 12:11:04 | sean-k-mooney | songwenping_: we do not split singlel testcase between workers by the way so each test runs in a singel threadn adn we distribute tests between threads | |
| 12:39:03 | opendevreview | Fabian Wiesel proposed openstack/nova master: Do not close returned image-chunk iterator & get_verifier early https://review.opendev.org/c/openstack/nova/+/909474 | |
| 22:14:02 | simondodsley | sean-k-mooney: the `CONF.host` doesn't gaurantee a unique name when taken in the context of a storage platform that can be used by systems other than OpenStack. It is possible that hosts in an organization coul dhave the same name, but with different storage prototocls, different IQNs, etc. This is why the compute node UUID would be ideal to use as this should be unique. Is there any chance this could be added to `CONF`? | |
| #openstack-nova - 2024-04-26 | |||
| 03:12:14 | opendevreview | Ilia Baikov proposed openstack/nova-specs master: Initial commit on blueprint https://review.opendev.org/c/openstack/nova-specs/+/917108 | |
| 03:19:18 | opendevreview | Ilia Baikov proposed openstack/nova-specs master: Initial spec on cloud-init for dynamic vendordata https://review.opendev.org/c/openstack/nova-specs/+/917109 | |
| 03:38:34 | opendevreview | Ilia Baikov proposed openstack/nova-specs master: Initial spec on cloud-init for dynamic vendordata https://review.opendev.org/c/openstack/nova-specs/+/917109 | |
| 03:46:13 | opendevreview | Ilia Baikov proposed openstack/nova-specs master: Initial spec on cloud-init for dynamic vendordata https://review.opendev.org/c/openstack/nova-specs/+/917109 | |
| 03:59:03 | opendevreview | Ilia Baikov proposed openstack/nova-specs master: Initial spec on cloud-init for dynamic vendordata https://review.opendev.org/c/openstack/nova-specs/+/917109 | |
| 04:07:50 | opendevreview | Ilia Baikov proposed openstack/nova-specs master: Initial spec on cloud-init for dynamic vendordata https://review.opendev.org/c/openstack/nova-specs/+/917109 | |
| 04:55:22 | opendevreview | Ilia Baikov proposed openstack/nova-specs master: Initial spec on cloud-init for dynamic vendordata https://review.opendev.org/c/openstack/nova-specs/+/917109 | |
| 07:07:15 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: scheduler: AggregateMultitenancyIsolation to support unlimited tenant https://review.opendev.org/c/openstack/nova/+/896512 | |
| 07:28:25 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix concurrent migration of vms with multiattach volume failure https://review.opendev.org/c/openstack/nova/+/872278 | |
| 07:58:58 | sahid | o/ bauzas sean-k-mooney dansmith, any chance to make progress on that one? https://review.opendev.org/c/openstack/nova/+/896512 | |
| 08:04:20 | bauzas | \o | |
| 08:04:56 | bauzas | sahid: I need to do a couple of reviews next week, tbh those two last weeks, I was off Gerrit :) | |
| 08:05:48 | bauzas | it's tracked in https://etherpad.opendev.org/p/nova-dalmatian-status, so don't worry, we'll look at it :) | |
| 08:07:45 | sahid | ah cool thanks | |
| 10:25:06 | elodilles | bauzas and release liaisons: when you have time please review these open release patches: https://review.opendev.org/q/project:openstack/releases+file:%5Edeliverables/.*/nova.yaml+is:open | |
| 10:25:16 | bauzas | I'll try | |
| 10:27:02 | elodilles | thanks in advance o/ | |
| 10:27:09 | elodilles | i know you are busy :S | |
| 11:48:37 | sean-k-mooney | simondodsley: no, and CONF.host is requried to be unique in the cloud or at least within a cell. if the there are other things usign the stroage backend that could have the same hostnames but different FQDNs then CONF.Host shoudl be an FQDN | |
| 11:49:13 | sean-k-mooney | simondodsley: today there is not api conrtract between nova and cinder that allows cidner to depend on the compute node uuid | |
| 11:49:25 | sean-k-mooney | at least not that im aware of | |
| 11:50:34 | sean-k-mooney | the attachment api is specified in teerms of the host field https://docs.openstack.org/api-ref/block-storage/v3/index.html?expanded=detach-volume-from-server-detail#create-attachment | |
| 11:51:05 | opendevreview | Konrad Gube proposed openstack/nova-specs master: Re-propose using extend volume completion action for 2024.2 https://review.opendev.org/c/openstack/nova-specs/+/917133 | |
| 11:51:32 | sean-k-mooney | we also pass the iqn in the connector | |
| 11:52:22 | sean-k-mooney | simondodsley: is there a particalr feature or bug you are tring to work on? | |
| 11:54:13 | sean-k-mooney | simondodsley: we recentlly added a stable compute uuid feature. the closet thing i think we might consider would be a config flag to use the compute uuid as the value fo host | |
| 13:18:21 | simondodsley | backends | |
| 13:18:21 | simondodsley | sean-k-mooney: it's actually a bug i'm trying to fix related to synchronously replicated cinder volumes. When configured so the volume can attach to the hypervisor from both backends at the same time (for redundancy) the host object on the array is allocated different names, so it can cause issues during cinder failover events. To get the UUID of the hypervisor will allow us to have consistent host object names across the | |
| 13:19:41 | sean-k-mooney | ok but that is not something cinder should be lookign up | |