opendevreview | Michal Nasiadka proposed openstack/kolla master: mariadb: Bump to 11.4 https://review.opendev.org/c/openstack/kolla/+/926981 | 06:37 |
---|---|---|
opendevreview | Michal Nasiadka proposed openstack/kolla master: mariadb: Bump to 11.4 https://review.opendev.org/c/openstack/kolla/+/926981 | 06:39 |
opendevreview | Seunghun Lee proposed openstack/kolla master: Fix symlinking of openstack-base source https://review.opendev.org/c/openstack/kolla/+/924759 | 06:45 |
opendevreview | Seunghun Lee proposed openstack/kolla master: Fix symlinking of openstack-base source https://review.opendev.org/c/openstack/kolla/+/924759 | 06:51 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Revert "[release] Change RDO's DELOREAN_DEPS repo to Caracal" https://review.opendev.org/c/openstack/kolla/+/926982 | 07:00 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: ubuntu: switch to Dalmatian UCA https://review.opendev.org/c/openstack/kolla/+/926983 | 07:01 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Switch to Ubuntu 24.04 LTS https://review.opendev.org/c/openstack/kolla/+/907589 | 07:04 |
opendevreview | Pierre Riteau proposed openstack/kayobe master: [DNM] Test seed-vm job with Rocky Linux image https://review.opendev.org/c/openstack/kayobe/+/926984 | 07:07 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: mariadb: Bump to 11.4 https://review.opendev.org/c/openstack/kolla/+/926981 | 07:13 |
mnasiadka | frickler, bbezak: willing to have a look on https://review.opendev.org/c/openstack/kolla/+/926541 https://review.opendev.org/c/openstack/kolla/+/926542 and https://review.opendev.org/c/openstack/kolla/+/924246 ? these are mostly simple Ubuntu Noble pre-reqs | 07:15 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: mariadb: Add lookup_cluster in post-deploy https://review.opendev.org/c/openstack/kolla-ansible/+/926987 | 07:31 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: [WIP] Enable ML2/OVN and distributed FIP by default. https://review.opendev.org/c/openstack/kolla-ansible/+/904959 | 07:37 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: nova: drop dmidecode https://review.opendev.org/c/openstack/kolla/+/926542 | 07:49 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: bifrost: bump Ansible version to 8/9 (rpm) and 9/10 (deb) https://review.opendev.org/c/openstack/kolla/+/924246 | 07:52 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: bifrost: bump Ansible version to 8 (rpm) and 9 (deb) https://review.opendev.org/c/openstack/kolla/+/924246 | 07:52 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: mariadb: Bump to 11.4 https://review.opendev.org/c/openstack/kolla/+/926981 | 08:16 |
opendevreview | Matt Crees proposed openstack/kolla-ansible stable/2024.1: CI: Only migrate RMQ queues during SLURP https://review.opendev.org/c/openstack/kolla-ansible/+/926129 | 08:34 |
opendevreview | Matt Crees proposed openstack/kolla-ansible master: Add a precheck to catch RMQ SLURP upgrades https://review.opendev.org/c/openstack/kolla-ansible/+/926146 | 08:37 |
opendevreview | Matt Crees proposed openstack/kolla-ansible stable/2024.1: Add command to upgrade to a target version of RMQ https://review.opendev.org/c/openstack/kolla-ansible/+/926997 | 08:39 |
opendevreview | Matt Crees proposed openstack/kolla-ansible stable/2023.2: Add command to upgrade to a target version of RMQ https://review.opendev.org/c/openstack/kolla-ansible/+/926998 | 08:39 |
opendevreview | Matt Crees proposed openstack/kolla-ansible stable/2023.1: Add command to upgrade to a target version of RMQ https://review.opendev.org/c/openstack/kolla-ansible/+/918976 | 08:40 |
kevko | \o | 08:43 |
opendevreview | Merged openstack/kayobe master: CI: Return to openstack-python3-jobs, fix pep8 failures https://review.opendev.org/c/openstack/kayobe/+/926619 | 08:57 |
opendevreview | Bartosz Bezak proposed openstack/kolla master: Drop prometheus-msteams container images https://review.opendev.org/c/openstack/kolla/+/927000 | 09:10 |
opendevreview | Bartosz Bezak proposed openstack/kolla master: Drop prometheus-msteams container images https://review.opendev.org/c/openstack/kolla/+/927000 | 09:17 |
opendevreview | Bartosz Bezak proposed openstack/kolla-ansible master: Drop prometheus-msteams support https://review.opendev.org/c/openstack/kolla-ansible/+/927001 | 09:25 |
opendevreview | Pierre Riteau proposed openstack/kayobe stable/2024.1: CI: Add openstack-tox-pep8 job, fix pep8 failures https://review.opendev.org/c/openstack/kayobe/+/927002 | 09:34 |
opendevreview | Verification of a change to openstack/kolla master failed: nova: Drop xen remnants https://review.opendev.org/c/openstack/kolla/+/926541 | 09:37 |
frickler | meh, what now, gate? | 09:41 |
frickler | 504 gateway timeout from galaxy, didn't we add retries for those? maybe not in all relevant locations I fear https://zuul.opendev.org/t/openstack/build/c367bcb1415645df85b6b6692c75fbcc | 09:42 |
frickler | mnasiadka: can you check the failure at the end of https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_d29/926542/2/check/kolla-ansible-debian/d295fba/primary/logs/ansible/reconfigure ? doesn't look related to your patch, but maybe triggered by some new deprecation warnings? | 09:46 |
opendevreview | Merged openstack/kolla-ansible stable/2024.1: CI: Only migrate RMQ queues during SLURP https://review.opendev.org/c/openstack/kolla-ansible/+/926129 | 12:11 |
opendevreview | Verification of a change to openstack/kolla master failed: nova: Drop xen remnants https://review.opendev.org/c/openstack/kolla/+/926541 | 12:48 |
opendevreview | Pierre Riteau proposed openstack/kayobe master: seed-vm/infra-vms: Add support for EFI and q35 https://review.opendev.org/c/openstack/kayobe/+/921660 | 12:53 |
opendevreview | Pierre Riteau proposed openstack/kayobe master: seed-vm/infra-vms: Use efi as default https://review.opendev.org/c/openstack/kayobe/+/927015 | 12:53 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible master: CI: validate prometheus configs https://review.opendev.org/c/openstack/kolla-ansible/+/926209 | 12:53 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible master: CI: validate prometheus configs https://review.opendev.org/c/openstack/kolla-ansible/+/926209 | 13:00 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible master: CI: validate prometheus configs https://review.opendev.org/c/openstack/kolla-ansible/+/926209 | 13:09 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/2024.1: external/internal vip: use proper ip comparison https://review.opendev.org/c/openstack/kolla-ansible/+/927018 | 13:11 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/2023.2: external/internal vip: use proper ip comparison https://review.opendev.org/c/openstack/kolla-ansible/+/927019 | 13:11 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/2023.1: external/internal vip: use proper ip comparison https://review.opendev.org/c/openstack/kolla-ansible/+/927020 | 13:12 |
SvenKieske | I think I have seen this error before, do we need to add a new release or something to the docs? kolla-ansible/releasenotes/source/unreleased.rst:: ERROR: Anonymous hyperlink mismatch: 1 references but 0 targets. See "backrefs" attribute for IDs. | 13:30 |
SvenKieske | ah nvm, missed a space | 13:31 |
opendevreview | Simon Dodsley proposed openstack/kolla-ansible master: Add clarity to user guide for different Pure options https://review.opendev.org/c/openstack/kolla-ansible/+/927021 | 13:32 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible master: CI: validate prometheus configs https://review.opendev.org/c/openstack/kolla-ansible/+/926209 | 13:37 |
opendevreview | Grzegorz Bialas proposed openstack/kolla-ansible master: Fixes bug: https://bugs.launchpad.net/kolla-ansible/+bug/2066052 https://review.opendev.org/c/openstack/kolla-ansible/+/927022 | 13:43 |
mnasiadka | frickler: I've seen that, it doesn't happen everytime, that's why I raised https://review.opendev.org/c/openstack/kolla-ansible/+/926831 - but I doubt that's going to fix it | 13:46 |
SvenKieske | mhm, which cases does that fix exactly? isn't the default decoding utf-8 these days? | 13:50 |
opendevreview | Merged openstack/kolla-ansible master: Revert "Allow setting any_errors_fatal true for gather-facts" https://review.opendev.org/c/openstack/kolla-ansible/+/910601 | 13:52 |
mnasiadka | SvenKieske: well, just look in that file - we treat that properly in podman case, and we don't care if it's a bytes object in docker case | 13:53 |
mnasiadka | but yes, it might be related to those deprecation warnings ;-) | 13:54 |
mnasiadka | but I think we have a patch in review for that | 13:54 |
mnasiadka | frickler: https://review.opendev.org/c/openstack/kolla-ansible/+/857003 - here | 13:55 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: bifrost: bump Ansible version to 8 (rpm) and 9 (deb) https://review.opendev.org/c/openstack/kolla/+/924246 | 13:57 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: mariadb: Bump to 11.4 https://review.opendev.org/c/openstack/kolla/+/926981 | 14:01 |
SvenKieske | mhm, the current docker/podman implementation still has quite some code duplication I guess.. :/ | 14:01 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: mariadb: Bump to 11.4 https://review.opendev.org/c/openstack/kolla/+/926981 | 14:04 |
opendevreview | Matt Crees proposed openstack/kolla-ansible master: Add a precheck to catch RMQ SLURP upgrades https://review.opendev.org/c/openstack/kolla-ansible/+/926146 | 14:04 |
opendevreview | Matt Crees proposed openstack/kolla-ansible master: Add a precheck to catch RMQ SLURP upgrades https://review.opendev.org/c/openstack/kolla-ansible/+/926146 | 14:05 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible master: CI: validate prometheus configs https://review.opendev.org/c/openstack/kolla-ansible/+/926209 | 14:12 |
opendevreview | Grzegorz Bialas proposed openstack/kolla-ansible master: Fix links in docs for unmaintained releases https://review.opendev.org/c/openstack/kolla-ansible/+/927022 | 14:13 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Switch to Ubuntu 24.04 LTS https://review.opendev.org/c/openstack/kolla/+/907589 | 14:20 |
arcayne | has anyone seen the mlnx-ofed package being replaced by doca-host/doca-ofed? All vf-lag info/docs for kayobe use mlnx packages. Wondering if switching to doca is a major task | 14:26 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Enable internal/backend TLS in LE scenario https://review.opendev.org/c/openstack/kolla-ansible/+/926210 | 14:28 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: glance: Add support for unified limits https://review.opendev.org/c/openstack/kolla-ansible/+/924701 | 14:29 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: inspector: Add support for copying known_devices.yaml https://review.opendev.org/c/openstack/kolla-ansible/+/923946 | 14:30 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Make ironic-inspector optional https://review.opendev.org/c/openstack/kolla-ansible/+/923948 | 14:35 |
mnasiadka | SvenKieske: feel free to refactor, but I didn't see the need to spend time to unduplicate 4 or 5 lines ;-) | 14:35 |
SvenKieske | I'm currenlty staring at our awesome "if /plugins/*" logic and how to best move that to the macro or if it's even necessary.. | 14:38 |
mnasiadka | well, I was sort of surprised we don't clear plugins-archive for each container and we get the same plugins as on parent | 14:39 |
mnasiadka | well the same + the ones that are defined for the currently built image | 14:39 |
mnasiadka | it doesn't make any sense | 14:40 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: kolla_toolbox: Add handling for bytes literals in docker https://review.opendev.org/c/openstack/kolla-ansible/+/926831 | 14:40 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: inspector: Add support for copying known_devices.yaml https://review.opendev.org/c/openstack/kolla-ansible/+/923946 | 14:43 |
SvenKieske | as far as I can see the logic is quite broken, yes. there doesn't seem any actual need in the code to require "/plugins/*" in order to be able to install "requests-kerberos", it's just that the check if that directory is there is at the wrong place (before installation, not before adding it to the list of stuff to install). | 14:43 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Make ironic-inspector optional https://review.opendev.org/c/openstack/kolla-ansible/+/923948 | 14:45 |
SvenKieske | macros.install_pip does even already have a check, so an empty directory passed to it does just return true | 14:49 |
opendevreview | Matt Crees proposed openstack/kolla-ansible stable/2024.1: [2024.1 only] CI: RMQ version upgrade before SLURP https://review.opendev.org/c/openstack/kolla-ansible/+/927037 | 14:54 |
SvenKieske | I have a suspicion, let's test this.. | 14:55 |
opendevreview | Sven Kieske proposed openstack/kolla master: remove check for plugins directory https://review.opendev.org/c/openstack/kolla/+/927038 | 14:59 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: mariadb: switch to mariadbd-safe https://review.opendev.org/c/openstack/kolla-ansible/+/927039 | 15:02 |
SvenKieske | I'm fairly certain now the "if ls /plugins" foo is just a cargo cult, it was needed in https://review.opendev.org/c/openstack/kolla/+/401145/ but macros.install_pip already does check package list before invoking pip. | 15:07 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: mariadb: Bump to 11.4 https://review.opendev.org/c/openstack/kolla/+/926981 | 15:07 |
mnasiadka | TIL - mariadb 11.4 decided to stop shipping links mylsqd -> mariadbd anymore (well, it does but in a separate -compat package which we don't need I guess) | 15:09 |
opendevreview | Pierre Riteau proposed openstack/kayobe master: seed-vm/infra-vms: Add support for EFI and q35 https://review.opendev.org/c/openstack/kayobe/+/921660 | 15:10 |
opendevreview | Pierre Riteau proposed openstack/kayobe master: seed-vm/infra-vms: Use efi as default https://review.opendev.org/c/openstack/kayobe/+/927015 | 15:10 |
opendevreview | Pierre Riteau proposed openstack/kayobe master: [DNM] Test seed-vm job with Rocky Linux image https://review.opendev.org/c/openstack/kayobe/+/926984 | 15:10 |
SvenKieske | mnasiadka: if you have some minutes patchset 11 now works for the prometheus linter: https://review.opendev.org/c/openstack/kolla-ansible/+/926209 (zuul reports aren't in gerrit yet) | 15:17 |
SvenKieske | build that succeeded: https://zuul.opendev.org/t/openstack/build/40909287fc6b412c8e1998ff12bbac27 | 15:18 |
mnasiadka | looks good, will wait for the results | 15:21 |
mnasiadka | which makes me think - that for some services we could think about running those checks before we deploy? | 15:21 |
mnasiadka | like in prechecks or something | 15:22 |
simondodsley | Can I get some reviews for https://review.opendev.org/c/openstack/kolla-ansible/+/879846. Been waiting for over a year | 15:22 |
opendevreview | Sven Kieske proposed openstack/kolla master: remove check for plugins directory https://review.opendev.org/c/openstack/kolla/+/927038 | 15:22 |
SvenKieske | mnasiadka: I already said it before: for that to work, we need to really generate all configs before deployment, afaik we don't do that for everything? not sure though. | 15:24 |
SvenKieske | second problem is, some configs, e.g. prometheus require special linters to be installed, which is currently circumvented by just reusing the live container images from the deployment, that could be fixed though I guess | 15:25 |
SvenKieske | but I agree in general, it should all be moved to prechecks imho, and it should not be necessary to actually deploy anything in order to validate config, because you usually want to do that before deploying :D | 15:25 |
SvenKieske | but last time I tried, afaik that didn't work, because at precheck stage configs were not rendered, iirc that was the problem with the haproxy lints. | 15:26 |
opendevreview | Merged openstack/kolla-ansible master: Remove RabbitMQ queue migrations from upgrade test https://review.opendev.org/c/openstack/kolla-ansible/+/926130 | 15:26 |
SvenKieske | with the configure stage not triggering restart handlers refactoring this might become smoother | 15:27 |
SvenKieske | that one: https://review.opendev.org/c/openstack/kolla-ansible/+/745164 | 15:27 |
opendevreview | Roman KrĨek proposed openstack/kolla-ansible master: Performance: Don't notify handlers during config https://review.opendev.org/c/openstack/kolla-ansible/+/745164 | 15:28 |
SvenKieske | mhm, guess it needs a proper rebase | 15:28 |
SvenKieske | but I need a proper weekend as well :D | 15:29 |
opendevreview | Merged openstack/kolla-ansible stable/2023.1: Add command to upgrade to a target version of RMQ https://review.opendev.org/c/openstack/kolla-ansible/+/918976 | 15:56 |
opendevreview | Verification of a change to openstack/kolla-ansible stable/2024.1 failed: Add command to upgrade to a target version of RMQ https://review.opendev.org/c/openstack/kolla-ansible/+/926997 | 15:56 |
opendevreview | Merged openstack/kolla master: Add support for checking incoming RabbitMQ version https://review.opendev.org/c/openstack/kolla/+/926145 | 16:44 |
opendevreview | Pierre Riteau proposed openstack/kayobe master: [DNM] Test seed-vm job with Ubuntu image https://review.opendev.org/c/openstack/kayobe/+/926984 | 16:47 |
opendevreview | Merged openstack/kolla-ansible master: Bump codespell pin to <3 https://review.opendev.org/c/openstack/kolla-ansible/+/925874 | 16:59 |
opendevreview | Merged openstack/kolla-ansible master: Remove "distutils" library https://review.opendev.org/c/openstack/kolla-ansible/+/857003 | 16:59 |
opendevreview | Merged openstack/kolla master: nova: Drop xen remnants https://review.opendev.org/c/openstack/kolla/+/926541 | 16:59 |
opendevreview | Merged openstack/kolla master: nova: drop dmidecode https://review.opendev.org/c/openstack/kolla/+/926542 | 17:05 |
opendevreview | Merged openstack/kolla master: bifrost: bump Ansible version to 8 (rpm) and 9 (deb) https://review.opendev.org/c/openstack/kolla/+/924246 | 17:09 |
opendevreview | Simon Dodsley proposed openstack/kolla-ansible master: Add Pure Storage FlashBlade as Manila backend https://review.opendev.org/c/openstack/kolla-ansible/+/879846 | 18:56 |
opendevreview | Merged openstack/kolla-ansible stable/2024.1: Add command to upgrade to a target version of RMQ https://review.opendev.org/c/openstack/kolla-ansible/+/926997 | 20:10 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: mariadb: switch to use mariadb flavored commands https://review.opendev.org/c/openstack/kolla-ansible/+/927039 | 20:16 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!