| opendevreview | Artom Lifshitz proposed openstack/whitebox-tempest-plugin master: Refactor ceph job definition. https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/828800 | 01:22 |
|---|---|---|
| opendevreview | Artom Lifshitz proposed openstack/whitebox-tempest-plugin master: Refactor ceph job definition https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/828800 | 02:29 |
| opendevreview | Artom Lifshitz proposed openstack/whitebox-tempest-plugin master: Add barbican to our devstack job https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/779043 | 02:29 |
| opendevreview | Artom Lifshitz proposed openstack/whitebox-tempest-plugin master: Consolidate roles into pre playbook as tasks https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/828589 | 03:00 |
| opendevreview | Artom Lifshitz proposed openstack/whitebox-tempest-plugin master: [WIP] Add tasks that adds hugepages to computes https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/825011 | 03:00 |
| opendevreview | Artom Lifshitz proposed openstack/whitebox-tempest-plugin master: Refactor ceph job definition https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/828800 | 03:00 |
| opendevreview | Artom Lifshitz proposed openstack/whitebox-tempest-plugin master: Add barbican to our devstack job https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/779043 | 03:00 |
| *** akahat|PTO is now known as akahat | 05:22 | |
| *** clarkb is now known as Guest32 | 05:22 | |
| *** jpena|off is now known as jpena | 08:36 | |
| opendevreview | Bernard Cafarelli proposed openstack/devstack stable/stein: Add python3.6 pip support https://review.opendev.org/c/openstack/devstack/+/828769 | 09:11 |
| *** arxcruz is now known as arxcruz|ruck | 09:13 | |
| *** pojadhav is now known as pojadhav|afk | 09:46 | |
| *** pojadhav|afk is now known as pojadhav | 10:51 | |
| opendevreview | Balazs Gibizer proposed openstack/devstack master: DNM: change detach timeout in centos-9-stream https://review.opendev.org/c/openstack/devstack/+/828705 | 11:00 |
| *** pojadhav is now known as pojadhav|brb | 11:25 | |
| *** pojadhav|brb is now known as pojadhav | 11:39 | |
| opendevreview | Balazs Gibizer proposed openstack/tempest master: DNM: test if detach fails due to guest not fully booted https://review.opendev.org/c/openstack/tempest/+/828863 | 11:49 |
| opendevreview | Balazs Gibizer proposed openstack/devstack master: DNM: change detach timeout in centos-9-stream https://review.opendev.org/c/openstack/devstack/+/828705 | 12:02 |
| opendevreview | Balazs Gibizer proposed openstack/tempest master: Introduce @serial test execution decorator https://review.opendev.org/c/openstack/tempest/+/821732 | 12:26 |
| *** pojadhav is now known as pojadhav|brb | 13:05 | |
| opendevreview | Balazs Gibizer proposed openstack/tempest master: DNM: test if detach fails due to guest not fully booted https://review.opendev.org/c/openstack/tempest/+/828863 | 13:19 |
| opendevreview | Balazs Gibizer proposed openstack/devstack master: DNM: troubleshoot detach in centos-9-stream job https://review.opendev.org/c/openstack/devstack/+/828705 | 13:23 |
| opendevreview | Rodolfo Alonso proposed openstack/devstack master: Fix installation with OVN backend and compilation https://review.opendev.org/c/openstack/devstack/+/828877 | 13:39 |
| *** ykarel is now known as ykarel|away | 14:56 | |
| opendevreview | Rodolfo Alonso proposed openstack/devstack master: Fix installation with OVN backend and compilation https://review.opendev.org/c/openstack/devstack/+/828877 | 15:06 |
| opendevreview | Rodolfo Alonso proposed openstack/devstack master: [DNM] Test patch 828877 https://review.opendev.org/c/openstack/devstack/+/828895 | 15:06 |
| jparoly | frickler: Regarding https://bugs.launchpad.net/devstack/+bug/1939230, should we lower the priority now that the bug is in the "incomplete" state? | 15:37 |
| frickler | jparoly: I'd give the original reporter a bit more time to respond, then together with your feedback I'd mark it as resolved | 15:43 |
| *** Guest32 is now known as clarkb | 15:47 | |
| opendevreview | Rodolfo Alonso proposed openstack/devstack master: Fix installation with OVN backend and compilation https://review.opendev.org/c/openstack/devstack/+/828877 | 15:53 |
| *** pojadhav|brb is now known as pojadhav|out | 16:24 | |
| opendevreview | Merged openstack/devstack stable/ussuri: Drop centos8 nodeset https://review.opendev.org/c/openstack/devstack/+/828804 | 16:53 |
| *** artom__ is now known as artom | 16:56 | |
| artom | jparker_, btw, I stacked a a couple of job-related patches on top of your hugepage changes. I needed yours to get the proper hugepage handling when there's no num_hugepages defined for the job. | 16:57 |
| artom | Just a heads up, we can continue reviewing the hugepages stuff as normal, I just wanted CI to confirm whether my things worked | 16:57 |
| artom | ... and they did :) The ceph job was mostly broken, it turns out, and because of that we couldn't get the barbican addition to pass CI. Now both work. | 16:57 |
| artom | (Pending +A, obviously) | 16:58 |
| *** jpena is now known as jpena|off | 17:33 | |
| opendevreview | Luigi Toscano proposed openstack/devstack-plugin-ceph master: Disable horizon from basic https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/828910 | 17:41 |
| jparker_ | artom: ack, sorry was in a triage meeting early and then went off to lunch | 18:22 |
| jparker_ | artom: I'll try and update the base commits to get them passing today | 18:22 |
| artom | jparker_, no rush, just an FYI | 18:26 |
| opendevreview | Brian Haley proposed openstack/devstack master: Add support for IPv6 tunnel endpoints https://review.opendev.org/c/openstack/devstack/+/710519 | 18:50 |
| opendevreview | Brian Haley proposed openstack/tempest master: Add an IPv6 tunnel endpoint job definition https://review.opendev.org/c/openstack/tempest/+/828431 | 18:51 |
| *** carloss is now known as carloss|afk | 19:12 | |
| jparoly | frickler: ack, thanks | 19:20 |
| opendevreview | James Parker proposed openstack/whitebox-tempest-plugin master: Consolidate roles into pre playbook as tasks https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/828589 | 19:20 |
| opendevreview | James Parker proposed openstack/whitebox-tempest-plugin master: Add tasks that adds hugepages to computes https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/825011 | 19:20 |
| opendevreview | Merged openstack/devstack stable/stein: Add python3.6 pip support https://review.opendev.org/c/openstack/devstack/+/828769 | 19:57 |
| opendevreview | Luigi Toscano proposed openstack/devstack-plugin-ceph master: Disable horizon from basic tempest tests https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/828910 | 20:23 |
| opendevreview | Merged openstack/whitebox-tempest-plugin master: Consolidate roles into pre playbook as tasks https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/828589 | 21:39 |
| opendevreview | Artom Lifshitz proposed openstack/whitebox-tempest-plugin master: Refactor ceph job definition https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/828800 | 21:46 |
| opendevreview | Artom Lifshitz proposed openstack/whitebox-tempest-plugin master: Add barbican to our devstack job https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/779043 | 21:46 |
| opendevreview | Artom Lifshitz proposed openstack/whitebox-tempest-plugin master: Refactor ceph job definition https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/828800 | 21:52 |
| opendevreview | Artom Lifshitz proposed openstack/whitebox-tempest-plugin master: Add barbican to our devstack job https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/779043 | 21:52 |
| ade_lee | tosky, gmann hey - do you have an example again on how to skip a test? | 21:58 |
| ade_lee | as a definition within a job | 21:58 |
| *** carloss|afk is now known as carloss | 21:58 | |
| artom | ade_lee, the tempest_exclude_regex decalaration? | 22:01 |
| ade_lee | artom, yeah .. | 22:01 |
| artom | ade_lee, I mean, that's just your answer, no? :) | 22:02 |
| ade_lee | artom, I think so --- so if I want to exclude .. | 22:02 |
| tosky | ade_lee: https://opendev.org/openstack/cinder-tempest-plugin/src/branch/master/.zuul.yaml#L111 | 22:03 |
| tosky | ade_lee: documentation: https://opendev.org/openstack/tempest/raw/branch/master/roles/run-tempest/README.rst | 22:03 |
| tosky | oh, right, also tempest_exclude_regex | 22:03 |
| tosky | it depends on how many regexp you want to put there I guess | 22:03 |
| ade_lee | test_encrypted_cinder_volumes_cryptsetup | 22:03 |
| ade_lee | as in tempest.scenario.test_encrypted_cinder_volumes.TestEncryptedCinderVolumes.test_encrypted_cinder_volumes_cryptsetup | 22:04 |
| ade_lee | and as in .. | 22:04 |
| artom | `tempest_exclude_regex: test_encrypted_cinder_volumes_cryptsetup` would do it, methinks | 22:04 |
| artom | Anything that matches that would not get run | 22:04 |
| tosky | ade_lee: yep, like https://opendev.org/openinfra/python-tempestconf/src/branch/master/.zuul.yaml#L270 | 22:04 |
| artom | Right, it's in `vars`, that's the other important piece | 22:05 |
| ade_lee | cool, and it would also exclude a test with the same name in the barbcian_tempest _plugin | 22:05 |
| tosky | ade_lee: if you want to exclude all cryptsetup tests, it's better to use test_encrypted_cinder_volumes_cryptsetup, so you will skip also the other test in barbican_tempest_plugin if that one is loaded | 22:05 |
| artom | Anything that matches baby :) | 22:05 |
| tosky | yep | 22:05 |
| tosky | :D | 22:05 |
| ade_lee | excellent - thanks! | 22:05 |
| ade_lee | artom, tosky got a different question .. | 22:21 |
| ade_lee | https://review.opendev.org/c/openstack/horizon/+/825875 | 22:22 |
| ade_lee | artom, tosky so the fips job there (one of them) fails because its trying to run selenium which requires ffmpeg | 22:22 |
| ade_lee | which isn't in centos .. | 22:23 |
| ade_lee | and which I can get by getting another repo | 22:23 |
| ade_lee | right now its being pulled in by bindeps | 22:23 |
| artom | ade_lee, ah, and I'm assuming the base horizon-integration-tests job passes because selenium is in Ubuntu, which it runs on? | 22:23 |
| artom | ... and we can't test FIPS on Ubuntu, we need CentOS... | 22:23 |
| ade_lee | yup | 22:24 |
| ade_lee | artom, but we can get it in a separate repo .. | 22:24 |
| artom | I'd add a pre-run playbook that justs adds the repo (EPEL?) and installs the package... | 22:24 |
| artom | IIRC pre-run plabooks in Zuul are "stacked", not replaced | 22:24 |
| artom | So parent playbooks are run with whatever you add in your job | 22:25 |
| artom | Not sure about the order tho | 22:25 |
| tosky | uhm | 22:25 |
| ade_lee | ack yeah, that sounds good, as long as it runs before the bindep | 22:25 |
| tosky | ade_lee: do you have a quick link to the traceback? | 22:25 |
| ade_lee | then again , I can always take it out of the bindep for centos .. | 22:25 |
| ade_lee | tosky, sure .. | 22:26 |
| tosky | ade_lee: I'd say: remove the bindep for centos, I think ffmpeg is there so that you can record the operation and the error | 22:26 |
| tosky | but last time I've touched horizon tests was loooong time ago | 22:26 |
| ade_lee | tosky, https://zuul.opendev.org/t/openstack/build/b4ebdb8ec82a4fdc9f63718adf5c6689 | 22:26 |
| ade_lee | tosky, ffmpeg aint there .. selenium might not be either tbh | 22:27 |
| ade_lee | it just hasn't failed there yet | 22:28 |
| ade_lee | https://linuxize.com/post/how-to-install-ffmpeg-on-centos-8/ | 22:28 |
| tosky | ade_lee: then yes, just fix bindeps.txt | 22:30 |
| ade_lee | tosky, artom https://opendev.org/openstack/horizon/src/branch/master/bindep.txt | 22:30 |
| ade_lee | tosky, fix bindep and then add pre-run to install ffmpeg :) | 22:31 |
| tosky | ffmpeg [selenium !platform:redhat] | 22:31 |
| tosky | the code survives, it just skips the videorecording | 22:31 |
| ade_lee | oh, ok .. | 22:32 |
| tosky | https://opendev.org/openstack/horizon/src/branch/master/openstack_dashboard/test/integration_tests/video_recorder.py#L42 | 22:32 |
| tosky | oh, you need to set FFMPEG_INSTALLED: False | 22:32 |
| tosky | as zuul variable for the job | 22:33 |
| ade_lee | ah cool | 22:33 |
| artom | ade_lee, wait, wouldn't bindep be OK if you add the repo in the pre-run playbook? | 22:33 |
| tosky | uhm, maybe this code needs to have some conditional https://opendev.org/openstack/horizon/src/branch/master/playbooks/horizon-devstack-integration/run.yaml | 22:33 |
| ade_lee | what about xvfb? | 22:33 |
| artom | Though I suppose the technically correct thing to do is still to fix it | 22:33 |
| artom | Since selenium is not there in the default repos | 22:33 |
| tosky | ade_lee: uh, it's not in cs9 either ? | 22:34 |
| ade_lee | tosky, I dont see it in brew .. unless thats not the package name .. | 22:34 |
| ade_lee | oh wait -- does "xvfb [selenium platform:dpkg]" mean that its needed when runnng selenium and debian? | 22:36 |
| ade_lee | key being AND | 22:36 |
| ade_lee | so it shouldn't be an issue for centos .. | 22:36 |
| tosky | I mean, it's probably faster to disable it for now and see what else breaks (I think it may be a reuqirement for polarion) | 22:37 |
| tosky | ade_lee: yes, it's an AND | 22:37 |
| tosky | (should that be useful, the package is xorg-x11-server-Xvfb) | 22:38 |
| tosky | but again I guess you want to stabilize non-selenium tests first | 22:38 |
| ade_lee | actually in this case, its not an AND | 22:38 |
| ade_lee | Profiles can also be grouped together using (). In a group, all profiles must match for the group to match. Given the example [test (ceph glance !lvm)], to select the package you must either specify test OR (ceph AND glance AND NOT lvm). | 22:39 |
| ade_lee | so OR unless in () | 22:39 |
| tosky | ade_lee: not for platforms | 22:39 |
| tosky | Note that platform selectors are treated as kind of filter: If a line contains a platform selector, then the package only gets installed if at least one of the platform selectors matches in addition to the match on the other selectors. As an example, [platform:rpm test] would only install a package on a RPM platform if the test selector is used. | 22:40 |
| tosky | ade_lee: the line above | 22:40 |
| tosky | s/line/paragraph/ | 22:40 |
| ade_lee | ah cool | 22:40 |
| ade_lee | so we should be good | 22:40 |
| ade_lee | cool - let me give it a shot | 22:41 |
| opendevreview | Brian Haley proposed openstack/devstack master: Add support for IPv6 tunnel endpoints https://review.opendev.org/c/openstack/devstack/+/710519 | 22:46 |
| opendevreview | Brian Haley proposed openstack/tempest master: Add an IPv6 tunnel endpoint job definition https://review.opendev.org/c/openstack/tempest/+/828431 | 22:50 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!