| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-04-23 | |||
| 16:36:07 | fwiesel | Depends on the context... c-extensions may release the lock and run really in parallel | |
| 16:36:15 | dansmith | right | |
| 16:36:33 | bauzas | anyway, let me reexplain it | |
| 16:36:34 | dansmith | they're parallel but with a ton of shared/locked data structures :) | |
| 16:36:35 | sean-k-mooney | ya and there are other case wehre the gil is droped too | |
| 16:36:36 | fwiesel | Would help with the parsing of the huge xml documents we sometimes get in the vmwareapi | |
| 16:36:47 | dansmith | let's just move on, I think we know what we agreed | |
| 16:36:54 | sean-k-mooney | sure | |
| 16:37:09 | bauzas | we will use 'python threads" instead of "other concurrent/green threading way" | |
| 16:37:37 | sean-k-mooney | yes throught the api of a futureist executor in most cases | |
| 16:37:41 | bauzas | so, anything else to say? | |
| 16:37:53 | sean-k-mooney | nope | |
| 16:37:56 | bauzas | cool | |
| 16:37:59 | bauzas | thanks all | |
| 16:38:16 | bauzas | #endmeeting | |
| 16:38:16 | opendevmeet | Meeting ended Tue Apr 23 16:38:16 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:38:16 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2024/nova.2024-04-23-16.00.html | |
| 16:38:16 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2024/nova.2024-04-23-16.00.txt | |
| 16:38:16 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2024/nova.2024-04-23-16.00.log.html | |
| 16:38:27 | fwiesel | Thanks and bye | |
| 16:38:42 | elodilles | thanks o/ | |
| 16:38:42 | gibi | 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: Fix: migration configuration with cpu_shared_set (libvirt part) https://review.opendev.org/c/openstack/nova/+/915139 | |
| 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 | |
| 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 | |