*** pojadhav|afk is now known as pojadhav | 04:45 | |
opendevreview | Francesco Pantano proposed openstack/devstack-plugin-ceph master: Deploy with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/826484 | 07:08 |
---|---|---|
*** arxcruz|off is now known as arxcruz | 07:47 | |
*** jpena|off is now known as jpena | 08:38 | |
opendevreview | Benny Kopilov proposed openstack/tempest master: Fix test_basic_metadata_definition_namespaces https://review.opendev.org/c/openstack/tempest/+/833750 | 08:38 |
opendevreview | Benny Kopilov proposed openstack/tempest master: Fix test_basic_metadata_definition_namespaces https://review.opendev.org/c/openstack/tempest/+/833750 | 08:41 |
*** arxcruz is now known as arxcruz|brb | 09:20 | |
opendevreview | mitya-eremeev-2 proposed openstack/tempest master: Tempest tests are aware of networks https://review.opendev.org/c/openstack/tempest/+/831784 | 09:38 |
opendevreview | Soniya Murlidhar Vyas proposed openstack/tempest master: Add compute_security_client and compute_security_ group_rules_client https://review.opendev.org/c/openstack/tempest/+/833758 | 10:05 |
*** pojadhav- is now known as pojadhav | 11:05 | |
opendevreview | Francesco Pantano proposed openstack/devstack-plugin-ceph master: Deploy with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/826484 | 11:20 |
*** pojadhav is now known as pojadhav|break | 11:35 | |
*** arxcruz|brb is now known as arxcruz | 11:42 | |
*** pojadhav|break is now known as pojadhav | 12:08 | |
opendevreview | Benny Kopilov proposed openstack/tempest master: Fix test_basic_metadata_definition_namespaces https://review.opendev.org/c/openstack/tempest/+/833750 | 12:54 |
opendevreview | Douglas Viroel proposed openstack/tempest master: DNM - Testing CI https://review.opendev.org/c/openstack/tempest/+/833778 | 12:58 |
opendevreview | Soniya Murlidhar Vyas proposed openstack/tempest master: Add compute_floating_ips_client https://review.opendev.org/c/openstack/tempest/+/833788 | 13:30 |
opendevreview | Francesco Pantano proposed openstack/devstack-plugin-ceph master: Deploy with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/826484 | 13:51 |
dansmith | clarkb: frickler: did you see my comment here? https://review.opendev.org/c/openstack/devstack/+/831245 | 14:12 |
frickler | dansmith: yes, but wanted to discuss at the upcoming meeting | 14:15 |
dansmith | frickler: cool | 14:18 |
kopecmartin | #startmeeting qa | 15:00 |
opendevmeet | Meeting started Tue Mar 15 15:00:13 2022 UTC and is due to finish in 60 minutes. The chair is kopecmartin. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
opendevmeet | The meeting name has been set to 'qa' | 15:00 |
kopecmartin | #link https://wiki.openstack.org/wiki/Meetings/QATeamMeeting#Weekly_QA_Team_meeting | 15:00 |
soniya29 | kopecmartin, hello | 15:00 |
kopecmartin | agenda ^^ | 15:00 |
kopecmartin | soniya29: hi | 15:00 |
gmann | o/ | 15:01 |
kopecmartin | o/ | 15:01 |
kopecmartin | let's start | 15:01 |
kopecmartin | #topic Announcement and Action Item (Optional) | 15:01 |
kopecmartin | we have release a new version / tag of tempest | 15:01 |
kopecmartin | 30.0.0 | 15:01 |
kopecmartin | gmann: anything else we need to do? i'm checking the qa's release wiki page | 15:02 |
kopecmartin | #link https://wiki.openstack.org/wiki/QA/releases | 15:02 |
kopecmartin | w.r.t. devstack and grenade we're waiting for a stable branch i guess | 15:03 |
gmann | I think yes, for tempest all good. and next is to wait for signal from release team once we need to start the devstack and grenade things | 15:04 |
gmann | elodilles: ^^ | 15:04 |
kopecmartin | good | 15:04 |
kopecmartin | #topic Yoga Priority Items progress | 15:05 |
kopecmartin | #link https://etherpad.opendev.org/p/qa-yoga-priority | 15:05 |
kopecmartin | any updates? | 15:05 |
soniya29 | yes | 15:05 |
opendevreview | Stephen Finucane proposed openstack/hacking master: Update CI to use unversioned jobs template https://review.opendev.org/c/openstack/hacking/+/833811 | 15:06 |
soniya29 | https://review.opendev.org/q/topic:tempest-scenario-manager-cleanup+project:+openstack/barbican-tempest-plugin+owner:+self | 15:06 |
gmann | soniya29: you are including self in query. you mean this series right? #link https://review.opendev.org/q/topic:tempest-scenario-manager-cleanup+project:+openstack/barbican-tempest-plugin | 15:07 |
soniya29 | gmann, yeah | 15:07 |
kopecmartin | nice, there are many patches incomming related to the cleanup | 15:08 |
kopecmartin | #link https://review.opendev.org/q/topic:tempest-scenario-manager-cleanup | 15:08 |
kopecmartin | thank you soniya29 and rpopelka | 15:08 |
soniya29 | apart from this | 15:08 |
kopecmartin | i'm gonna review (at least some) right after this meeting | 15:08 |
gmann | soniya29: why we are replacing the methods one by one instead of replacing whole scenario manager? | 15:08 |
gmann | at least for all methods we have in tempest.scenario manager as stable | 15:08 |
kopecmartin | gmann: plugins may have their own custom methods in there | 15:09 |
kopecmartin | plus it's easier to have smaller reviews | 15:09 |
soniya29 | gmann, we have some customization in the plugin methods | 15:09 |
soniya29 | gmann, so need to review the method implementation first and the remove them | 15:09 |
gmann | kopecmartin: yeah, I mean just leave those and replace everything in one shot ? there are less plugins having their custom method | 15:09 |
gmann | soniya29: I doubt, bcz I copied them as it is from what tempest has | 15:10 |
soniya29 | in next meeting, i will make a list of those custom methods, so that we can discuss about them | 15:10 |
soniya29 | but yes there are such methods whose implementation varies from tempest scenario manager | 15:11 |
gmann | for example, barbican case what is the last patch? is this? #link https://review.opendev.org/c/openstack/barbican-tempest-plugin/+/833801 | 15:11 |
soniya29 | gmann, kopecmartin, also i found that in some methods we are missing on some clients in tempest | 15:12 |
kopecmartin | gmann: well, sure, if there is a plugin, which doesn't have anything custom in their manager, we can do that in one shot, but there might be even a small customization, like port name, let's take blazar as an example - https://review.opendev.org/c/openstack/blazar-tempest-plugin/+/830595/5/blazar_tempest_plugin/tests/scenario/manager_freeze.py | 15:12 |
soniya29 | gmann, yes, that's the last one i posted | 15:12 |
gmann | soniya29: it is hard to see the series, is it new gerrit UI or just me not able to see the dependencies? you did it with dependencies right? not all patch as separate in barbican tempest plugin? | 15:13 |
kopecmartin | gmann: i don't think they are dependent | 15:14 |
gmann | kopecmartin: yeah for such plugins we can do in multiple steps. | 15:14 |
kopecmartin | do they need to be? | 15:14 |
soniya29 | gmann, i have dependencies on tempest | 15:14 |
soniya29 | s/have/have added | 15:14 |
soniya29 | because we are missing some clients in tempest which are available in plugin | 15:14 |
soniya29 | here barbican-tempest-plugin | 15:14 |
gmann | kopecmartin: soniya29 ok then you will have merge conflict if single plugins changes are not dependent | 15:14 |
kopecmartin | oh, yeah , that's true | 15:15 |
kopecmartin | but first someone from the plugin needs to review | 15:15 |
gmann | that is why i was confused on how to review it and see what is end pic of scenario manager | 15:15 |
gmann | anyways I will review them and put comment there | 15:16 |
soniya29 | gmann, kopecmartin, https://review.opendev.org/c/openstack/tempest/+/833788 | 15:16 |
soniya29 | https://review.opendev.org/c/openstack/tempest/+/833758 | 15:16 |
gmann | kopecmartin: I think we can also verify it with what implementation we have or via git history is easy. | 15:16 |
gmann | soniya29: ok so those are conflict and plugins cannot use it from master | 15:17 |
gmann | or everything is from master now? | 15:18 |
kopecmartin | soniya29: please rebase those 2 patches in tempest, put one of them on top of the other .. we can merge them , however, plugins will be able to use them in a few cycles, not right now | 15:19 |
kopecmartin | otherwise we would break the plugins compatibility with the older tempest | 15:19 |
soniya29 | gmann, i am not sure :) | 15:19 |
gmann | kopecmartin: I am trying to recall on that why we waited for replacement until train use old tempest? | 15:19 |
gmann | ohk | 15:20 |
soniya29 | kopecmartin, okay | 15:20 |
gmann | so if plugin master is using the old tempest and these new things can break them | 15:20 |
kopecmartin | gmann: i think we waited a few cycles only because we didn't want to make the plugins incompatible with one release older tempest | 15:20 |
gmann | yeah | 15:20 |
kopecmartin | and we kinda agreed that a few cycle difference = a few releases of tempest (3?) are enough | 15:21 |
soniya29 | kopecmartin, gmann, and we wait more now then we will have more divergence i guess :) | 15:21 |
soniya29 | s/and/and if | 15:21 |
soniya29 | kopecmartin, yupp | 15:21 |
gmann | soniya29: we can do place the things what present in tempest 27.0.0 not what we moved/added after that | 15:21 |
gmann | I will review those patches and add detail comment. | 15:22 |
soniya29 | gmann, sure | 15:22 |
kopecmartin | i'll review too | 15:22 |
kopecmartin | good | 15:22 |
kopecmartin | any other updates regarding the priority items? | 15:23 |
soniya29 | nope | 15:24 |
gmann | yeah, let's review and decide the direction before we have lot more patches | 15:24 |
gmann | nothing else from me | 15:24 |
kopecmartin | gmann: you mean right now? | 15:24 |
gmann | kopecmartin: i mean in review | 15:25 |
kopecmartin | oh, yeah, we can do that , sure :) | 15:25 |
kopecmartin | moving on then | 15:25 |
kopecmartin | #topic OpenStack Events Updates and Planning | 15:25 |
gmann | or we can discuss in PTG | 15:25 |
kopecmartin | we have etherpad for the topics to discuss during PTG sessions | 15:25 |
kopecmartin | #link https://etherpad.opendev.org/p/qa-zed-ptg | 15:25 |
gmann | if review comments are not understandable or easy to understand. | 15:25 |
kopecmartin | anyone, please add there any topic you want to discuss | 15:25 |
kopecmartin | gmann: i agree | 15:26 |
soniya29 | gmann, sure | 15:26 |
kopecmartin | in the ptg etherpad time slots we booked for QA can be found as well | 15:26 |
gmann | kopecmartin: soniya29 added it in etherpad, I will add detail/overview later | 15:26 |
kopecmartin | gmann: perfect | 15:27 |
gmann | so we can reivew add our thoughts and then discuss and finalize the direction in PTG | 15:27 |
kopecmartin | i have to add a few topics too, can't force myself to write it up | 15:27 |
kopecmartin | gmann: +1 | 15:27 |
soniya29 | gmann, till then, should we slow down/stop the patches? | 15:27 |
kopecmartin | soniya29: let's see based on the number of comments and things raised in them | 15:28 |
soniya29 | kopecmartin, sounds good :) | 15:28 |
gmann | yeah | 15:29 |
kopecmartin | #topic Gate Status Checks | 15:29 |
kopecmartin | #link https://review.opendev.org/q/label:Review-Priority%253D%252B2+status:open+(project:openstack/tempest+OR+project:openstack/patrole+OR+project:openstack/devstack+OR+project:openstack/grenade+OR+project:openstack/hacking) | 15:29 |
kopecmartin | one review there | 15:30 |
kopecmartin | i workflowed that tls-proxy patch | 15:30 |
kopecmartin | frickler: dansmith ^^ | 15:30 |
dansmith | thanks | 15:30 |
kopecmartin | dansmith: thank you for chasing the errors | 15:31 |
kopecmartin | now let's see what will get broken :D | 15:31 |
dansmith | yep :) | 15:31 |
kopecmartin | .. i think it will be fine, either way we would need to address that , sooner or later | 15:31 |
kopecmartin | #topic Periodic jobs Status Checks | 15:32 |
kopecmartin | #link https://zuul.openstack.org/builds?job_name=tempest-full-xena-py3&job_name=tempest-full-wallaby-py3&job_name=tempest-full-victoria-py3&job_name=tempest-full-ussuri-py3&job_name=tempest-full-train-py3&pipeline=periodic-stable | 15:32 |
kopecmartin | #link https://zuul.openstack.org/builds?project=openstack%2Ftempest&project=openstack%2Fdevstack&pipeline=periodic | 15:32 |
kopecmartin | seems all good there | 15:32 |
kopecmartin | #topic Sub Teams highlights | 15:32 |
kopecmartin | Changes with Review-Priority == +1 | 15:32 |
kopecmartin | #link https://review.opendev.org/q/label:Review-Priority%253D%252B1+status:open+(project:openstack/tempest+OR+project:openstack/patrole+OR+project:openstack/devstack+OR+project:openstack/grenade+OR+project:openstack/hacking) | 15:32 |
kopecmartin | no changes | 15:33 |
kopecmartin | #topic Open Discussion | 15:33 |
kopecmartin | anything for the open discussion? | 15:33 |
gmann | nothing else from me. I need to find time to check the SSH-able failure on rescue tests. I will try today or tomorrow | 15:34 |
kopecmartin | ok, thanks gmann | 15:34 |
kopecmartin | #topic Bug Triage | 15:35 |
kopecmartin | #link https://etherpad.opendev.org/p/qa-bug-triage-yoga | 15:35 |
kopecmartin | i see a decrease in open/new bugs for tempest and devstack projects, thank you for triaging | 15:35 |
kopecmartin | that's all from my side | 15:36 |
kopecmartin | thank you everyone | 15:36 |
kopecmartin | see you around | 15:36 |
soniya29 | kopecmartin, thanks :) | 15:36 |
kopecmartin | #endmeeting | 15:36 |
opendevmeet | Meeting ended Tue Mar 15 15:36:36 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:36 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/qa/2022/qa.2022-03-15-15.00.html | 15:36 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/qa/2022/qa.2022-03-15-15.00.txt | 15:36 |
opendevmeet | Log: https://meetings.opendev.org/meetings/qa/2022/qa.2022-03-15-15.00.log.html | 15:36 |
opendevreview | Artom Lifshitz proposed openstack/tempest master: DNM: Try integration tests with Python 3.9 on Focal https://review.opendev.org/c/openstack/tempest/+/833844 | 15:36 |
gmann | kopecmartin: thanks | 15:36 |
opendevreview | Soniya Murlidhar Vyas proposed openstack/tempest master: Add compute_security_client and compute_security_ group_rules_client https://review.opendev.org/c/openstack/tempest/+/833758 | 15:40 |
opendevreview | Merged openstack/devstack master: Fix tls-proxy on newer versions of openssl https://review.opendev.org/c/openstack/devstack/+/831245 | 16:11 |
*** jpena is now known as jpena|off | 17:14 | |
*** arxcruz is now known as arxcruz|off | 17:55 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!