| opendevreview | Merged openstack/grenade stable/wallaby: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895882 | 01:24 |
|---|---|---|
| opendevreview | Pranali Deore proposed openstack/tempest master: Add tests for new add location api of glance https://review.opendev.org/c/openstack/tempest/+/891026 | 06:34 |
| *** elodilles_pto is now known as elodilles | 08:47 | |
| opendevreview | Lukas Piwowarski proposed openstack/tempest master: Revert "Revert "Fix cleanup for volume backup tests"" https://review.opendev.org/c/openstack/tempest/+/896011 | 09:19 |
| opendevreview | Lukas Piwowarski proposed openstack/tempest master: Revert "Revert "Fix cleanup for volume backup tests"" https://review.opendev.org/c/openstack/tempest/+/896011 | 09:22 |
| opendevreview | Lukas Piwowarski proposed openstack/tempest master: Revert "Revert "Fix cleanup for volume backup tests"" https://review.opendev.org/c/openstack/tempest/+/896011 | 09:26 |
| opendevreview | Lukas Piwowarski proposed openstack/tempest master: Revert "Revert "Fix cleanup for volume backup tests"" https://review.opendev.org/c/openstack/tempest/+/896011 | 09:30 |
| opendevreview | Lukas Piwowarski proposed openstack/tempest master: Revert "Revert "Fix cleanup for volume backup tests"" https://review.opendev.org/c/openstack/tempest/+/896011 | 09:33 |
| opendevreview | Merged openstack/tempest master: Remove test duplication between tempest and n-t-p QuotasTest https://review.opendev.org/c/openstack/tempest/+/892677 | 11:39 |
| opendevreview | Merged openstack/tempest master: Update Tempest supported releases and python versions https://review.opendev.org/c/openstack/tempest/+/895982 | 12:17 |
| kopecmartin | gmann: do we have any specific rules for new tempest plugins? i mean for the content , type of tests etc? tempest related as well as upstream related | 12:27 |
| opendevreview | Dan Smith proposed openstack/devstack master: Fix g-api-r for non-global venv https://review.opendev.org/c/openstack/devstack/+/896123 | 14:13 |
| opendevreview | Dan Smith proposed openstack/devstack master: Fix g-api-r for non-global venv https://review.opendev.org/c/openstack/devstack/+/896123 | 14:22 |
| -opendevstatus- NOTICE: The lists.openinfra.dev and lists.starlingx.io sites will be offline briefly for migration to a new server | 15:31 | |
| gmann | kopecmartin: which project? for service which Tempest has in-tree tests? or other? | 16:07 |
| opendevreview | Dan Smith proposed openstack/devstack master: Fix g-api-r for non-global venv https://review.opendev.org/c/openstack/devstack/+/896123 | 16:54 |
| gmann | kopecmartin: for in-tree services, we recommend to add tests in tempest itself but if more API and non-integrated tests needs to be added which are not good to add more in Tempest as per its scope then they can create tempest plugin but keep coverage be exlicit and do not duplicate the things. Add integration tests in tempest itself so that other services can also run them. | 17:06 |
| gmann | kopecmartin: for other services, it is all good and they can create plugins anytime and add the tests as they want. keeping same dir structure as we have for plugins so that tempest can detect that and run tests | 17:07 |
| JayF | gmann: Question for you, just wanna be sure we're on the same page. A former Ironic PTL and core mentioned that usually QA team cuts ironic-tempest-plugin releases even though it's listed in Ironic namespace. (I am 10000% OK with this arrangement, to be clear) Wanted to be sure that was the case and I wasn't leaving you to do something you're not signed up for :D | 19:03 |
| gmann | JayF: usually release team do before me but I do for EM things when we release last supported tag for tempest and plugins. but PTL doing the final release as per the changes they want to merge before is always better | 19:06 |
| JayF | So it sounds like it's in limbo where usually releases team does it but cutting it at the same time as the project is ideal (for seemingly obvious reasons) | 19:06 |
| gmann | JayF: I know many plugins are not active to I do for left over but as ironic-tempest-plugin is very active plugin I will leave that to you as PTL and I can review and do other pending things if needed | 19:07 |
| JayF | ack | 19:07 |
| gmann | yeah | 19:07 |
| JayF | I'll push it now then :D | 19:07 |
| gmann | +1, thanks | 19:07 |
| JayF | iurygregory: ^ re: if we should cut ironic-tempest-plugin (tl;dr: we don't have to cut it, but it's better if we do it so it gets done at the same time as ironic) | 19:07 |
| frickler | there was a stack of changes proposed at https://review.opendev.org/q/topic:bobcat-tp-latest , maybe elodilles can explain why ironic was missing | 19:07 |
| JayF | I'm not reporting an issue so much as trying to find the happiest possible path :D | 19:08 |
| JayF | I'll double check to ensure there's not one for i-t-p and put one up | 19:08 |
| gmann | yeah, there might be a few more plugins missing | 19:08 |
| JayF | hah, it's not in that because it's already landed | 19:10 |
| JayF | https://github.com/openstack/releases/commit/d52b3296c493fc6aa562256a7f724cea9f1669a3 that looks almost accidental | 19:11 |
| JayF | it's not in the commit message, at least. No big deal it's the right sha and everything | 19:12 |
| gmann | JayF: that is when I did for marking last supported release for xena | 19:12 |
| JayF | You are too fast, you're doing my job for me months before I needed to do it myself ;) | 19:12 |
| JayF | thank you | 19:12 |
| gmann | JayF: you can push new release if more tests were added after that | 19:12 |
| JayF | none were | 19:12 |
| frickler | oh, so no changes since then, ok | 19:12 |
| gmann | k | 19:12 |
| frickler | I just double checked the plugin is indeed listed at https://docs.openstack.org/tempest/latest/plugins/plugin-registry.html | 19:13 |
| gmann | yes | 19:13 |
| gmann | that explain why elodilles did not release it :) | 19:13 |
| gmann | and might be the case for other missing 6 plugins | 19:14 |
| gmann | I tagged them xena-last and no change after that | 19:14 |
| iurygregory | JayF, ack | 21:29 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!