opendevreview | Michal Nasiadka proposed openstack/kolla stable/2023.2: opensearch: set OPENSEARCH_INITIAL_ADMIN_PASSWORD https://review.opendev.org/c/openstack/kolla/+/909559 | 05:40 |
---|---|---|
opendevreview | Michal Nasiadka proposed openstack/kolla stable/2023.1: opensearch: set OPENSEARCH_INITIAL_ADMIN_PASSWORD https://review.opendev.org/c/openstack/kolla/+/909560 | 05:40 |
opendevreview | Michal Nasiadka proposed openstack/kolla stable/zed: opensearch: set OPENSEARCH_INITIAL_ADMIN_PASSWORD https://review.opendev.org/c/openstack/kolla/+/909821 | 05:41 |
mnasiadka | frickler: 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 review | 10:09 |
mnasiadka | skariyap: have you raised a bug in launchpad? | 10:12 |
frickler | mnasiadka: 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 progress | 10:19 |
mnasiadka | Well, 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 impossible | 10:20 |
opendevreview | Patrick Lüdeke proposed openstack/kolla-ansible master: Reduce RabbitMQ health check CPU load https://review.opendev.org/c/openstack/kolla-ansible/+/909797 | 10:21 |
frickler | mnasiadka: yes, I do see the advantage, that's why I gave you my +2 :-) | 10:22 |
mnasiadka | frickler: if it wouldn't be that sha256 is a feature - we could think about backporting | 10:26 |
mnasiadka | but that would break existing overrides | 10:26 |
mnasiadka | so we need to live with more complicated backports ;-) | 10:26 |
frickler | mnasiadka: I also wouldn't backport without that feature, too much possible impact on deployments with local changes | 10:35 |
mnasiadka | frickler: righrt | 10:46 |
opendevreview | Will Szumski proposed openstack/kayobe master: WIP: Add podman support https://review.opendev.org/c/openstack/kayobe/+/909686 | 10:49 |
opendevreview | Will Szumski proposed openstack/kayobe master: WIP: Add podman support https://review.opendev.org/c/openstack/kayobe/+/909686 | 10:51 |
opendevreview | Alex Welsh proposed openstack/kayobe stable/2023.1: CI: Add SLURP upgrade job https://review.opendev.org/c/openstack/kayobe/+/909854 | 11:09 |
jovial | mnasiadka, 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/ce3a6aff09f1880d77bdd70d9f029b0780d9 | 11:14 |
jovial | 3dd9/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 |
mnasiadka | I think install ironic-inspector in kolla base image | 11:15 |
jovial | Sounds good, I'll propose a patch | 11:15 |
mnasiadka | We assume that all installations are done in Kolla, to not break users with internet constraints :) | 11:15 |
jovial | makes sense, thank you :) | 11:15 |
opendevreview | Merged openstack/kolla stable/2023.2: opensearch: set OPENSEARCH_INITIAL_ADMIN_PASSWORD https://review.opendev.org/c/openstack/kolla/+/909559 | 11:16 |
opendevreview | Merged openstack/kolla stable/2023.1: opensearch: set OPENSEARCH_INITIAL_ADMIN_PASSWORD https://review.opendev.org/c/openstack/kolla/+/909560 | 11:16 |
opendevreview | Merged openstack/kolla stable/zed: opensearch: set OPENSEARCH_INITIAL_ADMIN_PASSWORD https://review.opendev.org/c/openstack/kolla/+/909821 | 11:16 |
opendevreview | Alex Welsh proposed openstack/kayobe stable/2023.1: CI: Add SLURP upgrade job https://review.opendev.org/c/openstack/kayobe/+/909854 | 11:21 |
opendevreview | Alex Welsh proposed openstack/kayobe stable/2023.1: CI: Add SLURP upgrade job https://review.opendev.org/c/openstack/kayobe/+/909854 | 12:30 |
opendevreview | Jake Hutchinson proposed openstack/kayobe master: Register baremetal compute nodes in Ironic. https://review.opendev.org/c/openstack/kayobe/+/909671 | 13:09 |
skariyap | mnasiadka: 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 |
mnasiadka | skariyap: this is a normal deploy, not an upgrade - it has surely been exercised before by many people | 13:16 |
opendevreview | Alex Welsh proposed openstack/kayobe master: CI: Add SLURP upgrade job https://review.opendev.org/c/openstack/kayobe/+/909863 | 13:28 |
opendevreview | Will Szumski proposed openstack/kolla master: Install ironic-inspector in bifrost https://review.opendev.org/c/openstack/kolla/+/909865 | 13:39 |
opendevreview | Merged openstack/kolla master: Move curl sources to sources.py https://review.opendev.org/c/openstack/kolla/+/908989 | 13:43 |
opendevreview | wu.chunyang proposed openstack/kolla-ansible master: CI: Trove: Create a database instance https://review.opendev.org/c/openstack/kolla-ansible/+/863521 | 13:43 |
jovial | Anyone 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 |
jovial | Hit that when using master upper constraints | 13:55 |
mnasiadka | not really - pip is latest? | 13:57 |
mnasiadka | as in latest version | 13:57 |
wuchunyang | or you already install GitPython in your environment with a different version? | 13:58 |
jovial | Yeah, latest pip (24.0). Will check for GitPython, thanks. | 14:00 |
jovial | wuchunyang, 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 |
wuchunyang | you are welcome. | 14:17 |
wuchunyang | Hi, cores, please help me to review the trove patch. thanks in advance. https://review.opendev.org/c/openstack/kolla-ansible/+/863321 | 14:22 |
skariyap | mnasiadka: 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 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: WIP: Add support for rpm to repos.yaml https://review.opendev.org/c/openstack/kolla/+/909879 | 14:52 |
SvenKieske | for 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 something | 14:56 |
matusjenca | Hello, can somebody please review my PR https://review.opendev.org/c/openstack/kolla-ansible/+/909188. Thank you. | 15:08 |
SvenKieske | matusjenca: 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 |
SvenKieske | I'm actually not sure, if this is correct, is it? | 15:11 |
SvenKieske | I'll put the whole series https://review.opendev.org/q/topic:%22redis-cache%22 on the agenda for the next meeting next tuesday I guess | 15:14 |
SvenKieske | matusjenca: 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 room | 15:16 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: WIP: Add support for rpm to repos.yaml https://review.opendev.org/c/openstack/kolla/+/909879 | 15:25 |
matusjenca | @SvenKieske thank you, I abandoned it and will join the meeting | 15:42 |
opendevreview | Will Szumski proposed openstack/kolla master: Install ironic-inspector in bifrost https://review.opendev.org/c/openstack/kolla/+/909865 | 15:43 |
opendevreview | Will Szumski proposed openstack/kolla-ansible master: Re-enable ironic-inspector in bifrost https://review.opendev.org/c/openstack/kolla-ansible/+/909910 | 15: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 |
SvenKieske | that's a question for the stackhpc guys (mnasiadka ?), as I'm not using kayobe :) maybe kevko or someone else can also answer that | 15:53 |
opendevreview | Will Szumski proposed openstack/kayobe master: DNM: Test bifrost fixes https://review.opendev.org/c/openstack/kayobe/+/909911 | 16:14 |
mnasiadka | SvenKieske: jovial is a kayobe guy from SHPC ;) | 16:27 |
mnasiadka | jovial: I think we could have a var in k-a for those that want it and just use it in kayobe? | 16:28 |
SvenKieske | mnasiadka: ah my bad! :D there are so many of you, I lose track.. ;) | 16:30 |
opendevreview | Alex Welsh proposed openstack/kayobe master: CI: Add SLURP upgrade jobs https://review.opendev.org/c/openstack/kayobe/+/909863 | 16:38 |
opendevreview | Matúš Jenča proposed openstack/kolla-ansible master: Add backend TLS between MariaDB and ProxySQL https://review.opendev.org/c/openstack/kolla-ansible/+/909912 | 16:40 |
opendevreview | Will Szumski proposed openstack/kolla-ansible master: Extract useful information from HAProxy logs https://review.opendev.org/c/openstack/kolla-ansible/+/705484 | 16:46 |
opendevreview | Will Szumski proposed openstack/kayobe master: Skip buggy release of docker PyPI package https://review.opendev.org/c/openstack/kayobe/+/909913 | 16:51 |
jovial | mnasiadka, Sure, I'll make it a config option in k-a. | 16:55 |
opendevreview | Will Szumski proposed openstack/kolla-ansible master: Adds feature flag for ironic-inspector in bifrost https://review.opendev.org/c/openstack/kolla-ansible/+/909910 | 17:13 |
opendevreview | Will Szumski proposed openstack/kayobe master: Fix issue registering inspection rules https://review.opendev.org/c/openstack/kayobe/+/909911 | 17:21 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: WIP: Fix arm builds https://review.opendev.org/c/openstack/kolla/+/909915 | 17:30 |
opendevreview | Russell 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/+/909919 | 18:06 |
opendevreview | Christian Berendt proposed openstack/kolla master: Use openstack-exporter 1.7.0 https://review.opendev.org/c/openstack/kolla/+/909591 | 19:42 |
opendevreview | Russell 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/+/909919 | 21:08 |
supamatt | is 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/!