| songwenping | sean-k-mooney: hi, is there concurrency case in tempest? | 09:10 |
|---|---|---|
| sean-k-mooney | songwenping: can you be more specific? tempest can run with multiple test in parallel | 10:20 |
| 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:22 |
| sean-k-mooney | anyway to fid it you can use https://docs.openstack.org/api-ref/compute/#list-hypervisors | 10:23 |
| sean-k-mooney | so /os-hypervisors?hypervisor_hostname_pattern=<host value goes here> | 10:24 |
| sean-k-mooney | use microveriosn 2.53 or higher to get teh id filed as a uuid | 10:24 |
| sean-k-mooney | <2.53 the id filed is a cell speific int so you should avoid that | 10:25 |
| auniyal | sean-k-mooney, how to set min_microversion in temepst, this https://paste.opendev.org/show/bJLSDBLxBgj2hzBcJEiq/ in tempest.conf | 10:27 |
| 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 |
| auniyal | I was refering this - https://docs.openstack.org/tempest/latest/sampleconf.html | 10:28 |
| sean-k-mooney | auniyal: you set it in teh test class | 10:49 |
| 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 |
| auniyal | yeah, in test class min_microversion = '2.11' | 10:49 |
| sean-k-mooney | you shoudl not need to chnage the tempest.conf | 10:50 |
| auniyal | I am getting SKIPPED: The microversion range[2.11 - latest] of this test is out of the configuration range[None - None] | 10:51 |
| sean-k-mooney | sure but devstack will produce a valid tempst conf for you | 10:51 |
| *** srelf_ is now known as ContinuitySR | 10:51 | |
| sean-k-mooney | and you should never realy need to change the values for the micorverion range in general | 10:51 |
| auniyal | ack, so change rearding microverion in tempest conf | 10:52 |
| auniyal | *no change | 10:52 |
| sean-k-mooney | what you have in the pastbin shoudl work | 10:52 |
| auniyal | sean-k-mooney, how can I fix skipping test then | 10:52 |
| sean-k-mooney | but my guess is that is nto the actul issue | 10:52 |
| sean-k-mooney | can you should be the test | 10:53 |
| auniyal | https://paste.opendev.org/show/bZgalcGsTREjXm08RT34/ | 10:53 |
| 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:54 |
| 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:55 |
| 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 |
| sean-k-mooney | so we need to check BaseV2ComputeAdminTest | 10:56 |
| sean-k-mooney | ok it should get it form here https://github.com/openstack/tempest/blob/1c3fc43c9dae274a19683b003f78b5f50edbc062/tempest/api/compute/base.py#L123 | 10:57 |
| auniyal | CONF.compute.min_microversion - so None | 10:58 |
| sean-k-mooney | from what i can see you have done it correctly | 10:58 |
| sean-k-mooney | auniyal: its proably best to ask about this isn the QA channel by the way | 10:59 |
| auniyal | sean-k-mooney, ack | 10:59 |
| sean-k-mooney | that is where we gernerally discuss tempest/whitebox thigns | 10:59 |
| auniyal | thanks for looking | 10:59 |
| sean-k-mooney | there are more people that understand it better there | 10:59 |
| 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:22 |
| 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:35 |
| sean-k-mooney | so that shoudl either never happen or test that share names need to have the @serial decorator applied | 11:36 |
| songwenping_ | thanks, got it. | 11:37 |
| 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:11 |
| 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 | 12:39 |
| 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`? | 22:14 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!