| *** markvoelker has joined #openstack-qa | 00:20 | |
| *** markvoelker has quit IRC | 00:25 | |
| *** ildikov has joined #openstack-qa | 02:30 | |
| *** yamamoto has joined #openstack-qa | 03:08 | |
| *** yamamoto has quit IRC | 03:20 | |
| *** yamamoto has joined #openstack-qa | 03:20 | |
| *** markvoelker has joined #openstack-qa | 03:22 | |
| *** yamamoto_ has joined #openstack-qa | 03:24 | |
| *** yamamoto has quit IRC | 03:25 | |
| *** markvoelker has quit IRC | 03:26 | |
| *** yamamoto_ has quit IRC | 03:30 | |
| *** pleia2 has quit IRC | 03:45 | |
| *** pleia2 has joined #openstack-qa | 03:47 | |
| *** jdwidari has quit IRC | 04:10 | |
| *** markvoelker has joined #openstack-qa | 05:11 | |
| *** markvoelker has quit IRC | 05:15 | |
| *** markvoelker has joined #openstack-qa | 06:11 | |
| *** markvoelker has quit IRC | 06:15 | |
| *** igordc has quit IRC | 06:50 | |
| *** markvoelker has joined #openstack-qa | 09:00 | |
| *** markvoelker has quit IRC | 09:05 | |
| *** markvoelker has joined #openstack-qa | 09:31 | |
| *** markvoelker has quit IRC | 09:35 | |
| *** jdwidari has joined #openstack-qa | 10:13 | |
| *** jdwidari has quit IRC | 10:18 | |
| *** tosky has joined #openstack-qa | 10:41 | |
| *** tosky_ has joined #openstack-qa | 12:16 | |
| *** tosky_ has quit IRC | 12:16 | |
| *** tosky_ has joined #openstack-qa | 12:16 | |
| *** tosky has quit IRC | 12:19 | |
| *** yamamoto has joined #openstack-qa | 12:25 | |
| *** markvoelker has joined #openstack-qa | 12:30 | |
| *** markvoelker has quit IRC | 12:35 | |
| *** yamamoto has quit IRC | 12:40 | |
| *** yamamoto has joined #openstack-qa | 12:48 | |
| *** yamamoto has quit IRC | 12:48 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest master: DNM:testing test timeout https://review.opendev.org/679539 | 13:01 |
|---|---|---|
| gmann | dtantsur|afk: setting OS_TEST_TIMEOUT with default value will affect existing default behavior as default OS_TEST_TIMEOUT for tempest run is not set. it is only set for 'all' and 'all-plugins' tox env. | 13:04 |
| gmann | i am trying to debug it via https://review.opendev.org/679539 to check all env var for tox env | 13:04 |
| *** tosky_ has quit IRC | 13:21 | |
| clarkb | gmann: it is set for other envs too | 13:23 |
| gmann | frickler: with the current list, i am ok to add opendev.org/performa also if base job is used there. | 13:25 |
| gmann | clarkb: you mean for unit tests env or tempest run env ? all other tempest run env use [tempestenv]setenv for example- https://opendev.org/openstack/tempest/src/commit/9e63a0323e48543c47d733ce45cafde45fc2ddcb/tox.ini#L114 | 13:26 |
| gmann | and they do not set the default OS_TEST_TIMEOUT | 13:27 |
| clarkb | the default testenv sets it to 160 for exanple | 13:27 |
| gmann | but setenv is overridden by them with {[tempestenv]setenv} which does not set the OS_TEST_TIMEOUT default | 13:28 |
| clarkb | and all is literally all tests right? whoch is super set of eg full | 13:28 |
| clarkb | whoch should mean tmeout for all works for full | 13:28 |
| gmann | clarkb: 'all' env is not super set actually. that is independent env. | 13:29 |
| clarkb | if not then the timwout set by all is wrong (which may be what dtantsur|afk is hitting via plugins?) | 13:29 |
| clarkb | oh why us it called all? | 13:29 |
| gmann | clarkb: that is to run the *ALL* tempest tests (including slow, scenario etc). | 13:30 |
| gmann | dtantsur|afk patch is hitting timeout with 1200 also as ironic job use 'all' tox env at the end | 13:31 |
| gmann | 'all' and 'full' is something confusing name. | 13:31 |
| clarkb | right full is all without slow I thought | 13:32 |
| gmann | yeah and running scenario is serial not parallel | 13:32 |
| clarkb | right so a timwout for all should work for full toi | 13:32 |
| clarkb | since all is a super set of tests? | 13:32 |
| clarkb | meaning if you run all that timeout applies to the same tests full will run | 13:33 |
| gmann | you mean to set that default timeout (what we have for 'all' env) for 'full' and all other tox env also ? | 13:34 |
| clarkb | yes as it makes fixing ironics problem simpler | 13:34 |
| clarkb | and ensures we test the tineouts | 13:34 |
| gmann | but that will not solve ironic issue as they are using 'all' tox env which default value 1200 is less for them | 13:35 |
| clarkb | I think the underlying issue here is we run all-plugins with a filter to run tempest plugins and the timeout applies there | 13:35 |
| clarkb | which means we never test the timeout when tempest jobs run unless testing plugins | 13:35 |
| clarkb | we may gind that timeout is too low for full too | 13:36 |
| gmann | ironic need to set higher value via job and which should work fine as it wa tried originally just by setting the env var via devstack-localrc | 13:36 |
| clarkb | but at least it will be configurable per job in cases that need it | 13:36 |
| gmann | we test that via tempest-all job but yes we can set the default values for all other tox env also which makes sense | 13:37 |
| clarkb | I dont think we run tempest with devstack lovalrc loaded | 13:37 |
| clarkb | its a separate ansible task so should have anew clean shell env | 13:37 |
| gmann | i see. | 13:38 |
| gmann | let's go with default(omit) option in https://review.opendev.org/#/c/679242/2/roles/run-tempest/tasks/main.yaml | 13:38 |
| gmann | and later with separate patch, i can set same default timeout for other tox env also so that we consistently test the timeout. | 13:39 |
| clarkb | sounds good | 13:41 |
| gmann | frickler: regarding this: https://review.opendev.org/#/c/580356/3 | 13:43 |
| gmann | frickler: running force-tenant-isolation test with predefined creds will not be good idea so I am going with option to skip the force-tenant-isolation tests for pre-provisioned cred. and then merge your patch | 13:44 |
| *** tosky_ has joined #openstack-qa | 14:16 | |
| *** e0ne has joined #openstack-qa | 14:24 | |
| *** artom has joined #openstack-qa | 14:35 | |
| *** artom has quit IRC | 14:49 | |
| *** rfolco has quit IRC | 14:55 | |
| *** tosky_ has quit IRC | 14:59 | |
| openstackgerrit | Merged openstack/hacking master: Add Python 3 Train unit tests https://review.opendev.org/670632 | 15:06 |
| *** markvoelker has joined #openstack-qa | 15:13 | |
| *** markvoelker has quit IRC | 15:17 | |
| *** e0ne has quit IRC | 15:33 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest master: DNM:testing test timeout https://review.opendev.org/679539 | 15:38 |
| *** e0ne has joined #openstack-qa | 15:44 | |
| *** e0ne has quit IRC | 16:29 | |
| *** e0ne has joined #openstack-qa | 17:01 | |
| *** e0ne has quit IRC | 17:05 | |
| *** e0ne has joined #openstack-qa | 17:28 | |
| *** markvoelker has joined #openstack-qa | 17:29 | |
| *** markvoelker has quit IRC | 17:38 | |
| *** e0ne has quit IRC | 17:44 | |
| *** markvoelker has joined #openstack-qa | 18:54 | |
| *** markvoelker has quit IRC | 19:07 | |
| *** slaweq has joined #openstack-qa | 19:33 | |
| *** e0ne has joined #openstack-qa | 19:36 | |
| *** e0ne has quit IRC | 19:45 | |
| *** e0ne has joined #openstack-qa | 19:56 | |
| *** slaweq has quit IRC | 20:05 | |
| *** markvoelker has joined #openstack-qa | 20:14 | |
| *** slaweq has joined #openstack-qa | 20:22 | |
| *** markvoelker has quit IRC | 20:24 | |
| *** slaweq has quit IRC | 20:27 | |
| *** kopecmartin|off has quit IRC | 20:36 | |
| openstackgerrit | Doug Schveninger proposed openstack/tempest master: Correcting concurrency parameter for negative value https://review.opendev.org/630128 | 21:18 |
| *** markvoelker has joined #openstack-qa | 22:56 | |
| *** markvoelker has quit IRC | 23:02 | |
| *** e0ne has quit IRC | 23:13 | |
| *** lbragstad has quit IRC | 23:29 | |
| *** lbragstad has joined #openstack-qa | 23:31 | |
| openstackgerrit | Phil Sphicas proposed openstack/tempest master: Update test_port_list_filter_by_ip pass conditions https://review.opendev.org/676852 | 23:48 |
| *** markvoelker has joined #openstack-qa | 23:48 | |
| *** markvoelker has quit IRC | 23:58 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!