| opendevreview | Tim Burke proposed openstack/tempest master: Stop sending underscores in header names to Swift https://review.opendev.org/c/openstack/tempest/+/937416 | 00:44 |
|---|---|---|
| *** mhen_ is now known as mhen | 03:02 | |
| *** ralonsoh_ is now known as ralonsoh | 09:21 | |
| opendevreview | Liron Kuchlani proposed openstack/tempest master: Add support for specifying fixed IP address in floating IP association https://review.opendev.org/c/openstack/tempest/+/937406 | 13:15 |
| opendevreview | Liron Kuchlani proposed openstack/tempest master: Add support for specifying fixed IP address in floating IP association https://review.opendev.org/c/openstack/tempest/+/937406 | 14:04 |
| kopecmartin | #startmeeting qa | 15:00 |
| opendevmeet | Meeting started Tue Dec 10 15:00:34 2024 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#Agenda_for_next_Office_hours | 15:00 |
| kopecmartin | agenda ^ | 15:00 |
| frickler | o/ | 15:00 |
| kopecmartin | \o | 15:02 |
| gmann | o/ | 15:02 |
| opendevreview | Nathan proposed openstack/tempest master: Fixed encryption type methods to comply with documentation https://review.opendev.org/c/openstack/tempest/+/935526 | 15:02 |
| kopecmartin | hey gmann isn't it too early for you? | 15:02 |
| lpiwowar | o/ | 15:03 |
| gmann | kopecmartin: I am attending board meeting today so can join here too :) | 15:03 |
| kopecmartin | :) | 15:03 |
| kopecmartin | #topic Announcements and Action Items | 15:03 |
| kopecmartin | this is the last office hour before the holidays which means the last office hour this year | 15:04 |
| nperez | o/ | 15:04 |
| kopecmartin | as I mentioned last time, i won't be able to host these in the new year, so we'll need a new volunteer :) | 15:04 |
| kopecmartin | hi nperez | 15:04 |
| kopecmartin | one of the discussed options was that gmann would take over, however, in that case we'd shift the meeting for a later time to better accomodate his time zone | 15:05 |
| kopecmartin | if that happens, it'll be shared on the ML as well | 15:05 |
| kopecmartin | oh, one other update, i've got a volunteer for reviews (as my capacity for this kind of thing in the new year will be decreased) | 15:07 |
| kopecmartin | tosky volunteered that he would like to help with reviews on tempest side | 15:08 |
| kopecmartin | you may have seen his commits to cinder-tempest-plugin and a few related projects, e.g. cinder, glance and even tempest | 15:08 |
| tosky | even though I don't know a lot of the internal bits, so I may not always be perfectly helpful | 15:09 |
| tosky | (I do have some experience on devstack and grenade, though, to be refreshed a bit) | 15:09 |
| kopecmartin | that is absolutely ok .. i'm sure tempest can benefit from that | 15:10 |
| frickler | cool, yes, I remember you were around when I started digging into devstack | 15:10 |
| gmann | perfect thanks tosky | 15:10 |
| kopecmartin | luigi is everywhere ;) | 15:10 |
| kopecmartin | https://review.opendev.org/q/from:ltoscano@redhat.com | 15:10 |
| kopecmartin | #action so, one action item for me is to hand over the hosting of this office hour and find (if needed) a new time | 15:12 |
| kopecmartin | that's all from announcements | 15:13 |
| kopecmartin | #topic Epoxy Priority Items progress | 15:13 |
| kopecmartin | #link https://etherpad.opendev.org/p/qa-epoxy-priority | 15:13 |
| * kopecmartin checking if there are any updates | 15:13 | |
| * kopecmartin feeling a bit ashamed as he didn't get to a few action items there :/ | 15:14 | |
| kopecmartin | the migration to noble went more or less fine | 15:15 |
| kopecmartin | the email thread: | 15:15 |
| kopecmartin | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/JOMDY26TCW7OX3NXRGOYQCIDXNNJ4E25/ | 15:15 |
| frickler | well still some cleanup happening, but nothing major that I'm aware of, yes | 15:15 |
| gmann | one of the bug is in devstack-plugin-container CRI-O installation on noble failing | 15:15 |
| kopecmartin | only 24 patches left open out of almost a hundred | 15:16 |
| kopecmartin | #link https://review.opendev.org/q/topic:%22migrate-to-noble%22+status:open | 15:16 |
| gmann | I pinged hongbin who is mainly maintaining this plugin and he will look into that | 15:16 |
| kopecmartin | great | 15:16 |
| gmann | yeah, I am trying to get those merged but they may take time | 15:16 |
| gmann | CRI-O thing only impact k8s jobs | 15:17 |
| gmann | and the doc job migration is still left | 15:18 |
| kopecmartin | ack , thanks | 15:18 |
| kopecmartin | let's move on | 15:19 |
| kopecmartin | #topic Gate Status Checks | 15:19 |
| 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) | 15:19 |
| kopecmartin | 2 patches, both approved | 15:19 |
| kopecmartin | #topic Sub Teams highlights | 15:19 |
| kopecmartin | Changes with Review-Priority == +1 | 15:19 |
| 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) | 15:19 |
| kopecmartin | 6 patches, most of them approved | 15:20 |
| kopecmartin | i need a second opinion on | 15:20 |
| kopecmartin | #link https://review.opendev.org/c/openstack/tempest/+/933308 | 15:20 |
| kopecmartin | and | 15:20 |
| kopecmartin | #link https://review.opendev.org/c/openstack/tempest/+/935526 | 15:20 |
| nperez | hey you wanted me to fix the "closes bug" part of my commit message and fixed it | 15:21 |
| kopecmartin | perfect, thank you | 15:21 |
| nperez | https://review.opendev.org/c/openstack/tempest/+/935526 | 15:21 |
| nperez | thank you for reviewing it | 15:21 |
| kopecmartin | thank you for working on it | 15:22 |
| kopecmartin | i reviewed it, let's see what the automatic jobs think | 15:23 |
| kopecmartin | #topic Periodic jobs Status Checks | 15:23 |
| kopecmartin | Periodic stable full: https://zuul.openstack.org/builds?pipeline=periodic-stable&job_name=tempest-full-2023-1&job_name=tempest-full-2023-2&job_name=tempest-full-2024-1&job_name=tempest-full-2024-2 | 15:23 |
| kopecmartin | Periodic stable slow: https://zuul.openstack.org/builds?job_name=tempest-slow-2024-2&job_name=tempest-slow-2024-1&job_name=tempest-slow-2023-2&job_name=tempest-slow-2023-1 | 15:23 |
| kopecmartin | Periodic extra tests: https://zuul.openstack.org/builds?job_name=tempest-full-2024-2-extra-tests&job_name=tempest-full-2024-1-extra-tests&job_name=tempest-full-2023-2-extra-tests&job_name=tempest-full-2023-1-extra-tests | 15:23 |
| kopecmartin | Periodic master: https://zuul.openstack.org/builds?project=openstack%2Ftempest&project=openstack%2Fdevstack&pipeline=periodic | 15:23 |
| frickler | you can drop 2023.1 from those searches now | 15:24 |
| frickler | that moved to unmaintained | 15:24 |
| kopecmartin | oh, right, thanks frickler | 15:24 |
| kopecmartin | all builds looks pretty standard | 15:26 |
| kopecmartin | #topic Distros check | 15:27 |
| kopecmartin | Centos 9: https://zuul.openstack.org/builds?job_name=tempest-full-centos-9-stream&job_name=devstack-platform-centos-9-stream&skip=0 | 15:27 |
| kopecmartin | Debian: https://zuul.openstack.org/builds?job_name=devstack-platform-debian-bookworm&skip=0 | 15:27 |
| kopecmartin | Rocky: https://zuul.openstack.org/builds?job_name=devstack-platform-rocky-blue-onyx | 15:27 |
| kopecmartin | Jammy: https://zuul.opendev.org/t/openstack/builds?job_name=devstack-platform-ubuntu-jammy-ovn-source&job_name=devstack-platform-ubuntu-jammy-ovs&skip=0 | 15:27 |
| kopecmartin | Noble: https://zuul.opendev.org/t/openstack/builds?job_name=devstack-platform-ubuntu-noble&skip=0 | 15:27 |
| frickler | the latter two also need updates now I think. more noble less jammy | 15:27 |
| kopecmartin | oh, right | 15:29 |
| kopecmartin | hmm, the search filters based on the name only | 15:29 |
| kopecmartin | .. on the other hand, it might be good to see the results on the unmaintained branch | 15:30 |
| kopecmartin | hmm, but if there is something wrong, we can't fix that anyway | 15:30 |
| kopecmartin | I'll try to limit the search | 15:30 |
| frickler | yes, but devstack-platform-ubuntu-jammy-ovn-source was replaced by devstack-platform-ubuntu-noble-ovn-source for example on master | 15:30 |
| kopecmartin | right .. ok, another action item then | 15:31 |
| frickler | and devstack-platform-ubuntu-noble is replaced by tempest-full-py3 ;) | 15:31 |
| kopecmartin | #action kopecmartin to update the above searches | 15:31 |
| frickler | instead there is devstack-platform-ubuntu-jammy now. or so I think | 15:32 |
| frickler | anyway, we can go on I think | 15:33 |
| kopecmartin | ack | 15:33 |
| kopecmartin | #topic Open Discussion | 15:33 |
| kopecmartin | anything for the open discussion? | 15:33 |
| clarkb | I wanted to call out https://bugs.launchpad.net/devstack/+bug/2082617 and less the specific issue and more the thing that made it difficult to debug | 15:34 |
| clarkb | as noted in there I think the |default(omit) default module arguments may mask early detection of failures to render the config dicts | 15:34 |
| clarkb | it would probably be a good idea for the qa team to investigate refactoring that in devstack so that we're less likely to pass a null default and wonder why the job isn't working as expected | 15:35 |
| clarkb | The good news is I don't think jobs would pass if they have broken jinja in their localrc config so I don't think we have a bunch of jobs doing the wrong thing | 15:36 |
| clarkb | but debugging is difficult as is | 15:36 |
| kopecmartin | that's a good point | 15:37 |
| frickler | ah, yes, that was a fancy issue. I just think the "qa team" is essentially near zero by now, so not sure who would take that up. except tosky maybe? :) | 15:38 |
| kopecmartin | i'll check it | 15:39 |
| clarkb | ya its an easy one to fix at a superficial level. Just update that CEPH ip address to the private ipv4 address and move on. My concern is more that debugging future issues may be more painful/ less obvious | 15:39 |
| clarkb | and I suspect changing how defaults are handled may expose the issue more directly with a "jinja broke" error | 15:40 |
| kopecmartin | right, that;s a good idea | 15:41 |
| kopecmartin | cool, anything else/ | 15:42 |
| kopecmartin | ? | 15:42 |
| kopecmartin | ok then, we're done | 15:43 |
| kopecmartin | thank you all | 15:43 |
| kopecmartin | see you around | 15:43 |
| frickler | o/ | 15:44 |
| kopecmartin | #endmeeting | 15:44 |
| opendevmeet | Meeting ended Tue Dec 10 15:44:04 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:44 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/qa/2024/qa.2024-12-10-15.00.html | 15:44 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/qa/2024/qa.2024-12-10-15.00.txt | 15:44 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/qa/2024/qa.2024-12-10-15.00.log.html | 15:44 |
| opendevreview | Emilien Macchi proposed openstack/devstack master: ovn_agent: start OVS processes with retries https://review.opendev.org/c/openstack/devstack/+/937462 | 15:58 |
| opendevreview | Merged openstack/devstack stable/2023.2: [stable-only] Drop Neutron grenade jobs in the CI https://review.opendev.org/c/openstack/devstack/+/936126 | 16:21 |
| frickler | kopecmartin: gmann: ^^ forgot I wanted to discuss how to disable grenade jobs with unmaintained branches as base. or do we just ignore that? | 16:38 |
| opendevreview | Merged openstack/devstack stable/2024.2: Install setuptools 'core' extra https://review.opendev.org/c/openstack/devstack/+/937203 | 17:08 |
| opendevreview | Merged openstack/devstack master: lib/placement: Migrate placement to WSGI module path https://review.opendev.org/c/openstack/devstack/+/919581 | 17:08 |
| elodilles | note that the jobs should work now. i know that we don't need to support the grenade jobs from an unmaintained branch, but let's not drop the definition globally, rather let the teams decide if they want to throw them away (or delete them when it is blocking their stable gates). this is maybe useful especially for 2023.1 Antelope, which is now unmaintained, but was the 1st official base for Skip | 17:09 |
| elodilles | Level Upgrade. | 17:09 |
| opendevreview | Merged openstack/devstack master: lib/cinder: Migrate cinder to WSGI module path https://review.opendev.org/c/openstack/devstack/+/902905 | 17:19 |
| gmann | frickler: elodilles yeah, as long as they work it is fine to run but if broken then I will suggest to drop unless unmaintained maintainers fix it and keep green. Since starting, we said upgrade testing guarantee is only on supported stable branches only. | 17:45 |
| elodilles | gmann: +1 | 17:49 |
| frickler | gmann: well the problem is that as long as the defition is made in the global template, projects cannot opt out except by dropping the integrated-gate project-template completely | 17:59 |
| gmann | frickler: I see, I think we should keep/maintained global template for unmaintained branches somewhere else. otherwise we need to maintained unmaintained templates in tempest and then release a new tempest tag to be used for unmaintained one (which we do but do not separate out the template for unmaitained) | 18:02 |
| gmann | frickler: or easy way is to stop using the global template in unmaintained and run jobs explicitly when they need to do | 18:02 |
| gmann | our requirement for unmaintained branches is to run at least one integrated testing job | 18:03 |
| gmann | I really want to avoid injecting the new things for unmaintained in QA tooling and we end up being responsible for those. | 18:04 |
| frickler | gmann: well this isn't about unmaintained branches directly, it is about grenade forced to run on 2023.2 and 2024.1, upgrading from unmaintained/2023.1 | 18:14 |
| frickler | (grenade-skip-level for the latter branch) | 18:15 |
| opendevreview | Merged openstack/tempest master: For test VM validation resource were added. https://review.opendev.org/c/openstack/tempest/+/933308 | 18:32 |
| gmann | frickler: ohk, so this one basically https://github.com/openstack/tempest/blob/master/zuul.d/integrated-gate.yaml#L633 | 18:44 |
| gmann | I will say, let it be there as long as it is green and if fail then we can stop running it on oldest supported stable branch. same way we did for EM | 18:46 |
| *** atmark_ is now known as atmark | 19:37 | |
| opendevreview | Tim Burke proposed openstack/tempest master: Stop sending underscores in header names to Swift https://review.opendev.org/c/openstack/tempest/+/937416 | 21:51 |
| opendevreview | James Parker proposed openstack/whitebox-tempest-plugin master: Test live migration after attach/detach port https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/937487 | 22:05 |
| JayF | I think https://review.opendev.org/c/openstack/devstack/+/934909 is a positive change -- given we don't actually ensure virtualenv package is installed, and the module is generally preferred now | 22:43 |
| JayF | please review when you have time | 22:43 |
| clarkb | looks like GLOBAL_VENV does use the venv module and not virtualenv | 22:52 |
| clarkb | so it helps align those two features. That said I wonder if we need the two different virtualenv setups? JayF out of curiousity do you have a specific use case for using venv per project? | 22:53 |
| JayF | I have a patch for ironic fixing USE_VENV, which I was going to try and use to go down a rabbithole of trying to make Ironic's BMC-emulators not need to be global-requirements compatible (by getting Ironic, and them, in their own independent venvs) | 22:54 |
| JayF | but at this point, realistically, it's there, and it should work or be removed | 22:54 |
| JayF | making it work is 100x less work than getting confidence it's unused | 22:54 |
| clarkb | sure. I only ask beacuse its been there for liek 10 years and I'm pretty sure it hasn't ever worked until you made it work :) | 22:54 |
| JayF | heh | 22:55 |
| clarkb | but also the use case of installing python things that don't use openstack requirements is a reasonable one | 22:55 |
| JayF | I have had a habit the last few months of running full speed into ancient brick walls, this wouldn't be the first time :D | 22:55 |
| clarkb | though specifically it was the functionality of running every service in its own venv that didn't work iirc. The underlying code tomanage various venvs probably was fine | 22:57 |
| clarkb | I looked into that when osc was slow because reducing the number of pacakges that osc saw would speed up entrypoint loading | 22:58 |
| JayF | the code is fine /if `virtualenv` exists on the path/ :D | 23:12 |
| JayF | which my change fixes | 23:12 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!