Friday, 2024-08-23

opendevreviewMichal Nasiadka proposed openstack/kolla master: mariadb: Bump to 11.4  https://review.opendev.org/c/openstack/kolla/+/92698106:37
opendevreviewMichal Nasiadka proposed openstack/kolla master: mariadb: Bump to 11.4  https://review.opendev.org/c/openstack/kolla/+/92698106:39
opendevreviewSeunghun Lee proposed openstack/kolla master: Fix symlinking of openstack-base source  https://review.opendev.org/c/openstack/kolla/+/92475906:45
opendevreviewSeunghun Lee proposed openstack/kolla master: Fix symlinking of openstack-base source  https://review.opendev.org/c/openstack/kolla/+/92475906:51
opendevreviewMichal Nasiadka proposed openstack/kolla master: Revert "[release] Change RDO's DELOREAN_DEPS repo to Caracal"  https://review.opendev.org/c/openstack/kolla/+/92698207:00
opendevreviewMichal Nasiadka proposed openstack/kolla master: ubuntu: switch to Dalmatian UCA  https://review.opendev.org/c/openstack/kolla/+/92698307:01
opendevreviewMichal Nasiadka proposed openstack/kolla master: Switch to Ubuntu 24.04 LTS  https://review.opendev.org/c/openstack/kolla/+/90758907:04
opendevreviewPierre Riteau proposed openstack/kayobe master: [DNM] Test seed-vm job with Rocky Linux image  https://review.opendev.org/c/openstack/kayobe/+/92698407:07
opendevreviewMichal Nasiadka proposed openstack/kolla master: mariadb: Bump to 11.4  https://review.opendev.org/c/openstack/kolla/+/92698107:13
mnasiadkafrickler, 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-reqs07:15
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: mariadb: Add lookup_cluster in post-deploy  https://review.opendev.org/c/openstack/kolla-ansible/+/92698707:31
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: [WIP] Enable ML2/OVN and distributed FIP by default.  https://review.opendev.org/c/openstack/kolla-ansible/+/90495907:37
opendevreviewMichal Nasiadka proposed openstack/kolla master: nova: drop dmidecode  https://review.opendev.org/c/openstack/kolla/+/92654207:49
opendevreviewMichal Nasiadka proposed openstack/kolla master: bifrost: bump Ansible version to 8/9 (rpm) and 9/10 (deb)  https://review.opendev.org/c/openstack/kolla/+/92424607:52
opendevreviewMichal Nasiadka proposed openstack/kolla master: bifrost: bump Ansible version to 8 (rpm) and 9 (deb)  https://review.opendev.org/c/openstack/kolla/+/92424607:52
opendevreviewMichal Nasiadka proposed openstack/kolla master: mariadb: Bump to 11.4  https://review.opendev.org/c/openstack/kolla/+/92698108:16
opendevreviewMatt Crees proposed openstack/kolla-ansible stable/2024.1: CI: Only migrate RMQ queues during SLURP  https://review.opendev.org/c/openstack/kolla-ansible/+/92612908:34
opendevreviewMatt Crees proposed openstack/kolla-ansible master: Add a precheck to catch RMQ SLURP upgrades  https://review.opendev.org/c/openstack/kolla-ansible/+/92614608:37
opendevreviewMatt 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/+/92699708:39
opendevreviewMatt 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/+/92699808:39
opendevreviewMatt 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/+/91897608:40
kevko\o08:43
opendevreviewMerged openstack/kayobe master: CI: Return to openstack-python3-jobs, fix pep8 failures  https://review.opendev.org/c/openstack/kayobe/+/92661908:57
opendevreviewBartosz Bezak proposed openstack/kolla master: Drop prometheus-msteams container images  https://review.opendev.org/c/openstack/kolla/+/92700009:10
opendevreviewBartosz Bezak proposed openstack/kolla master: Drop prometheus-msteams container images  https://review.opendev.org/c/openstack/kolla/+/92700009:17
opendevreviewBartosz Bezak proposed openstack/kolla-ansible master: Drop prometheus-msteams support  https://review.opendev.org/c/openstack/kolla-ansible/+/92700109:25
opendevreviewPierre Riteau proposed openstack/kayobe stable/2024.1: CI: Add openstack-tox-pep8 job, fix pep8 failures  https://review.opendev.org/c/openstack/kayobe/+/92700209:34
opendevreviewVerification of a change to openstack/kolla master failed: nova: Drop xen remnants  https://review.opendev.org/c/openstack/kolla/+/92654109:37
fricklermeh, what now, gate?09:41
frickler504 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/c367bcb1415645df85b6b6692c75fbcc09:42
fricklermnasiadka: 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
opendevreviewMerged openstack/kolla-ansible stable/2024.1: CI: Only migrate RMQ queues during SLURP  https://review.opendev.org/c/openstack/kolla-ansible/+/92612912:11
opendevreviewVerification of a change to openstack/kolla master failed: nova: Drop xen remnants  https://review.opendev.org/c/openstack/kolla/+/92654112:48
opendevreviewPierre Riteau proposed openstack/kayobe master: seed-vm/infra-vms: Add support for EFI and q35  https://review.opendev.org/c/openstack/kayobe/+/92166012:53
opendevreviewPierre Riteau proposed openstack/kayobe master: seed-vm/infra-vms: Use efi as default  https://review.opendev.org/c/openstack/kayobe/+/92701512:53
opendevreviewSven Kieske proposed openstack/kolla-ansible master: CI: validate prometheus configs  https://review.opendev.org/c/openstack/kolla-ansible/+/92620912:53
opendevreviewSven Kieske proposed openstack/kolla-ansible master: CI: validate prometheus configs  https://review.opendev.org/c/openstack/kolla-ansible/+/92620913:00
opendevreviewSven Kieske proposed openstack/kolla-ansible master: CI: validate prometheus configs  https://review.opendev.org/c/openstack/kolla-ansible/+/92620913:09
opendevreviewSven Kieske proposed openstack/kolla-ansible stable/2024.1: external/internal vip: use proper ip comparison  https://review.opendev.org/c/openstack/kolla-ansible/+/92701813:11
opendevreviewSven Kieske proposed openstack/kolla-ansible stable/2023.2: external/internal vip: use proper ip comparison  https://review.opendev.org/c/openstack/kolla-ansible/+/92701913:11
opendevreviewSven Kieske proposed openstack/kolla-ansible stable/2023.1: external/internal vip: use proper ip comparison  https://review.opendev.org/c/openstack/kolla-ansible/+/92702013:12
SvenKieskeI 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
SvenKieskeah nvm, missed a space13:31
opendevreviewSimon Dodsley proposed openstack/kolla-ansible master: Add clarity to user guide for different Pure options  https://review.opendev.org/c/openstack/kolla-ansible/+/92702113:32
opendevreviewSven Kieske proposed openstack/kolla-ansible master: CI: validate prometheus configs  https://review.opendev.org/c/openstack/kolla-ansible/+/92620913:37
opendevreviewGrzegorz Bialas proposed openstack/kolla-ansible master: Fixes bug: https://bugs.launchpad.net/kolla-ansible/+bug/2066052  https://review.opendev.org/c/openstack/kolla-ansible/+/92702213:43
mnasiadkafrickler: 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 it13:46
SvenKieskemhm, which cases does that fix exactly? isn't the default decoding utf-8 these days?13:50
opendevreviewMerged openstack/kolla-ansible master: Revert "Allow setting any_errors_fatal true for gather-facts"  https://review.opendev.org/c/openstack/kolla-ansible/+/91060113:52
mnasiadkaSvenKieske: 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 case13:53
mnasiadkabut yes, it might be related to those deprecation warnings ;-)13:54
mnasiadkabut I think we have a patch in review for that13:54
mnasiadkafrickler: https://review.opendev.org/c/openstack/kolla-ansible/+/857003 - here13:55
opendevreviewMichal Nasiadka proposed openstack/kolla master: bifrost: bump Ansible version to 8 (rpm) and 9 (deb)  https://review.opendev.org/c/openstack/kolla/+/92424613:57
opendevreviewMichal Nasiadka proposed openstack/kolla master: mariadb: Bump to 11.4  https://review.opendev.org/c/openstack/kolla/+/92698114:01
SvenKieskemhm, the current docker/podman implementation still has quite some code duplication I guess.. :/14:01
opendevreviewMichal Nasiadka proposed openstack/kolla master: mariadb: Bump to 11.4  https://review.opendev.org/c/openstack/kolla/+/92698114:04
opendevreviewMatt Crees proposed openstack/kolla-ansible master: Add a precheck to catch RMQ SLURP upgrades  https://review.opendev.org/c/openstack/kolla-ansible/+/92614614:04
opendevreviewMatt Crees proposed openstack/kolla-ansible master: Add a precheck to catch RMQ SLURP upgrades  https://review.opendev.org/c/openstack/kolla-ansible/+/92614614:05
opendevreviewSven Kieske proposed openstack/kolla-ansible master: CI: validate prometheus configs  https://review.opendev.org/c/openstack/kolla-ansible/+/92620914:12
opendevreviewGrzegorz Bialas proposed openstack/kolla-ansible master: Fix links in docs for unmaintained releases  https://review.opendev.org/c/openstack/kolla-ansible/+/92702214:13
opendevreviewMichal Nasiadka proposed openstack/kolla master: Switch to Ubuntu 24.04 LTS  https://review.opendev.org/c/openstack/kolla/+/90758914:20
arcaynehas 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 task14:26
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: CI: Enable internal/backend TLS in LE scenario  https://review.opendev.org/c/openstack/kolla-ansible/+/92621014:28
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: glance: Add support for unified limits  https://review.opendev.org/c/openstack/kolla-ansible/+/92470114:29
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: inspector: Add support for copying known_devices.yaml  https://review.opendev.org/c/openstack/kolla-ansible/+/92394614:30
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: WIP: Make ironic-inspector optional  https://review.opendev.org/c/openstack/kolla-ansible/+/92394814:35
mnasiadkaSvenKieske: feel free to refactor, but I didn't see the need to spend time to unduplicate 4 or 5 lines ;-)14:35
SvenKieskeI'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
mnasiadkawell, I was sort of surprised we don't clear plugins-archive for each container and we get the same plugins as on parent14:39
mnasiadkawell the same + the ones that are defined for the currently built image14:39
mnasiadkait doesn't make any sense14:40
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: kolla_toolbox: Add handling for bytes literals in docker  https://review.opendev.org/c/openstack/kolla-ansible/+/92683114:40
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: inspector: Add support for copying known_devices.yaml  https://review.opendev.org/c/openstack/kolla-ansible/+/92394614:43
SvenKieskeas 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
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: WIP: Make ironic-inspector optional  https://review.opendev.org/c/openstack/kolla-ansible/+/92394814:45
SvenKieskemacros.install_pip does even already have a check, so an empty directory passed to it does just return true14:49
opendevreviewMatt 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/+/92703714:54
SvenKieskeI have a suspicion, let's test this..14:55
opendevreviewSven Kieske proposed openstack/kolla master: remove check for plugins directory  https://review.opendev.org/c/openstack/kolla/+/92703814:59
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: mariadb: switch to mariadbd-safe  https://review.opendev.org/c/openstack/kolla-ansible/+/92703915:02
SvenKieskeI'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
opendevreviewMichal Nasiadka proposed openstack/kolla master: mariadb: Bump to 11.4  https://review.opendev.org/c/openstack/kolla/+/92698115:07
mnasiadkaTIL - 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
opendevreviewPierre Riteau proposed openstack/kayobe master: seed-vm/infra-vms: Add support for EFI and q35  https://review.opendev.org/c/openstack/kayobe/+/92166015:10
opendevreviewPierre Riteau proposed openstack/kayobe master: seed-vm/infra-vms: Use efi as default  https://review.opendev.org/c/openstack/kayobe/+/92701515:10
opendevreviewPierre Riteau proposed openstack/kayobe master: [DNM] Test seed-vm job with Rocky Linux image  https://review.opendev.org/c/openstack/kayobe/+/92698415:10
SvenKieskemnasiadka: 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
SvenKieskebuild that succeeded: https://zuul.opendev.org/t/openstack/build/40909287fc6b412c8e1998ff12bbac2715:18
mnasiadkalooks good, will wait for the results15:21
mnasiadkawhich makes me think - that for some services we could think about running those checks before we deploy?15:21
mnasiadkalike in prechecks or something15:22
simondodsleyCan I get some reviews for https://review.opendev.org/c/openstack/kolla-ansible/+/879846. Been waiting for over a year15:22
opendevreviewSven Kieske proposed openstack/kolla master: remove check for plugins directory  https://review.opendev.org/c/openstack/kolla/+/92703815:22
SvenKieskemnasiadka: 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
SvenKieskesecond 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 guess15:25
SvenKieskebut 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 :D15:25
SvenKieskebut 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
opendevreviewMerged openstack/kolla-ansible master: Remove RabbitMQ queue migrations from upgrade test  https://review.opendev.org/c/openstack/kolla-ansible/+/92613015:26
SvenKieskewith the configure stage not triggering restart handlers refactoring this might become smoother15:27
SvenKieskethat one: https://review.opendev.org/c/openstack/kolla-ansible/+/74516415:27
opendevreviewRoman KrĨek proposed openstack/kolla-ansible master: Performance: Don't notify handlers during config  https://review.opendev.org/c/openstack/kolla-ansible/+/74516415:28
SvenKieskemhm, guess it needs a proper rebase15:28
SvenKieskebut I need a proper weekend as well :D15:29
opendevreviewMerged openstack/kolla-ansible stable/2023.1: Add command to upgrade to a target version of RMQ  https://review.opendev.org/c/openstack/kolla-ansible/+/91897615:56
opendevreviewVerification 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/+/92699715:56
opendevreviewMerged openstack/kolla master: Add support for checking incoming RabbitMQ version  https://review.opendev.org/c/openstack/kolla/+/92614516:44
opendevreviewPierre Riteau proposed openstack/kayobe master: [DNM] Test seed-vm job with Ubuntu image  https://review.opendev.org/c/openstack/kayobe/+/92698416:47
opendevreviewMerged openstack/kolla-ansible master: Bump codespell pin to <3  https://review.opendev.org/c/openstack/kolla-ansible/+/92587416:59
opendevreviewMerged openstack/kolla-ansible master: Remove "distutils" library  https://review.opendev.org/c/openstack/kolla-ansible/+/85700316:59
opendevreviewMerged openstack/kolla master: nova: Drop xen remnants  https://review.opendev.org/c/openstack/kolla/+/92654116:59
opendevreviewMerged openstack/kolla master: nova: drop dmidecode  https://review.opendev.org/c/openstack/kolla/+/92654217:05
opendevreviewMerged openstack/kolla master: bifrost: bump Ansible version to 8 (rpm) and 9 (deb)  https://review.opendev.org/c/openstack/kolla/+/92424617:09
opendevreviewSimon Dodsley proposed openstack/kolla-ansible master: Add Pure Storage FlashBlade as Manila backend  https://review.opendev.org/c/openstack/kolla-ansible/+/87984618:56
opendevreviewMerged openstack/kolla-ansible stable/2024.1: Add command to upgrade to a target version of RMQ  https://review.opendev.org/c/openstack/kolla-ansible/+/92699720:10
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: mariadb: switch to use mariadb flavored commands  https://review.opendev.org/c/openstack/kolla-ansible/+/92703920:16

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!