| opendevreview | Dr. Jens Harbott proposed openstack/devstack master: CI: Make bookworm platform job voting https://review.opendev.org/c/openstack/devstack/+/895871 | 05:06 |
|---|---|---|
| *** ykarel|away is now known as ykarel | 06:33 | |
| opendevreview | Martin Kopec proposed openstack/tempest master: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/tempest/+/895876 | 07:13 |
| opendevreview | Martin Kopec proposed openstack/grenade master: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895877 | 07:16 |
| opendevreview | Martin Kopec proposed openstack/grenade stable/2023.1: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895878 | 07:28 |
| opendevreview | Martin Kopec proposed openstack/grenade stable/zed: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895879 | 07:30 |
| opendevreview | Martin Kopec proposed openstack/grenade stable/yoga: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895880 | 07:31 |
| opendevreview | Martin Kopec proposed openstack/grenade stable/xena: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895881 | 07:32 |
| opendevreview | Martin Kopec proposed openstack/grenade stable/wallaby: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895882 | 07:33 |
| opendevreview | Martin Kopec proposed openstack/grenade stable/victoria: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895883 | 07:34 |
| opendevreview | Martin Kopec proposed openstack/grenade stable/ussuri: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895884 | 07:34 |
| opendevreview | Martin Kopec proposed openstack/grenade stable/train: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895885 | 07:35 |
| kopecmartin | frickler: patches proposed ^ .. weird error on tempest, can't decode why o.O | 07:42 |
| frickler | kopecmartin: that's a 3rd party CI, got me confused multiple times, too. you can ignore that | 07:47 |
| frickler | clarkb: ^^ did you ever get feedback from storpool? else it might be time to block them after all | 07:48 |
| frickler | kopecmartin: btw., ocata, pike, queens and rocky have been eoled afaict, no branches in devstack anymore, so I think these could be dropped in the tempest matchers, too. in particular all the single "stable/ocata/ things | 07:51 |
| opendevreview | Pavlo Shchelokovskyy proposed openstack/tempest master: Use upper constraints in pep8 test run https://review.opendev.org/c/openstack/tempest/+/895627 | 08:49 |
| opendevreview | Jakub Skunda proposed openstack/tempest master: Remove test duplication between tempest and n-t-p RoutersDVRTest https://review.opendev.org/c/openstack/tempest/+/895892 | 08:51 |
| kopecmartin | frickler: right, good point, although technically it still works .. instead of 'not ^stable/ocata$' i'd need to match all not eoled branches - and it's 10 of those, i wonder if it's not easier to leave as is for now | 11:30 |
| opendevreview | Jake Yip proposed openstack/devstack master: Use OS_CLOUD in sample local.sh https://review.opendev.org/c/openstack/devstack/+/885764 | 11:32 |
| frickler | kopecmartin: you don't need to match anything in the latter case, simply drop the branch specifier? | 12:22 |
| kopecmartin | oh, i didn't think it would be that simple :D sure, i can do that then :) | 12:23 |
| frickler | I would never suggest anything making things more complicated, would I? ;;) | 12:24 |
| kopecmartin | no ;) | 12:24 |
| opendevreview | Martin Kopec proposed openstack/tempest master: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/tempest/+/895876 | 13:31 |
| opendevreview | Martin Kopec proposed openstack/grenade master: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895877 | 13:38 |
| opendevreview | Martin Kopec proposed openstack/grenade stable/2023.1: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895878 | 13:40 |
| opendevreview | Martin Kopec proposed openstack/grenade stable/xena: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895881 | 13:42 |
| opendevreview | Martin Kopec proposed openstack/grenade stable/wallaby: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895882 | 13:43 |
| opendevreview | Martin Kopec proposed openstack/grenade stable/zed: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895879 | 13:45 |
| opendevreview | Martin Kopec proposed openstack/grenade stable/train: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895885 | 13:46 |
| opendevreview | Martin Kopec proposed openstack/grenade stable/yoga: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895880 | 13:46 |
| opendevreview | Martin Kopec proposed openstack/grenade stable/victoria: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895883 | 13:47 |
| opendevreview | Martin Kopec proposed openstack/grenade stable/ussuri: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895884 | 13:48 |
| kopecmartin | gmann: hi, any chance you heard about issue with tempest when resource_setup doesn't honor the min_microversion set in the class while setUp method does? | 14:56 |
| clarkb | frickler: I don' | 15:03 |
| clarkb | frickler: er I don't see any response from storpool yet | 15:03 |
| clarkb | frickler: I would say let the cinder team know then set the account inactive and respond to the email I already sent saying this change has been made | 15:17 |
| gmann | kopecmartin: yes that is known one since starting | 15:32 |
| gmann | kopecmartin: i mean was known. and it was fixed in this, do you still see this? https://review.opendev.org/c/openstack/tempest/+/820932 | 15:34 |
| frickler | clarkb: +1 | 16:00 |
| kopecmartin | gmann: yeah, although i haven't tried with tempest, we saw that in cinder-tempest-plugin | 16:09 |
| gmann | kopecmartin: link? I can check that | 16:09 |
| kopecmartin | yeah, i don't have any, there is no review yet .. a colleague of mine is working on new tests and hit that | 16:10 |
| gmann | kopecmartin: ohk, let me fix that today | 16:10 |
| kopecmartin | gmann: he was adding new tests to this class - https://github.com/openstack/cinder-tempest-plugin/blob/master/cinder_tempest_plugin/rbac/v3/test_volume_types.py#L20 | 16:12 |
| kopecmartin | so maybe the plugin didn't consume the fix? need to check the inheritance | 16:12 |
| gmann | kopecmartin: yeah. checked the cinder-tempest-plugin code and it need fix. | 16:13 |
| gmann | I will push it today | 16:13 |
| kopecmartin | wau, thank you for such quick response | 16:15 |
| kopecmartin | btw, do we need to check other plugins to see if they might have the same issue? | 16:15 |
| gmann | yeah, I will check if they still use old base class way | 16:16 |
| gmann | I think keystone might be and not other services does not have microversion | 16:16 |
| gmann | for exmaple, glance, neutron all good as they do not have microversion | 16:17 |
| opendevreview | Merged openstack/tempest master: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/tempest/+/895876 | 17:26 |
| opendevreview | Merged openstack/grenade master: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895877 | 17:26 |
| frickler | bookworm are you kidding me? https://zuul.opendev.org/t/openstack/build/ade5df7d52994b8cb54181bb69c51041 | 17:43 |
| gmann | kopecmartin: this should fix the cinder-tempest-pluing, please check with your colleague if this fix the issue https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/895972 | 18:38 |
| kopecmartin | gmann: perfect! thank you very much | 18:46 |
| gmann | np! | 18:46 |
| gmann | kopecmartin: btw this is ready https://review.opendev.org/c/openstack/tempest/+/885841 | 18:53 |
| opendevreview | Merged openstack/grenade stable/2023.1: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895878 | 19:17 |
| opendevreview | Merged openstack/grenade stable/zed: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895879 | 19:17 |
| opendevreview | Merged openstack/grenade stable/yoga: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895880 | 19:17 |
| opendevreview | Merged openstack/grenade stable/xena: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895881 | 19:17 |
| opendevreview | Merged openstack/grenade stable/victoria: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895883 | 20:09 |
| opendevreview | Merged openstack/grenade stable/ussuri: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895884 | 20:09 |
| opendevreview | Merged openstack/grenade stable/train: Use re2 compatible regexes in branch matchers https://review.opendev.org/c/openstack/grenade/+/895885 | 20:09 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Update Tempest supported releases and python versions https://review.opendev.org/c/openstack/tempest/+/895982 | 21:31 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Add releasenote to tag the Tempest for 2023.2 release https://review.opendev.org/c/openstack/tempest/+/895984 | 21:36 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Use stable constraint in tox to release new tag for 2023.2 https://review.opendev.org/c/openstack/tempest/+/895985 | 21:40 |
| gmann | kopecmartin: ^^ starting the 2023.2 release for tempest, but I am waiting these to merge first | 21:44 |
| gmann | - https://review.opendev.org/c/openstack/tempest/+/887857 | 21:44 |
| gmann | - https://review.opendev.org/c/openstack/tempest/+/863854 | 21:44 |
| gmann | - https://review.opendev.org/c/openstack/tempest/+/894822 | 21:44 |
| gmann | - https://review.opendev.org/c/openstack/tempest/+/892677 | 21:44 |
| gmann | did recheck on those, let's see | 21:44 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Switch to the master constraint in tox.ini https://review.opendev.org/c/openstack/tempest/+/895986 | 21:44 |
| opendevreview | Merged openstack/tempest master: Update object_storage container creation using metadata tests to accept 204 reponses https://review.opendev.org/c/openstack/tempest/+/887857 | 22:21 |
| opendevreview | Merged openstack/tempest master: Remove python-devel package for rpm based distributions from bindep https://review.opendev.org/c/openstack/tempest/+/863854 | 22:57 |
| opendevreview | Merged openstack/tempest master: Fixed typo in override-checkout branch for stable/2023.1 https://review.opendev.org/c/openstack/tempest/+/894822 | 22:57 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!