| frickler | mnasiadka: slurp upgrade jobs are still failing, maybe they don't pick up the dependency properly? | 08:06 |
|---|---|---|
| opendevreview | Michael Still proposed openstack/kolla-ansible master: Fix docs rendering errors. https://review.opendev.org/c/openstack/kolla-ansible/+/944184 | 08:09 |
| mnasiadka | frickler: I don't know if slurp is failing because of that? | 08:12 |
| mnasiadka | but maybe dependency does not work, as you say | 08:12 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Fix docs rendering errors. https://review.opendev.org/c/openstack/kolla-ansible/+/944184 | 08:30 |
| mnasiadka | frickler: ^^ updated that patch and +2 - can you have a look and let's merge it? | 08:31 |
| frickler | mnasiadka: the slurp jobs were green before friday, and the failure looks the same | 08:32 |
| mnasiadka | so might be we're not building the images on upgrade target | 08:34 |
| frickler | mnasiadka: we need to get the docs job fix merged first https://review.opendev.org/c/openstack/kolla-ansible/+/944632/1 , either force-merge or wait for the neutron fix | 08:34 |
| mnasiadka | frickler: Actually I found https://docutils.sourceforge.io/docs/ref/rst/restructuredtext.html#bullet-lists - so dashes are a legit bullet list elements | 08:34 |
| frickler | or maybe we want a doc job fix that doesn't use bindep.txt? | 08:36 |
| mnasiadka | if that's an option - why not | 08:45 |
| mnasiadka | frickler: it seems that this job passes | 08:46 |
| frickler | mnasiadka: which job exactly? | 08:52 |
| mnasiadka | frickler: the docs job in 944184? | 08:52 |
| frickler | mnasiadka: yes, because I rebased it on top of the docs fix | 08:53 |
| mnasiadka | ah ok | 08:53 |
| mnasiadka | so how do we fix docs without bindep pcre entries? | 08:53 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix Redis Sentinel authentication for octavia's jobboard HA https://review.opendev.org/c/openstack/kolla-ansible/+/942799 | 11:49 |
| opendevreview | Pierre Riteau proposed openstack/kolla-ansible master: CI: Fix documentation jobs https://review.opendev.org/c/openstack/kolla-ansible/+/944632 | 12:45 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: Bump MichaelRigart.interfaces to v1.15.4 https://review.opendev.org/c/openstack/kayobe/+/943350 | 13:00 |
| opendevreview | Matt Anson proposed openstack/kolla master: Add support for aarch64 ipxe to ironic-pxe image https://review.opendev.org/c/openstack/kolla/+/944869 | 13:23 |
| opendevreview | Matt Anson proposed openstack/kolla master: Add support for aarch64 ipxe to ironic-pxe image https://review.opendev.org/c/openstack/kolla/+/944869 | 13:24 |
| yuval_ | Guys what do we do with neutron issue? and steps we can do to solve it? | 14:07 |
| yuval_ | CI is currently broken | 14:07 |
| priteau | The Neutron fix is approved now, just need to wait for check + gate | 15:12 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Fix docs rendering errors. https://review.opendev.org/c/openstack/kolla-ansible/+/944184 | 15:22 |
| frickler | ok, so 5-6h plus potentially some rechecks needed :-/ | 15:26 |
| priteau | And recheck will be needed (already one job failed) :( | 15:27 |
| priteau | Well, CI fixed by tomorrow morning hopefully | 15:28 |
| mnasiadka | frickler: since we use bindep.txt only for docs in https://review.opendev.org/c/openstack/kolla-ansible/+/944632 - should we just add it to irrelevant files for k-a jobs? | 15:41 |
| frickler | mnasiadka: maybe? but then we'll likely miss it if we ever do use bindep elsewhere. or would adding a big comment in bindep.txt itself be enough? | 15:46 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Fix documentation jobs https://review.opendev.org/c/openstack/kolla-ansible/+/944632 | 15:47 |
| mnasiadka | Yeah, just did that. | 15:48 |
| mnasiadka | since we use a-c-k to install deps - we probably shouldn't use bindep for anything else | 15:48 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Fix documentation jobs https://review.opendev.org/c/openstack/kolla-ansible/+/944632 | 15:58 |
| frickler | mnasiadka: why did you drop the "^ $" part? also I fear changing the job definition still causes all children of it to be ran in this particular change :( | 16:05 |
| mnasiadka | frickler: irrelevant-files are not working, wonder why | 16:05 |
| mnasiadka | ah right | 16:05 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Fix documentation jobs https://review.opendev.org/c/openstack/kolla-ansible/+/944632 | 16:05 |
| mnasiadka | Still it makes sense to add it to irrelevant-files | 16:06 |
| mnasiadka | let's wait for Neutron merge and then new images tomorrow | 16:06 |
| frickler | yes, it just doesn't help us in this particular case | 16:06 |
| frickler | tomorrow or whenever zuul sees fit :-/ | 16:06 |
| mnasiadka | well, I would ask why devstack works, but that's probably for another day :) | 16:09 |
| frickler | mnasiadka: I've asked the same question in the neutron meeting earlier ;) no idea so far | 16:09 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Fix documentation jobs https://review.opendev.org/c/openstack/kolla-ansible/+/944632 | 16:21 |
| priteau | Neutron fix now in gate | 17:48 |
| mnasiadka | if it doesn't fail in gate :) | 17:48 |
| mnasiadka | But fingers crossed we're back in business tomorrow | 17:49 |
| frickler | yay, merged | 20:14 |
| priteau | \o/ | 20:18 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!