opendevreview | Michal Arbet proposed openstack/kolla master: Add missing default policy files for debian-binary-horizon https://review.opendev.org/c/openstack/kolla/+/798212 | 05:36 |
---|---|---|
yoctozepto | morning | 05:58 |
hrw | morning | 06:19 |
hrw | https://review.opendev.org/c/openstack/kolla/+/798231 anycore? | 06:19 |
kevko | morning :) | 06:24 |
kevko | hrw: you are quick :) , thank you :) | 06:24 |
mnasiadka | morning | 06:28 |
hrw | mnasiadka: ^^ | 06:29 |
kevko | morning | 06:30 |
mnasiadka | done | 06:31 |
kevko | maybe you can check another trivial fix michal ? https://review.opendev.org/c/openstack/kolla/+/798212 << | 06:33 |
hrw | kevko: once it merge, please backport to wallaby | 06:33 |
kevko | hrw: yeah, i have eye on it | 06:33 |
kevko | hrw: could you approve also another trivial fix which i've mentioned above in message to michal ? | 06:34 |
hrw | already did | 06:35 |
hrw | kevko: and thanks for patches | 06:35 |
kevko | hrw: no problem :), i'm trying to do my best :) | 06:36 |
yoctozepto | what are you merging if CI is down... | 06:37 |
mnasiadka | just read the mail | 06:38 |
mnasiadka | so before we merge anything, we need to basically merge https://review.opendev.org/q/topic:%22ci-emergency-fix-for-zuul-4-6%22 | 06:39 |
yoctozepto | for k-a, yes | 06:39 |
yoctozepto | and then propose similar for k | 06:39 |
kevko | hmm, also reading email just now | 06:42 |
hrw | +2 on all zuul patches | 06:42 |
mnasiadka | yoctozepto: If I read Zuul's relnotes correctly - we just need to remove ansible_python_interpreter from kolla-base job definition? | 06:55 |
yoctozepto | mnasiadka: yeah, and make it work after the removal | 06:56 |
opendevreview | Radosław Piliszek proposed openstack/kolla master: [CI] Do not set ansible_python_interpreter for Zuul https://review.opendev.org/c/openstack/kolla/+/798253 | 06:58 |
opendevreview | Radosław Piliszek proposed openstack/kolla stable/wallaby: [CI] Do not set ansible_python_interpreter for Zuul https://review.opendev.org/c/openstack/kolla/+/798106 | 07:00 |
opendevreview | Radosław Piliszek proposed openstack/kolla stable/victoria: [CI] Do not set ansible_python_interpreter for Zuul https://review.opendev.org/c/openstack/kolla/+/798107 | 07:00 |
opendevreview | Radosław Piliszek proposed openstack/kolla stable/ussuri: [CI] Do not set ansible_python_interpreter for Zuul https://review.opendev.org/c/openstack/kolla/+/798108 | 07:01 |
jingvar | Morning | 07:26 |
opendevreview | Radosław Piliszek proposed openstack/kolla master: [CI] Do not set ansible_python_interpreter for Zuul https://review.opendev.org/c/openstack/kolla/+/798253 | 07:38 |
opendevreview | Radosław Piliszek proposed openstack/kolla stable/wallaby: [CI] Do not set ansible_python_interpreter for Zuul https://review.opendev.org/c/openstack/kolla/+/798106 | 07:39 |
opendevreview | Radosław Piliszek proposed openstack/kolla stable/victoria: [CI] Do not set ansible_python_interpreter for Zuul https://review.opendev.org/c/openstack/kolla/+/798107 | 07:39 |
opendevreview | Radosław Piliszek proposed openstack/kolla stable/ussuri: [CI] Do not set ansible_python_interpreter for Zuul https://review.opendev.org/c/openstack/kolla/+/798108 | 07:39 |
mnasiadka | yoctozepto: it's awesome that Zuul is just ignoring those jobs, from what I understand? | 07:56 |
yoctozepto | mnasiadka: yup, it's awesome, I agree | 08:04 |
opendevreview | Merged openstack/kolla-ansible master: [CI] Do not set ansible_python_interpreter for Zuul https://review.opendev.org/c/openstack/kolla-ansible/+/798201 | 08:13 |
opendevreview | Merged openstack/kolla-ansible stable/wallaby: [CI] Do not set ansible_python_interpreter for Zuul https://review.opendev.org/c/openstack/kolla-ansible/+/798202 | 08:13 |
opendevreview | Merged openstack/kolla-ansible stable/victoria: [CI] Do not set ansible_python_interpreter for Zuul https://review.opendev.org/c/openstack/kolla-ansible/+/798203 | 08:13 |
opendevreview | Merged openstack/kolla-ansible stable/ussuri: [CI] Do not set ansible_python_interpreter for Zuul https://review.opendev.org/c/openstack/kolla-ansible/+/798204 | 08:13 |
opendevreview | Merged openstack/kolla-ansible stable/train: [CI] Do not set ansible_python_interpreter for Zuul https://review.opendev.org/c/openstack/kolla-ansible/+/798205 | 08:13 |
yoctozepto | oh, nice, it has all merged | 08:15 |
yoctozepto | first try | 08:15 |
kevko | yoctozepto: do I need rebase patches ? probably no, right ? | 08:17 |
hrw | kevko: drop '-binary' from them and rebase? | 08:21 |
kevko | hrw: yeah ..i'm going to ..but I asked because of zuul patches | 08:21 |
hrw | stestr does not work with python 3.10 | 08:22 |
opendevreview | Michal Arbet proposed openstack/kolla master: Fix build of debian images https://review.opendev.org/c/openstack/kolla/+/798231 | 08:23 |
kevko | hrw: done | 08:23 |
hrw | let zuul finish first | 08:28 |
mgoddard | morning | 08:30 |
mgoddard | I go away for 1 day and everything breaks? | 08:30 |
mnasiadka | Isn't that the usual drill? | 08:33 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: stop using deprecated Jinja2 function https://review.opendev.org/c/openstack/kolla/+/798263 | 08:38 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: stop using deprecated Jinja2 function https://review.opendev.org/c/openstack/kolla/+/798263 | 08:43 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: stop using deprecated Jinja2 function https://review.opendev.org/c/openstack/kolla/+/798263 | 08:49 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: setDaemon() is deprecated in Python 3.10 https://review.opendev.org/c/openstack/kolla/+/798264 | 08:53 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: drop leftovers of RHEL support https://review.opendev.org/c/openstack/kolla/+/785569 | 09:04 |
hrw | rebased | 09:05 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: Debian/Ubuntu: handle APT keys in proper way https://review.opendev.org/c/openstack/kolla/+/784923 | 09:05 |
hrw | also rebased | 09:05 |
opendevreview | Radosław Piliszek proposed openstack/kolla master: [CI] Do not set ansible_python_interpreter for Zuul https://review.opendev.org/c/openstack/kolla/+/798253 | 09:29 |
opendevreview | Radosław Piliszek proposed openstack/kolla stable/wallaby: [CI] Do not set ansible_python_interpreter for Zuul https://review.opendev.org/c/openstack/kolla/+/798106 | 09:30 |
opendevreview | Radosław Piliszek proposed openstack/kolla stable/victoria: [CI] Do not set ansible_python_interpreter for Zuul https://review.opendev.org/c/openstack/kolla/+/798107 | 09:30 |
opendevreview | Radosław Piliszek proposed openstack/kolla stable/ussuri: [CI] Do not set ansible_python_interpreter for Zuul https://review.opendev.org/c/openstack/kolla/+/798108 | 09:31 |
opendevreview | Radosław Piliszek proposed openstack/kolla stable/train: [CI] Do not set ansible_python_interpreter for Zuul https://review.opendev.org/c/openstack/kolla/+/798110 | 09:32 |
opendevreview | jinyuanliu proposed openstack/kolla master: Clean up extra spaces https://review.opendev.org/c/openstack/kolla/+/798033 | 10:53 |
yoctozepto | kolla-cores: now onto merging kolla fixes: https://review.opendev.org/q/topic:%2522ci-emergency-fix-for-zuul-4-6%2522+project:openstack/kolla | 11:12 |
yoctozepto | debian is known to fail now, that's ok | 11:12 |
yoctozepto | I mean on wallaby and master | 11:12 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: Update code to Jinja 3.0 style https://review.opendev.org/c/openstack/kolla/+/798263 | 11:38 |
hrw | mgoddard: commit message updated | 11:38 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Add mariadb arbitrator to mariadb role https://review.opendev.org/c/openstack/kolla-ansible/+/780811 | 11:52 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [CI] Test Mariadb-Arbitrator with shards in the nova cells scenario https://review.opendev.org/c/openstack/kolla-ansible/+/780970 | 11:52 |
hrw | mgoddard, yoctozepto: replied to your inline comment | 11:53 |
opendevreview | Verification of a change to openstack/kolla failed: [CI] Do not set ansible_python_interpreter for Zuul https://review.opendev.org/c/openstack/kolla/+/798253 | 12:27 |
opendevreview | Merged openstack/kolla master: Clean up extra spaces https://review.opendev.org/c/openstack/kolla/+/798033 | 14:08 |
opendevreview | Merged openstack/kolla-ansible stable/wallaby: Allow user to set sysctl_net_ipv4_tcp_retries2 https://review.opendev.org/c/openstack/kolla-ansible/+/798096 | 14:24 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix horizon's healthcheck when SSL turned on https://review.opendev.org/c/openstack/kolla-ansible/+/798339 | 14:48 |
kevko | \o/ new bugfix :) | 14:51 |
opendevreview | Merged openstack/kolla master: [CI] Do not set ansible_python_interpreter for Zuul https://review.opendev.org/c/openstack/kolla/+/798253 | 15:37 |
opendevreview | Michal Arbet proposed openstack/kolla master: Fix build of debian images https://review.opendev.org/c/openstack/kolla/+/798231 | 15:47 |
opendevreview | Radosław Piliszek proposed openstack/kolla master: Fix build of debian images https://review.opendev.org/c/openstack/kolla/+/798231 | 16:50 |
kevko | yoctozepto: still here ? | 17:07 |
yoctozepto | kevko: nope | 17:43 |
kevko | yoctozepto: hmm, ok | 17:43 |
opendevreview | Merged openstack/kolla master: Fix build of debian images https://review.opendev.org/c/openstack/kolla/+/798231 | 20:06 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!