Thursday, 2024-02-22

opendevreviewMichal Nasiadka proposed openstack/kolla stable/2023.2: opensearch: set OPENSEARCH_INITIAL_ADMIN_PASSWORD  https://review.opendev.org/c/openstack/kolla/+/90955905:40
opendevreviewMichal Nasiadka proposed openstack/kolla stable/2023.1: opensearch: set OPENSEARCH_INITIAL_ADMIN_PASSWORD  https://review.opendev.org/c/openstack/kolla/+/90956005:40
opendevreviewMichal Nasiadka proposed openstack/kolla stable/zed: opensearch: set OPENSEARCH_INITIAL_ADMIN_PASSWORD  https://review.opendev.org/c/openstack/kolla/+/90982105:41
mnasiadkafrickler: https://review.opendev.org/c/openstack/kolla/+/908989 - I'd like to get this in before anybody merges any updates to the prometheus/exporters - would be grateful for another review10:09
mnasiadkaskariyap: have you raised a bug in launchpad?10:12
fricklermnasiadka: yeah, I know, christian has a conflicting patch up. will make it more difficult to backport the latter, but anyway, I don't want to block progress10:19
mnasiadkaWell, I'd like to get a script working, that we would run once in a while and check versions - and in current shape it's nearly impossible10:20
opendevreviewPatrick Lüdeke proposed openstack/kolla-ansible master: Reduce RabbitMQ health check CPU load  https://review.opendev.org/c/openstack/kolla-ansible/+/90979710:21
fricklermnasiadka: yes, I do see the advantage, that's why I gave you my +2 :-)10:22
mnasiadkafrickler: if it wouldn't be that sha256 is a feature - we could think about backporting10:26
mnasiadkabut that would break existing overrides10:26
mnasiadkaso we need to live with more complicated backports ;-)10:26
fricklermnasiadka: I also wouldn't backport without that feature, too much possible impact on deployments with local changes10:35
mnasiadkafrickler: righrt10:46
opendevreviewWill Szumski proposed openstack/kayobe master: WIP: Add podman support  https://review.opendev.org/c/openstack/kayobe/+/90968610:49
opendevreviewWill Szumski proposed openstack/kayobe master: WIP: Add podman support  https://review.opendev.org/c/openstack/kayobe/+/90968610:51
opendevreviewAlex Welsh proposed openstack/kayobe stable/2023.1: CI: Add SLURP upgrade job  https://review.opendev.org/c/openstack/kayobe/+/90985411:09
jovialmnasiadka, I think I've got to the bottom of the kayobe bifrost failures. They have switched to the native in-band inspection (https://github.com/openstack/bifrost/commit/dcbf1d98b92236be7e4b9d89169a66892127e63b). Issue is when I re-enable it (enable_inspector=true). I hit an issue with ironic-inspector-dbsync being missing. I notice we set skip_package_install here: https://github.com/openstack/kolla-ansible/blob/ce3a6aff09f1880d77bdd70d9f029b0780d911:14
jovial3dd9/ansible/roles/bifrost/tasks/bootstrap.yml#L24. Is it best to install ironic-inspector in the kolla base image or enable package installation in kolla-ansible?11:14
mnasiadkaI think install ironic-inspector in kolla base image11:15
jovialSounds good, I'll propose a patch11:15
mnasiadkaWe assume that all installations are done in Kolla, to not break users with internet constraints :)11:15
jovialmakes sense, thank you :)11:15
opendevreviewMerged openstack/kolla stable/2023.2: opensearch: set OPENSEARCH_INITIAL_ADMIN_PASSWORD  https://review.opendev.org/c/openstack/kolla/+/90955911:16
opendevreviewMerged openstack/kolla stable/2023.1: opensearch: set OPENSEARCH_INITIAL_ADMIN_PASSWORD  https://review.opendev.org/c/openstack/kolla/+/90956011:16
opendevreviewMerged openstack/kolla stable/zed: opensearch: set OPENSEARCH_INITIAL_ADMIN_PASSWORD  https://review.opendev.org/c/openstack/kolla/+/90982111:16
opendevreviewAlex Welsh proposed openstack/kayobe stable/2023.1: CI: Add SLURP upgrade job  https://review.opendev.org/c/openstack/kayobe/+/90985411:21
opendevreviewAlex Welsh proposed openstack/kayobe stable/2023.1: CI: Add SLURP upgrade job  https://review.opendev.org/c/openstack/kayobe/+/90985412:30
opendevreviewJake Hutchinson proposed openstack/kayobe master: Register baremetal compute nodes in Ironic.  https://review.opendev.org/c/openstack/kayobe/+/90967113:09
skariyapmnasiadka: No, I have not raised a bug yet. I thought of checking with kolla team if what I am trying(in series upgrade with new images in Antelope) has been attempted before. 13:15
mnasiadkaskariyap: this is a normal deploy, not an upgrade - it has surely been exercised before by many people13:16
opendevreviewAlex Welsh proposed openstack/kayobe master: CI: Add SLURP upgrade job  https://review.opendev.org/c/openstack/kayobe/+/90986313:28
opendevreviewWill Szumski proposed openstack/kolla master: Install ironic-inspector in bifrost  https://review.opendev.org/c/openstack/kolla/+/90986513:39
opendevreviewMerged openstack/kolla master: Move curl sources to sources.py  https://review.opendev.org/c/openstack/kolla/+/90898913:43
opendevreviewwu.chunyang proposed openstack/kolla-ansible master: CI: Trove: Create a database instance  https://review.opendev.org/c/openstack/kolla-ansible/+/86352113:43
jovialAnyone seen this: "kolla 17.1.0.dev40 depends on GitPython>=1.0.1\n    The user requested (constraint) gitpython===3.1.42\n\nTo fix this you could try to:\n1. loosen the range of package versions you've specified"?13:53
jovialHit that when using master upper constraints13:55
mnasiadkanot really - pip is latest?13:57
mnasiadkaas in latest version13:57
wuchunyangor you already install GitPython in your environment with a different version?13:58
jovialYeah, latest pip (24.0). Will check for GitPython, thanks.14:00
jovialwuchunyang, thanks. I think you might be right, an older version was installed in the venv from a previous installation. I deleted the venv and recreated without issue. Thanks again.14:16
wuchunyangyou are welcome. 14:17
wuchunyangHi, cores, please help me  to review the trove patch. thanks in advance. https://review.opendev.org/c/openstack/kolla-ansible/+/86332114:22
skariyapmnasiadka: Thanks, shall I raise a bug in that case. With some consistency I was able to get a repro of the issue with just by re-deploying rabbitmq with a different image (no change, just different tag).14:37
opendevreviewMichal Nasiadka proposed openstack/kolla master: WIP: Add support for rpm to repos.yaml  https://review.opendev.org/c/openstack/kolla/+/90987914:52
SvenKieskefor people interested in ironic-inspector | HA deployments, just created this bug: https://bugs.launchpad.net/kolla-ansible/+bug/2054705 I'd like to have some input if my proposed solution is fine or if I maybe missed something14:56
matusjencaHello, can somebody please review my PR https://review.opendev.org/c/openstack/kolla-ansible/+/909188. Thank you.15:08
SvenKieskematusjenca: I take it, https://review.opendev.org/c/openstack/kolla-ansible/+/903978 is no longer needed? if this is the case, could you "abandon" that patch, to make that clear?15:10
SvenKieskeI'm actually not sure, if this is correct, is it?15:11
SvenKieskeI'll put the whole series https://review.opendev.org/q/topic:%22redis-cache%22  on the agenda for the next meeting next tuesday I guess15:14
SvenKieskematusjenca: I have put it on the agenda for the next meeting (which takes place next wednesday at 15:00 CET): https://etherpad.opendev.org/p/KollaWhiteBoard#L68 If you like you can also join, it's a plain text IRC meeting in this room15:16
opendevreviewMichal Nasiadka proposed openstack/kolla master: WIP: Add support for rpm to repos.yaml  https://review.opendev.org/c/openstack/kolla/+/90987915:25
matusjenca@SvenKieske thank you, I abandoned it and will join the meeting15:42
opendevreviewWill Szumski proposed openstack/kolla master: Install ironic-inspector in bifrost  https://review.opendev.org/c/openstack/kolla/+/90986515:43
opendevreviewWill Szumski proposed openstack/kolla-ansible master: Re-enable ironic-inspector in bifrost  https://review.opendev.org/c/openstack/kolla-ansible/+/90991015:51
jovial^ Does it make sense to re-enable ironic-inspector in kolla-ansible or should I just do so in kayobe (where we are using introspection rules)?15:52
SvenKieskethat's a question for the stackhpc guys (mnasiadka ?), as I'm not using kayobe :) maybe kevko or someone else can also answer that15:53
opendevreviewWill Szumski proposed openstack/kayobe master: DNM: Test bifrost fixes  https://review.opendev.org/c/openstack/kayobe/+/90991116:14
mnasiadkaSvenKieske: jovial is a kayobe guy from SHPC ;)16:27
mnasiadkajovial: I think we could have a var in k-a for those that want it and just use it in kayobe?16:28
SvenKieskemnasiadka: ah my bad! :D there are so many of you, I lose track.. ;)16:30
opendevreviewAlex Welsh proposed openstack/kayobe master: CI: Add SLURP upgrade jobs  https://review.opendev.org/c/openstack/kayobe/+/90986316:38
opendevreviewMatúš Jenča proposed openstack/kolla-ansible master: Add backend TLS between MariaDB and ProxySQL  https://review.opendev.org/c/openstack/kolla-ansible/+/90991216:40
opendevreviewWill Szumski proposed openstack/kolla-ansible master: Extract useful information from HAProxy logs  https://review.opendev.org/c/openstack/kolla-ansible/+/70548416:46
opendevreviewWill Szumski proposed openstack/kayobe master: Skip buggy release of docker PyPI package  https://review.opendev.org/c/openstack/kayobe/+/90991316:51
jovialmnasiadka, Sure, I'll make it a config option in k-a. 16:55
opendevreviewWill Szumski proposed openstack/kolla-ansible master: Adds feature flag for ironic-inspector in bifrost  https://review.opendev.org/c/openstack/kolla-ansible/+/90991017:13
opendevreviewWill Szumski proposed openstack/kayobe master: Fix issue registering inspection rules  https://review.opendev.org/c/openstack/kayobe/+/90991117:21
opendevreviewMichal Nasiadka proposed openstack/kolla master: WIP: Fix arm builds  https://review.opendev.org/c/openstack/kolla/+/90991517:30
opendevreviewRussell Tweed proposed openstack/kolla-ansible master: Avoids loop where 'murano_api_listen_port' is referenced to itself. Set to 'murani_api_port' instead.  https://review.opendev.org/c/openstack/kolla-ansible/+/90991918:06
opendevreviewChristian Berendt proposed openstack/kolla master: Use openstack-exporter 1.7.0  https://review.opendev.org/c/openstack/kolla/+/90959119:42
opendevreviewRussell Tweed proposed openstack/kolla-ansible master: Avoids loop where 'murano_api_listen_port' is referenced to itself. Set to 'murani_api_port' instead.  https://review.opendev.org/c/openstack/kolla-ansible/+/90991921:08
supamattis Kolla going to be supporting unified limits now that it is the recommended quota system for Nova in the C release?21:59

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