clarkb | if this is a devstack job then yes | 00:04 |
---|---|---|
*** bigdogstl has joined #openstack-qa | 00:06 | |
bigdogstl | Review request https://review.opendev.org/#/c/630128/ | 00:08 |
*** bigdogstl has quit IRC | 00:14 | |
*** markvoelker has joined #openstack-qa | 00:15 | |
*** markvoelker has quit IRC | 00:20 | |
*** markvoelker has joined #openstack-qa | 00:22 | |
gmann | eandersson: try with list the tests using tempest run --list-tests | 00:24 |
gmann | with the new changes of skip exception workaround skipUnless at class level should work fine to list the tests in skip list | 00:26 |
gmann | but that is the only thing i doubt which might causing this issue if there is any | 00:28 |
*** mriedem_afk has quit IRC | 00:30 | |
gmann | eandersson: what version of Tempest and stestr you have locally where it work fine. work fine means skipping tests or run ? | 00:33 |
*** haixin has joined #openstack-qa | 00:49 | |
*** rfolco has joined #openstack-qa | 00:49 | |
*** rcernin has joined #openstack-qa | 00:51 | |
*** brinzhang has joined #openstack-qa | 00:53 | |
eandersson | gmann, yea they run locally | 01:01 |
eandersson | skipTest basically works, there just isn't an error message | 01:01 |
eandersson | or well sorry warning saying hey skipping these tests | 01:02 |
eandersson | and the skipUnless was skipping the tests because senlin isn't in the environment | 01:02 |
eandersson | We just assumed skipUnless would print a message or something when skipping | 01:02 |
*** brinzhang_ has joined #openstack-qa | 01:11 | |
*** haixin_inspur has joined #openstack-qa | 01:12 | |
*** brinzhang has quit IRC | 01:14 | |
*** haixin has quit IRC | 01:15 | |
gmann | eandersson: yeah it should add the test in skip tests list and print the reason passed in that decorator | 01:16 |
*** haixin_inspur has quit IRC | 01:20 | |
gmann | eandersson: it seems same behavior with old Tempest and stestr also - https://zuul.opendev.org/t/openstack/build/b3b3c22f62764c55a43644a72b9310ce/log/job-output.txt#52834 | 01:24 |
eandersson | I think what changed is that tempest is in a venv | 01:24 |
gmann | testtools is not adding the test in skip list when using skipUnless at class level | 01:24 |
eandersson | or at some point (however long ago) senlin was installed | 01:25 |
eandersson | in the same enviornment as the senlin plugin | 01:25 |
gmann | i think it was same always. | 01:25 |
eandersson | It's possible it was never working, or maybe something changed on the senlin tempest side | 01:26 |
gmann | yeah, same for py27 job also - https://019ab552bc17f89947ce-f1e24edd0ae51a8de312c1bf83189630.ssl.cf2.rackcdn.com/662677/4/check/senlin-dsvm-tempest-py27-integration/dc1e2c7/testr_results.html.gz | 01:26 |
*** anastzhyr has quit IRC | 01:26 | |
eandersson | dtruong, swears that it used to work thou :p | 01:26 |
gmann | eandersson: i see, this is the one would have stopped this - https://github.com/openstack/senlin-tempest-plugin/commit/c0d869a36d4dd9f39c55fde9df862ac617fb7e6d#diff-3ae915a8bf84c12e9cb441e5a195304b | 01:27 |
*** mtreinish has quit IRC | 01:27 | |
gmann | eandersson: in Tempest i think we never used the class level skip using decorator. I can debug the testtool decorator tomorrow and see if using decorator at class level is ok or not. | 01:29 |
gmann | otherwise senlin-tempest-plugin can use skip_checks() in that tests | 01:29 |
gmann | that is standard way in Tempest like tests | 01:29 |
gmann | example- https://github.com/openstack/tempest/blob/7fdd39c6dbde37bccd419c4037e1e352a5189c5a/tempest/api/compute/volumes/test_volumes_get.py#L35 | 01:30 |
eandersson | Thanks gmann | 01:32 |
gmann | eandersson: np! | 01:33 |
*** brinzhang has joined #openstack-qa | 01:34 | |
*** brinzhang_ has quit IRC | 01:36 | |
*** rfolco has quit IRC | 01:42 | |
eandersson | gmann, do we need that check? It was meant for backwards compatibility | 01:56 |
eandersson | does tempest plugins need to keep compatibility in master? | 01:56 |
*** rh-jelabarre has joined #openstack-qa | 01:57 | |
gmann | eandersson: yes, because master version should be used for stable branch testing. If that version is there in all supported stable branches (stable/queens onwards) then we can remove otherwise we should keep the check | 01:57 |
eandersson | I see - guess I need to figure out a better way of doing that check | 01:58 |
eandersson | Is installing senlin in the tempest venv an option? lol | 01:58 |
gmann | skip_check is best way to do that. | 01:58 |
eandersson | ah and set it it .zuul.yaml or something? | 01:59 |
eandersson | or the tempest config | 01:59 |
gmann | no, in tests only. remove the skipUnless and add the same condition in skip_checks class method | 01:59 |
gmann | tempest setUpClass will take care of everything else | 01:59 |
eandersson | I see | 02:00 |
gmann | same way you are doing in base class - https://github.com/openstack/tempest/blob/7fdd39c6dbde37bccd419c4037e1e352a5189c5a/tempest/api/compute/volumes/test_volumes_get.py#L35 | 02:00 |
*** brinzhang_ has joined #openstack-qa | 02:05 | |
*** brinzhang_ has quit IRC | 02:05 | |
*** brinzhang has quit IRC | 02:08 | |
eandersson | gmann, guessing something like this https://review.opendev.org/#/c/689054/ ? | 02:11 |
gmann | eandersson: yes. it should work fine now. | 02:13 |
*** dave-mccowan has quit IRC | 03:00 | |
*** pleia2 has quit IRC | 03:04 | |
*** xinranwang has joined #openstack-qa | 03:07 | |
*** rh-jelabarre has quit IRC | 03:11 | |
*** pleia2 has joined #openstack-qa | 03:13 | |
*** nweinber has joined #openstack-qa | 03:32 | |
*** ramishra has joined #openstack-qa | 03:44 | |
*** nweinber has quit IRC | 03:51 | |
*** larainema has joined #openstack-qa | 03:55 | |
*** ramishra has quit IRC | 03:56 | |
*** igordc has quit IRC | 04:15 | |
*** ramishra has joined #openstack-qa | 04:17 | |
*** pcaruana has joined #openstack-qa | 04:18 | |
*** soniya29 has joined #openstack-qa | 04:34 | |
*** markvoelker has quit IRC | 04:45 | |
*** gcheresh_ has joined #openstack-qa | 04:51 | |
*** lbragstad has quit IRC | 04:53 | |
*** lbragstad has joined #openstack-qa | 04:53 | |
*** dansmith has quit IRC | 04:54 | |
*** ianw has quit IRC | 04:54 | |
*** ianw_ has joined #openstack-qa | 04:55 | |
*** dansmith has joined #openstack-qa | 04:55 | |
*** ianw_ is now known as ianw | 04:56 | |
*** Luzi has joined #openstack-qa | 05:04 | |
*** surpatil has joined #openstack-qa | 05:22 | |
*** gcheresh_ has quit IRC | 05:25 | |
*** sridharg has joined #openstack-qa | 05:33 | |
*** brinzhang has joined #openstack-qa | 05:34 | |
*** gcheresh_ has joined #openstack-qa | 05:38 | |
*** brinzhang has quit IRC | 05:39 | |
*** raukadah is now known as chandankumar | 05:49 | |
*** gcheresh_ has quit IRC | 06:03 | |
*** rcernin has quit IRC | 06:18 | |
*** udesale has joined #openstack-qa | 06:22 | |
*** ccamposr__ has quit IRC | 06:33 | |
*** ccamposr has joined #openstack-qa | 06:33 | |
*** markvoelker has joined #openstack-qa | 06:46 | |
*** markvoelker has quit IRC | 06:51 | |
*** trident has quit IRC | 06:55 | |
*** slaweq has joined #openstack-qa | 06:55 | |
*** trident has joined #openstack-qa | 06:58 | |
*** tesseract has joined #openstack-qa | 07:09 | |
*** tosky has joined #openstack-qa | 07:16 | |
*** jpena|off is now known as jpena | 07:24 | |
*** rpittau|afk is now known as rpittau | 07:52 | |
*** xinranwang has quit IRC | 07:57 | |
*** dtantsur|afk is now known as dtantsur | 07:59 | |
*** e0ne has joined #openstack-qa | 08:01 | |
*** noama has joined #openstack-qa | 08:05 | |
openstackgerrit | Slawek Kaplonski proposed openstack/devstack master: Add possibility to configure manually MYSQL_SERVICE_NAME https://review.opendev.org/681201 | 08:08 |
openstackgerrit | Jens Harbott (frickler) proposed openstack/devstack master: Change USE_PYTHON3=True by default https://review.opendev.org/649097 | 08:14 |
*** lucasagomes has joined #openstack-qa | 08:15 | |
*** e0ne_ has joined #openstack-qa | 08:17 | |
*** e0ne has quit IRC | 08:17 | |
*** kopecmartin|off is now known as kopecmartin | 08:23 | |
openstackgerrit | Abhishek Kekane proposed openstack/devstack master: Glance: Add support to configure multiple file stores https://review.opendev.org/689104 | 09:03 |
openstackgerrit | Soniya Murlidhar Vyas proposed openstack/tempest master: Moved subunit_describe_calls from argparse to cliff https://review.opendev.org/682835 | 09:09 |
*** e0ne has joined #openstack-qa | 09:10 | |
*** e0ne_ has quit IRC | 09:11 | |
*** brinzhang has joined #openstack-qa | 09:57 | |
*** gfidente has joined #openstack-qa | 09:58 | |
*** Luzi has quit IRC | 10:00 | |
openstackgerrit | Jens Harbott (frickler) proposed openstack/devstack master: Change USE_PYTHON3=True by default https://review.opendev.org/649097 | 10:25 |
openstackgerrit | Jens Harbott (frickler) proposed openstack/devstack master: Handle uninstalled python3 https://review.opendev.org/688799 | 10:25 |
*** soniya29 has quit IRC | 10:26 | |
*** brinzhang_ has joined #openstack-qa | 10:34 | |
*** brinzhang has quit IRC | 10:36 | |
*** brinzhang has joined #openstack-qa | 10:37 | |
*** brinzhang_ has quit IRC | 10:38 | |
*** Luzi has joined #openstack-qa | 10:44 | |
*** dmellado has quit IRC | 10:55 | |
*** dmellado has joined #openstack-qa | 10:57 | |
*** brinzhang_ has joined #openstack-qa | 10:57 | |
*** udesale has quit IRC | 10:57 | |
*** brinzhang_ has quit IRC | 10:58 | |
*** brinzhang has quit IRC | 11:00 | |
*** jpena is now known as jpena|lunch | 11:09 | |
*** mtreinish has joined #openstack-qa | 11:12 | |
*** nweinber has joined #openstack-qa | 11:32 | |
openstackgerrit | Merged openstack/devstack master: Remove deprecated PostgreSQL database driver https://review.opendev.org/678496 | 11:39 |
*** dviroel has joined #openstack-qa | 11:45 | |
*** nweinber has quit IRC | 11:46 | |
*** Luzi has quit IRC | 11:48 | |
*** markvoelker has joined #openstack-qa | 11:48 | |
*** raildo has joined #openstack-qa | 11:56 | |
*** ccamposr has quit IRC | 12:00 | |
*** rh-jelabarre has joined #openstack-qa | 12:02 | |
*** Luzi has joined #openstack-qa | 12:04 | |
*** ccamposr has joined #openstack-qa | 12:06 | |
*** ccamposr has quit IRC | 12:10 | |
openstackgerrit | Jens Harbott (frickler) proposed openstack/devstack master: Change USE_PYTHON3=True by default https://review.opendev.org/649097 | 12:18 |
*** jpena|lunch is now known as jpena | 12:19 | |
openstackgerrit | Jens Harbott (frickler) proposed openstack/devstack master: Handle uninstalled python3 https://review.opendev.org/688799 | 12:22 |
*** rfolco has joined #openstack-qa | 12:23 | |
*** larainema has quit IRC | 12:26 | |
openstackgerrit | Jens Harbott (frickler) proposed openstack/devstack master: Handle uninstalled python3 https://review.opendev.org/688799 | 12:27 |
brtknr | how do i use python3 by default in devstack? | 12:33 |
brtknr | i looked in the docs and dont see anything obvious | 12:34 |
*** ccamposr has joined #openstack-qa | 12:34 | |
tosky | brtknr: see one of the proposed change above, like https://review.opendev.org/649097 - you just need to set USE_PYTHON3=True in the devstack configuration file | 12:35 |
brtknr | tosky: tyvm | 12:36 |
*** brinzhang has joined #openstack-qa | 12:37 | |
*** dtantsur is now known as dtantsur|brb | 12:38 | |
*** ccamposr has quit IRC | 12:39 | |
*** ccamposr has joined #openstack-qa | 12:42 | |
*** mriedem has joined #openstack-qa | 12:42 | |
*** dave-mccowan has joined #openstack-qa | 12:44 | |
*** ccamposr has quit IRC | 12:46 | |
*** Luzi has quit IRC | 12:48 | |
*** Luzi has joined #openstack-qa | 12:49 | |
*** ramishra has quit IRC | 12:49 | |
*** ccamposr has joined #openstack-qa | 12:52 | |
*** brinzhang_ has joined #openstack-qa | 12:54 | |
*** brinzhang has quit IRC | 12:56 | |
*** rfolco is now known as rfolco|ruck | 13:03 | |
*** nweinber has joined #openstack-qa | 13:04 | |
*** iurygregory has joined #openstack-qa | 13:12 | |
*** gcheresh_ has joined #openstack-qa | 13:14 | |
*** udesale has joined #openstack-qa | 13:16 | |
*** gcheresh_ has quit IRC | 13:22 | |
*** lbragsta_ has joined #openstack-qa | 13:42 | |
*** brinzhang has joined #openstack-qa | 13:47 | |
openstackgerrit | Matt Riedemann proposed opendev/elastic-recheck master: Add query for networkx 2.4 bug 1848499 https://review.opendev.org/689160 | 13:49 |
openstack | bug 1848499 in OpenStack Compute (nova) "tests fail with networkx 2.4: "AttributeError: 'DiGraph' object has no attribute 'node'"" [Critical,In progress] https://launchpad.net/bugs/1848499 - Assigned to Balazs Gibizer (balazs-gibizer) | 13:49 |
*** brinzhang_ has quit IRC | 13:50 | |
*** Luzi has quit IRC | 13:57 | |
*** ramishra has joined #openstack-qa | 14:03 | |
*** brinzhang_ has joined #openstack-qa | 14:08 | |
*** brinzhang has quit IRC | 14:12 | |
openstackgerrit | Merged opendev/elastic-recheck master: Add query for networkx 2.4 bug 1848499 https://review.opendev.org/689160 | 14:12 |
openstack | bug 1848499 in OpenStack Compute (nova) "tests fail with networkx 2.4: "AttributeError: 'DiGraph' object has no attribute 'node'"" [Critical,In progress] https://launchpad.net/bugs/1848499 - Assigned to Balazs Gibizer (balazs-gibizer) | 14:12 |
*** dtantsur|brb is now known as dtantsur | 14:14 | |
*** jdwidari has joined #openstack-qa | 14:15 | |
*** surpatil has quit IRC | 14:16 | |
*** noama has quit IRC | 14:38 | |
*** sridharg has quit IRC | 14:43 | |
*** jpena is now known as jpena|off | 14:47 | |
*** mattw4 has joined #openstack-qa | 15:18 | |
*** igordc has joined #openstack-qa | 15:24 | |
*** surpatil has joined #openstack-qa | 15:25 | |
*** surpatil is now known as surpatil|dinner | 15:26 | |
*** gcheresh_ has joined #openstack-qa | 15:29 | |
*** e0ne has quit IRC | 15:29 | |
*** gcheresh_ has quit IRC | 15:38 | |
*** ramishra has quit IRC | 15:41 | |
*** surpatil|dinner is now known as surpatil | 15:47 | |
*** brinzhang_ has quit IRC | 15:49 | |
*** brinzhang_ has joined #openstack-qa | 15:50 | |
*** brinzhang_ has quit IRC | 15:51 | |
openstackgerrit | Stephen Finucane proposed openstack/devstack master: stackrc: Change default Python 3 version https://review.opendev.org/687588 | 15:52 |
*** lucasagomes has quit IRC | 15:59 | |
*** udesale has quit IRC | 16:22 | |
*** ccamposr__ has joined #openstack-qa | 16:23 | |
*** ccamposr has quit IRC | 16:25 | |
*** markvoelker has quit IRC | 16:26 | |
*** rpittau is now known as rpittau|afk | 16:27 | |
*** SurajPatil has joined #openstack-qa | 16:33 | |
*** surpatil has quit IRC | 16:36 | |
*** dtantsur is now known as dtantsur|afk | 16:37 | |
*** ccamposr has joined #openstack-qa | 16:38 | |
*** ccamposr__ has quit IRC | 16:40 | |
*** jpena|off is now known as jpena | 16:44 | |
*** markvoelker has joined #openstack-qa | 17:05 | |
*** amodi has joined #openstack-qa | 17:07 | |
*** mriedem has quit IRC | 17:55 | |
*** mriedem has joined #openstack-qa | 17:56 | |
*** jpena is now known as jpena|off | 18:24 | |
*** lbragsta_ has quit IRC | 18:29 | |
*** SurajPatil has quit IRC | 18:40 | |
*** gfidente has quit IRC | 18:59 | |
*** gcheresh_ has joined #openstack-qa | 18:59 | |
*** tesseract has quit IRC | 19:07 | |
*** iurygregory has quit IRC | 19:18 | |
*** e0ne has joined #openstack-qa | 19:25 | |
*** gcheresh_ has quit IRC | 19:31 | |
openstackgerrit | Matt Riedemann proposed openstack/devstack master: Revert "Remove deprecated PostgreSQL database driver" https://review.opendev.org/689250 | 19:34 |
*** e0ne has quit IRC | 19:34 | |
clarkb | mriedem: re ^ I believe the messaging around deprecating it in pike is related to the session we had in boston (boston was pike right?) where sdague said lets remove postgres because it doesn't work and no one cares for it | 19:35 |
clarkb | mriedem: in that session Suse stood up and said "we care for it and will make it work" | 19:35 |
mriedem | the tc resolution was abandoned, | 19:36 |
clarkb | if I had a hunch the recent Suse news has bubbled that back up again as "lets delete it then" | 19:36 |
mriedem | and yes the conclusion was there are people that use it and it's not broken so why drop it | 19:36 |
clarkb | but I don't know that anyone other than suse was consulted first | 19:36 |
* clarkb will type what was typed here on the change one moment pleas | 19:36 | |
mriedem | ok, well no one is using the sev stuff that suse added to nova in train so we can drop that too | 19:36 |
clarkb | well I don't think we should drop it | 19:37 |
mriedem | i'm cool with dropping most things that are not being used and causing a nuisance, like nova-network and cells v1, but this is like super minimal energy to "maintain" | 19:37 |
clarkb | I think not dropping it if it works (your assertion with devstack) is the better path | 19:37 |
mriedem | stephen posted the patch in august so i don't think it has anything to do with the recent suse news btw | 19:38 |
clarkb | oh huh | 19:38 |
mriedem | my guess is it's just stephen wanting to delete stuff | 19:38 |
clarkb | personally I think it would help to condense the non uiser facing feature stuff (postgres vs mysql doesn't really enable user functionality) to simplify things overall in openstack. But I've made that argument for moer than half a decade and always been on the wrong side. As long as people on the other side are keeping things working I have no strong objections to making it work. Also in this particular | 19:43 |
clarkb | case having multiple DBs has been useful to shake out bugs in the past. | 19:43 |
clarkb | semi related, I think we made etcd a base service and no one really uses it at all yet, but it consumes a bunch of memory that could be used to avoid swapping in devstack jobs instead | 19:44 |
clarkb | mriedem: any idea if cinder backup ended up being removed from the default service list to help with ^ | 19:44 |
mriedem | cinder services are the only things in the integrated gate | 19:44 |
mriedem | clarkb: my patch is still sitting | 19:45 |
clarkb | :/ | 19:45 |
mriedem | in my local.conf i disable tempest, etcd, c-bak, and horizon | 19:45 |
clarkb | ya I think for our nodepool jobs we do similar | 19:45 |
* clarkb double checks | 19:45 | |
mriedem | https://review.opendev.org/#/c/651865/ is the c-bak change | 19:46 |
openstackgerrit | Matt Riedemann proposed openstack/tempest master: Disable c-bak service in tempest-full* jobs https://review.opendev.org/651865 | 19:48 |
clarkb | we disable tempest cinder and horizon but not etcd | 19:48 |
*** ccamposr__ has joined #openstack-qa | 19:49 | |
*** pcaruana has quit IRC | 19:49 | |
*** ccamposr has quit IRC | 19:50 | |
openstackgerrit | Eric Harney proposed openstack/devstack master: Cinder: only set volume_clear for LVM https://review.opendev.org/689255 | 19:51 |
*** e0ne has joined #openstack-qa | 19:52 | |
*** e0ne has quit IRC | 19:53 | |
openstackgerrit | Matt Riedemann proposed openstack/devstack master: Revert "Remove deprecated PostgreSQL database driver" https://review.opendev.org/689250 | 19:58 |
openstackgerrit | Matt Riedemann proposed openstack/devstack master: Revert "Generate deprecation warning for postgresql" https://review.opendev.org/490083 | 19:58 |
*** jdwidari has quit IRC | 20:14 | |
*** igordc has quit IRC | 20:26 | |
*** nweinber has quit IRC | 20:30 | |
openstackgerrit | Slawek Kaplonski proposed openstack/devstack master: Add possibility to configure manually MYSQL_SERVICE_NAME https://review.opendev.org/681201 | 20:34 |
*** mattw4 has quit IRC | 20:48 | |
*** mattw4 has joined #openstack-qa | 20:48 | |
*** mriedem has quit IRC | 20:49 | |
*** e0ne has joined #openstack-qa | 20:57 | |
*** igordc has joined #openstack-qa | 20:58 | |
*** trident has quit IRC | 20:58 | |
*** raildo has quit IRC | 21:01 | |
*** igordc has quit IRC | 21:03 | |
*** trident has joined #openstack-qa | 21:04 | |
*** igordc has joined #openstack-qa | 21:09 | |
*** e0ne has quit IRC | 21:11 | |
*** lbragstad has quit IRC | 21:14 | |
*** rfolco|ruck has quit IRC | 21:17 | |
*** lbragstad has joined #openstack-qa | 21:24 | |
*** rcernin has joined #openstack-qa | 21:39 | |
*** markvoelker has quit IRC | 21:39 | |
*** rcernin has quit IRC | 21:53 | |
*** slaweq has quit IRC | 21:56 | |
openstackgerrit | Phil Sphicas proposed openstack/tempest master: Update test_port_list_filter_by_ip pass conditions https://review.opendev.org/676852 | 21:59 |
*** tosky has quit IRC | 22:02 | |
*** rcernin has joined #openstack-qa | 22:09 | |
*** rcernin has quit IRC | 22:27 | |
*** rcernin has joined #openstack-qa | 22:27 | |
*** adriant has quit IRC | 22:46 | |
*** kopecmartin is now known as kopecmartin|off | 22:56 | |
*** mattw4 has quit IRC | 22:58 | |
*** mattw4 has joined #openstack-qa | 22:59 | |
*** markvoelker has joined #openstack-qa | 23:40 | |
*** mattw4 has quit IRC | 23:41 | |
*** markvoelker has quit IRC | 23:45 | |
*** nweinber has joined #openstack-qa | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!