frickler | mnasiadka: kevko: looks like everything is still failing, we'll either need to squash fixes or disable even more tests temporarily, wdyt? | 06:27 |
---|---|---|
kevko | frickler: well, it's because there are several jobs blocking .. build an mirrors - https://review.opendev.org/c/openstack/kolla/+/951329 , mariadb job for backup - https://review.opendev.org/c/openstack/kolla/+/950948 , debian-upgrade - proxysql issue ( working on it ) | 07:18 |
kevko | frickler: I suggest temporarily marking the Debian upgrade as non-voting … merge everything (since Ubuntu passes – we know it's OK), and then I’ll handle the Debian upgrade afterward… There are several issues and I’m not sure if I can get all of them to pass at once… | 07:19 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [CI] Temporarily make kolla-ansible-debian-upgrade non-voting https://review.opendev.org/c/openstack/kolla-ansible/+/951368 | 07:22 |
kevko | frickler: ^^ | 07:22 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [CI] Temporarily make kolla-ansible-debian-upgrade non-voting https://review.opendev.org/c/openstack/kolla-ansible/+/951368 | 07:23 |
frickler | kevko: that doesn't work with the depends-on, because that creates a cycle. let's do 951368 without the dep and I'll force merge it. you should drop the job from the gate pipeline, though, no need to run a n-v job there | 07:29 |
kevko | frickler: but https://review.opendev.org/c/openstack/kolla-ansible/+/951368 without Depends-On will fail on mariadb job, won't it ? | 07:32 |
kevko | frickler: OR, I can make non-voting also mariadb jobs ...without depends on | 07:33 |
kevko | frickler: wdyt | 07:33 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [CI] Temporarily make kolla-ansible-debian-upgrade non-voting https://review.opendev.org/c/openstack/kolla-ansible/+/951368 | 07:34 |
kevko | frickler: depends-on removed ...make mariadb also non-voting | 07:35 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [CI] Temporarily make kolla-ansible-debian-upgrade non-voting https://review.opendev.org/c/openstack/kolla-ansible/+/951368 | 07:36 |
kevko | frickler: ^^ check if OK | 07:36 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [CI] Temporarily make kolla-ansible-debian-upgrade non-voting https://review.opendev.org/c/openstack/kolla-ansible/+/951368 | 07:39 |
kevko | frickler: remove from gate ... n-v reverted as is ... | 07:39 |
mnasiadka | non-voting jobs should not be in gate queue | 07:51 |
mnasiadka | Remove it from there | 07:51 |
frickler | we'll still need the n-v part for the kolla repo I think | 07:55 |
mnasiadka | so, let me get this straight - kolla-ansible is broken because of debian-upgrade only - so that shouldn't need the kolla patch | 08:05 |
mnasiadka | but yes, we'll need to set n-v in the kolla repo as well | 08:05 |
opendevreview | Merged openstack/kayobe master: CI: Bump ansible-lint and prune skip list https://review.opendev.org/c/openstack/kayobe/+/951204 | 09:16 |
kevko | As if on purpose, the Debian upgrade went through this time, but it usually doesn't. haha | 09:51 |
opendevreview | Joachim de Groot proposed openstack/kolla-ansible stable/2024.2: Implement neutron-ovn-vpn-agent https://review.opendev.org/c/openstack/kolla-ansible/+/951389 | 10:00 |
opendevreview | Joachim de Groot proposed openstack/kolla-ansible master: Implement neutron-ovn-vpn-agent https://review.opendev.org/c/openstack/kolla-ansible/+/924575 | 10:07 |
frickler | well in the current status 951368 isn't needed, because all jobs are green. if they weren't green, the patch would get V-1 and need force-merging. so let's recheck kolla patches and see if they pass now, too? | 10:22 |
kevko | frickler: better to do a recheck | 10:26 |
frickler | done. any idea what changed from yesterday? | 10:35 |
opendevreview | Joachim de Groot proposed openstack/kolla master: Fix glance-api: add missing lsscsi package https://review.opendev.org/c/openstack/kolla/+/951090 | 10:51 |
opendevreview | Joachim de Groot proposed openstack/kolla master: Fix glance-api: add missing lsscsi package https://review.opendev.org/c/openstack/kolla/+/951090 | 10:56 |
opendevreview | Merged openstack/kayobe stable/2024.1: Fix syntax error in drac-pxe role https://review.opendev.org/c/openstack/kayobe/+/951122 | 11:37 |
opendevreview | Merged openstack/kayobe stable/2024.2: Fix syntax error in drac-pxe role https://review.opendev.org/c/openstack/kayobe/+/951120 | 11:37 |
tafkamax | https://status.canonical.com/ | 11:49 |
tafkamax | I've seen reports of people complaining on the internet regarding the ubuntu/canonical repos/mirrors | 11:49 |
opendevreview | Merged openstack/kayobe stable/2025.1: Fix syntax error in drac-pxe role https://review.opendev.org/c/openstack/kayobe/+/951118 | 12:12 |
opendevreview | Merged openstack/kayobe stable/2024.2: Use 'pip_upper_constraints_file' for openstacksdk installation https://review.opendev.org/c/openstack/kayobe/+/950313 | 12:12 |
opendevreview | Merged openstack/kayobe stable/2024.1: Use 'pip_upper_constraints_file' for openstacksdk installation https://review.opendev.org/c/openstack/kayobe/+/950362 | 12:12 |
kevko | regarding proxysql and mariadb -> It will be a bit of a revolution, but it's going to be great :) | 12:58 |
kevko | i am in half maybe | 12:58 |
sahid | iiq:q | 13:23 |
sahid | oops sorry | 13:23 |
jaydg | I'm working on a test case for 924575: Implement neutron-ovn-vpn-agent | https://review.opendev.org/c/openstack/kolla-ansible/+/924575 | 13:53 |
jaydg | My first thought is to add another function to tests/test-ovn.sh. Does that make sense or should I start a new file? | 13:56 |
opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: Improve backend TLS certificate and key lookup efficiency https://review.opendev.org/c/openstack/kolla-ansible/+/951414 | 14:14 |
kevko | frickler you see ? Failed job for debian-upgrade ...it's passing very from time to time ...I will fix by another patch which is wip | 14:14 |
kevko | But now can we merge ? | 14:14 |
opendevreview | Joachim de Groot proposed openstack/kolla-ansible master: WIP: VPNaaS tests https://review.opendev.org/c/openstack/kolla-ansible/+/951420 | 14:50 |
opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: Improve backend TLS certificate and key lookup efficiency https://review.opendev.org/c/openstack/kolla-ansible/+/951414 | 15:10 |
opendevreview | Michal Arbet proposed openstack/kolla master: [mariadb] Fix incremental backup using correct base dir https://review.opendev.org/c/openstack/kolla/+/950948 | 16:08 |
opendevreview | Michal Arbet proposed openstack/kolla master: [base] Ensure full control over APT sources for Ubuntu https://review.opendev.org/c/openstack/kolla/+/951329 | 16:08 |
opendevreview | Michal Arbet proposed openstack/kolla master: debian: Use Epoxy extrepo https://review.opendev.org/c/openstack/kolla/+/949752 | 16:08 |
opendevreview | Michal Arbet proposed openstack/kolla master: debian: Use bookworkm suite for rabbitmq APT repo https://review.opendev.org/c/openstack/kolla/+/950531 | 16:08 |
opendevreview | Michal Arbet proposed openstack/kolla master: Bump proxysql to 3.0.x https://review.opendev.org/c/openstack/kolla/+/951347 | 16:08 |
opendevreview | Michal Arbet proposed openstack/kolla master: Update OpenSearch and OpenSearch Dashboards to version 3.x https://review.opendev.org/c/openstack/kolla/+/949444 | 16:08 |
opendevreview | Seunghun Lee proposed openstack/kayobe master: Make custom Gather facts run with default python for kayobe-target-venv https://review.opendev.org/c/openstack/kayobe/+/951425 | 16:13 |
opendevreview | Seunghun Lee proposed openstack/kayobe master: Make custom Gather facts run with default python for kayobe-target-venv https://review.opendev.org/c/openstack/kayobe/+/951425 | 16:14 |
frickler | kevko: what exactly do you want to merge while CI is red? let me do a patch to n-v failing stuff in kolla, then maybe we can proceed | 17:32 |
opendevreview | Dr. Jens Harbott proposed openstack/kolla master: zuul: make failing job temporarily non-voting https://review.opendev.org/c/openstack/kolla/+/951434 | 18:37 |
frickler | mnasiadka: kevko: still around to check ^^? I can also self-approve otherwise since we essentially discussed this | 18:38 |
mnasiadka | frickler: done | 18:39 |
frickler | thx | 18:40 |
opendevreview | Merged openstack/kolla master: zuul: make failing job temporarily non-voting https://review.opendev.org/c/openstack/kolla/+/951434 | 19:56 |
*** darmach9 is now known as darmach | 21:05 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!