opendevreview | Michal Nasiadka proposed openstack/kayobe-config-dev master: Store ironic-inspector data in database https://review.opendev.org/c/openstack/kayobe-config-dev/+/936123 | 09:18 |
---|---|---|
PrzemekK | Please look at https://www.reddit.com/r/openstack/comments/1gyiobu/kolla_ansible_deployment/ | 09:19 |
mnasiadka | PrzemekK: Yes, that's the effect of rewriting CLI to Python/Cliff, and it's mentioned in the release notes. | 09:29 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/2024.2: Give ironic-inspector system scope ``all`` https://review.opendev.org/c/openstack/kolla-ansible/+/936125 | 09:43 |
opendevreview | Maksim Malchuk proposed openstack/kolla-ansible master: Make certificates role more configurable https://review.opendev.org/c/openstack/kolla-ansible/+/934514 | 09:44 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: nova: Add support for using uWSGI https://review.opendev.org/c/openstack/kolla-ansible/+/935975 | 09:47 |
SvenKieske | there's a bug to fix the docs for this, maybe I got time to actually do this today. | 09:49 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: nova: Add support for using uWSGI https://review.opendev.org/c/openstack/kolla-ansible/+/935975 | 09:53 |
*** olafb` is now known as olafb | 09:56 | |
kevko | \o/ | 09:57 |
opendevreview | Verification of a change to openstack/kolla-ansible stable/2024.1 failed: Change from PERCONA_SCHEMA to mysql for mariabackup https://review.opendev.org/c/openstack/kolla-ansible/+/934338 | 09:59 |
SvenKieske | o/ kevko, back from vacation? | 10:01 |
opendevreview | Rafal Lewandowski proposed openstack/kolla-ansible master: Enable ML2/OVN by default https://review.opendev.org/c/openstack/kolla-ansible/+/904959 | 10:03 |
kevko | SvenKieske: yep | 10:05 |
kevko | something new ? | 10:05 |
SvenKieske | lot's of stuff in the changes I guess. I didn't review quite some time as well, but today I got some time for it | 10:08 |
SvenKieske | I actually would be interested in more opinions about https://review.opendev.org/c/openstack/kolla-ansible/+/934106 the idea is good imho, but maybe a little bit too complex/complicated in the execution? | 10:09 |
SvenKieske | it's about increased election times for ovs raft, which is an issue during high load. | 10:09 |
kevko | SvenKieske: i see | 10:14 |
SvenKieske | some of your stuff got merged, e.g. the ability to apply custom patches :) | 10:16 |
SvenKieske | and you got a +2 for the ceph stuff from me now: https://review.opendev.org/c/openstack/kolla-ansible/+/907166 | 10:16 |
kevko | SvenKieske: i've seen ...thanks god | 10:16 |
kevko | SvenKieske: so if ceph is +2 now from you ..you can also https://review.opendev.org/c/openstack/kolla-ansible/+/926408 this | 10:17 |
SvenKieske | ah thx for the pointer, there's a lot of open reviews, looking into it | 10:19 |
deflated | Morning all, trying to run kolla-ansible octavia-certificates -i multinode and keep getting Error while reading certificate file from disk: [Errno 2] No such file or directory even though its the first run, am i missing something or is octavia-certificates currently broken? | 10:21 |
SvenKieske | I'm sure sean mooney would disagree with you regarding the naming of nova storage as "ephemeral" (see one openstack-discuss ML for details) :D but I guess it's okay. | 10:21 |
kevko | SvenKieske: why ? :D | 10:22 |
kevko | SvenKieske: some tldr version ? | 10:22 |
SvenKieske | kevko: "in nova an ephemeral disk is an addition nova provisioned disk that is separate form teh root disk" see: https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/message/J2B6EPHVTO2WKGHHT7AX2UAE6BH6UTKI/ for details | 10:24 |
SvenKieske | quite educational, actually - as most of seans mails :) | 10:25 |
SvenKieske | "the nova root disk is no more or less ephemeral then a cidner voluem. but have a life time tied to the api object used to create them." | 10:25 |
SvenKieske | that's a good point. when the api object is deleted, the object (disk) is deleted. it just happens to be, that the api object is the virtual machine, in novas case. | 10:26 |
SvenKieske | and there are really ephemeral disk objects in the nova api, which are distinct from root disks. | 10:27 |
PrzemekK | @deflated for me it worked fine some time ago . what -vvvv show ? | 10:28 |
kevko | SvenKieske: from user perspective nova is ephemeral ...because it's deleted with VM ...cinder is not ... | 10:30 |
kevko | SvenKieske: https://docs.openstack.org/arch-design/design-storage/design-storage-concepts.html << official doc from openstack ? :D | 10:30 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: nova: Add support for using uWSGI https://review.opendev.org/c/openstack/kolla-ansible/+/935975 | 10:30 |
SvenKieske | yeah, from a user pov this is confusing, so I think I gotta +2 that patch :9 | 10:31 |
mnasiadka | SvenKieske: https://review.opendev.org/c/openstack/kolla-ansible/+/926408 - that is dependent on the Ceph stuff | 10:32 |
opendevreview | Merged openstack/kolla-ansible stable/2023.2: Change from PERCONA_SCHEMA to mysql for mariabackup https://review.opendev.org/c/openstack/kolla-ansible/+/934339 | 10:33 |
SvenKieske | ah sorry | 10:33 |
SvenKieske | right | 10:33 |
opendevreview | Verification of a change to openstack/kolla-ansible master failed: Use more descriptive libvirt secret names corresponding to reality https://review.opendev.org/c/openstack/kolla-ansible/+/926408 | 10:33 |
opendevreview | Verification of a change to openstack/kolla-ansible master failed: Enhance Ceph Integration for Multiple Clusters https://review.opendev.org/c/openstack/kolla-ansible/+/907166 | 10:33 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: WIP: Move pre tasks into roles https://review.opendev.org/c/openstack/kolla/+/920590 | 10:34 |
deflated | @PrzemekK Here's the results from vvvv: https://paste.openstack.org/show/bkGukBgreiLaw4un3F3o/ | 10:34 |
SvenKieske | uhm, the ceph stuff failed with a dependency on the libvirt secret stuff? is there some circular dependency?oO | 10:35 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: WIP: Move pre tasks into roles https://review.opendev.org/c/openstack/kolla/+/920590 | 10:35 |
mnasiadka | SvenKieske: you took back workflow? | 10:37 |
SvenKieske | yep, did that trip up zuul now? :( | 10:37 |
SvenKieske | mnasiadka: https://review.opendev.org/c/openstack/kolla-ansible/+/934429 should we mandate adding cyborg tests? seems this was broken due to lack of testing? | 10:37 |
opendevreview | Jakub Darmach proposed openstack/kolla-ansible master: Quickstart guide - remove Ansible dependency https://review.opendev.org/c/openstack/kolla-ansible/+/936127 | 10:38 |
mnasiadka | SvenKieske: I remember asking about cyborg related CI in some patch | 10:39 |
SvenKieske | I asked about it in the above patch as well | 10:40 |
opendevreview | Bartosz Bezak proposed openstack/kayobe master: CI: use ECR registry [1] for alpine due too dockerhub rate limits https://review.opendev.org/c/openstack/kayobe/+/936130 | 10:43 |
opendevreview | Jakub Darmach proposed openstack/kolla-ansible master: Quickstart guide - remove Ansible dependency https://review.opendev.org/c/openstack/kolla-ansible/+/936127 | 10:46 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Use more descriptive libvirt secret names corresponding to reality https://review.opendev.org/c/openstack/kolla-ansible/+/926408 | 10:53 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Enhance Ceph Integration for Multiple Clusters https://review.opendev.org/c/openstack/kolla-ansible/+/907166 | 10:53 |
opendevreview | Merged openstack/kolla-ansible stable/2023.2: Fix mariadb backup unable to create history table https://review.opendev.org/c/openstack/kolla-ansible/+/934335 | 10:58 |
opendevreview | Pierre Riteau proposed openstack/kayobe master: CI: use ECR registry [1] for alpine due to dockerhub rate limits https://review.opendev.org/c/openstack/kayobe/+/936130 | 11:16 |
opendevreview | Pierre Riteau proposed openstack/kolla-ansible master: Avoid use of quotes in HAProxy configuration https://review.opendev.org/c/openstack/kolla-ansible/+/936139 | 11:26 |
opendevreview | Merged openstack/kayobe-config-dev master: Store ironic-inspector data in database https://review.opendev.org/c/openstack/kayobe-config-dev/+/936123 | 11:34 |
opendevreview | Christian Schilling proposed openstack/kolla-ansible master: Fix external ceph cinder keyring https://review.opendev.org/c/openstack/kolla-ansible/+/935860 | 11:37 |
opendevreview | Merged openstack/kolla-ansible stable/2024.1: Add configuration for NTP server in Ironic Inspector DHCP server https://review.opendev.org/c/openstack/kolla-ansible/+/931955 | 11:37 |
opendevreview | Merged openstack/ansible-collection-kolla master: Revert "docker_sdk: Pin requests to <2.32 due to docker-py incompatibility" https://review.opendev.org/c/openstack/ansible-collection-kolla/+/935757 | 11:37 |
opendevreview | Verification of a change to openstack/kolla-ansible stable/2024.1 failed: Change from PERCONA_SCHEMA to mysql for mariabackup https://review.opendev.org/c/openstack/kolla-ansible/+/934338 | 12:05 |
opendevreview | Maksim Malchuk proposed openstack/kolla-ansible master: Make certificates role more configurable https://review.opendev.org/c/openstack/kolla-ansible/+/934514 | 12:07 |
opendevreview | Maksim Malchuk proposed openstack/kolla-ansible master: Make certificates role more configurable https://review.opendev.org/c/openstack/kolla-ansible/+/934514 | 12:07 |
opendevreview | Jakub Darmach proposed openstack/kolla-ansible master: Quickstart guide - remove Ansible dependency https://review.opendev.org/c/openstack/kolla-ansible/+/936127 | 12:07 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/2024.2: set haproxy for cyborg module https://review.opendev.org/c/openstack/kolla-ansible/+/936153 | 13:00 |
SvenKieske | mnasiadka, frickler: openeuler jobs seem only successfull on stable/2023.1 last success on msater was in july: https://zuul.opendev.org/t/openstack/builds?job_name=kolla-ansible-openeuler&project=openstack%2Fkolla-ansible&branch=master&result=SUCCESS&skip=0 shouldn't we disable the job on master at least? | 13:11 |
frickler | SvenKieske: iirc mnasiadka wanted to give interested parties some time to repair the CI, but I haven't seen that happen, so yes, let's clean up | 13:37 |
SvenKieske | yep; I'll upload a patch | 13:37 |
frickler | mnasiadka: semi-related question: are we ready to move 2023.1 to unmaintained? I had asked for a bit of time on https://review.opendev.org/c/openstack/releases/+/934490 so we could discuss this, but then I forgot and then meetings were canceled. we could still wait some more if needed I guess | 13:40 |
opendevreview | Pierre Riteau proposed openstack/kolla-ansible master: Avoid double quotes in HAProxy configuration https://review.opendev.org/c/openstack/kolla-ansible/+/936139 | 13:58 |
SvenKieske | frickler: maybe I'm lost in maze, but this is weird, this job is seemingly only present on stable branches, not master? https://zuul.opendev.org/t/openstack/job/kolla-ansible-openeuler I guess because of https://review.opendev.org/c/openstack/kolla-ansible/+/923355 | 14:13 |
SvenKieske | ah right, I first assumed we had this still running on master, seems I just need to remove it from 2024.1 | 14:15 |
SvenKieske | mhm | 14:15 |
SvenKieske | but can we even remove platform support from a stable branch? | 14:15 |
SvenKieske | the last email regarding openeuler is from april | 14:29 |
opendevreview | Merged openstack/kolla-ansible stable/2024.1: Updating proxysql.yaml.j2 to use mariadb_port for backends https://review.opendev.org/c/openstack/kolla-ansible/+/934441 | 14:32 |
opendevreview | Verification of a change to openstack/kolla-ansible stable/2024.1 failed: Change from PERCONA_SCHEMA to mysql for mariabackup https://review.opendev.org/c/openstack/kolla-ansible/+/934338 | 14:37 |
lanson | Hello ! can someone have a look at this ? https://review.opendev.org/c/openstack/kolla-ansible/+/934429 I think it should be ready to be merged | 14:57 |
opendevreview | Verification of a change to openstack/kayobe master failed: CI: use ECR registry [1] for alpine due to dockerhub rate limits https://review.opendev.org/c/openstack/kayobe/+/936130 | 15:08 |
mnasiadka | frickler: We should check kolla builds if they don't fail | 15:12 |
mnasiadka | And if they work after all other projects transitioned - then I guess we're good to go | 15:12 |
opendevreview | yosef proposed openstack/kolla-ansible master: Add election-timer option to ovn-db role https://review.opendev.org/c/openstack/kolla-ansible/+/934106 | 15:16 |
opendevreview | yosef proposed openstack/kolla-ansible master: Add election-timer option to ovn-db role https://review.opendev.org/c/openstack/kolla-ansible/+/934106 | 15:18 |
opendevreview | Bartosz Bezak proposed openstack/kayobe-config-dev stable/2024.2: Store ironic-inspector data in database https://review.opendev.org/c/openstack/kayobe-config-dev/+/936167 | 15:20 |
opendevreview | yosef proposed openstack/kolla-ansible master: Add election-timer option to ovn-db role https://review.opendev.org/c/openstack/kolla-ansible/+/934106 | 15:20 |
opendevreview | Merged openstack/kolla-ansible master: Use more descriptive libvirt secret names corresponding to reality https://review.opendev.org/c/openstack/kolla-ansible/+/926408 | 15:23 |
opendevreview | Bartosz Bezak proposed openstack/kayobe-config-dev stable/2024.1: Store ironic-inspector data in database https://review.opendev.org/c/openstack/kayobe-config-dev/+/936168 | 15:25 |
opendevreview | Bartosz Bezak proposed openstack/kolla-ansible master: update openstack_previous_release_name var to 2024.1 https://review.opendev.org/c/openstack/kolla-ansible/+/936169 | 15:39 |
andreykurilin | Hi folks! I need a suggestion - backported patch https://review.opendev.org/c/openstack/kolla/+/934314 causes an issue (it breaks nova-libvirt) with an issue described in https://bugs.launchpad.net/kolla-ansible/+bug/2073678 . This bug is fixed in master and marked as fixed. As root cause is different, should I raise a new bug report or leave a comment to the existing one? | 16:05 |
andreykurilin | ^ 2024.1 release | 16:06 |
opendevreview | Christian Schilling proposed openstack/kolla-ansible master: Fix external ceph cinder keyring https://review.opendev.org/c/openstack/kolla-ansible/+/935860 | 16:22 |
kevko | andreykurilin: what's the branch you see that error ? | 16:28 |
SvenKieske | mhm, second time I got this retry_limit error due to "can't find LABEL=kolla": https://zuul.opendev.org/t/openstack/build/3c5d5810c9f346988299064d907da4e9 | 16:32 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible stable/2023.1: Fix nova-libvirt's secrets - device or resource is busy https://review.opendev.org/c/openstack/kolla-ansible/+/936174 | 16:33 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible stable/2023.2: Fix nova-libvirt's secrets - device or resource is busy https://review.opendev.org/c/openstack/kolla-ansible/+/936175 | 16:34 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible stable/2024.1: Fix nova-libvirt's secrets - device or resource is busy https://review.opendev.org/c/openstack/kolla-ansible/+/936176 | 16:36 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible stable/2024.1: Fix nova-libvirt's secrets - device or resource is busy https://review.opendev.org/c/openstack/kolla-ansible/+/936176 | 16:36 |
frickler | SvenKieske: known issue, kolla jobs broken on th raxflex cloud, I added it to the meeting agenda already | 16:36 |
SvenKieske | ah thx, thought as much but didn't want to spam #opendev :) | 16:37 |
SvenKieske | but I guess I should hold the rechecks then for now? | 16:38 |
andreykurilin | kevko: stable/2024.1 | 16:39 |
frickler | SvenKieske: rechecks should have a good chance to land on a different cloud, so I wouldn't advise against them | 16:41 |
SvenKieske | ok, good to know :) | 16:42 |
opendevreview | Merged openstack/kolla-ansible stable/2024.1: Fix IPv6 address format in ProxySQL configuration https://review.opendev.org/c/openstack/kolla-ansible/+/934442 | 17:53 |
opendevreview | Merged openstack/kayobe-config-dev stable/2024.2: Store ironic-inspector data in database https://review.opendev.org/c/openstack/kayobe-config-dev/+/936167 | 17:53 |
opendevreview | Merged openstack/kayobe-config-dev stable/2024.1: Store ironic-inspector data in database https://review.opendev.org/c/openstack/kayobe-config-dev/+/936168 | 17:54 |
opendevreview | Merged openstack/kolla-ansible master: Enhance Ceph Integration for Multiple Clusters https://review.opendev.org/c/openstack/kolla-ansible/+/907166 | 18:01 |
opendevreview | Felipe Figueroa Vergara proposed openstack/kolla master: Adds missing nvme package in cinder-backup https://review.opendev.org/c/openstack/kolla/+/936191 | 18:28 |
opendevreview | Felipe Figueroa Vergara proposed openstack/kolla master: Adds missing nvme package in cinder-backup https://review.opendev.org/c/openstack/kolla/+/936191 | 18:37 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix TLS settings when letsencrypt turned on https://review.opendev.org/c/openstack/kolla-ansible/+/925971 | 19:14 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Don't notify handlers during copy-cert https://review.opendev.org/c/openstack/kolla-ansible/+/924651 | 19:14 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Performance: Don't notify handlers during config https://review.opendev.org/c/openstack/kolla-ansible/+/745164 | 19:14 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Refactor services' check-containers and optimise https://review.opendev.org/c/openstack/kolla-ansible/+/773243 | 19:14 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix unintentional trigger of ansible handlers https://review.opendev.org/c/openstack/kolla-ansible/+/924145 | 19:14 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix proxysql-config's TLS DB configuration https://review.opendev.org/c/openstack/kolla-ansible/+/933922 | 19:14 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Add TLS support for MariaDB Connections https://review.opendev.org/c/openstack/kolla-ansible/+/933924 | 19:14 |
supamatt | anyone still working on the ovs and ovn exporters that are wip in the kolla backlog? | 19:15 |
supamatt | Just asking, because this seems to work really well for just ovs | 19:15 |
supamatt | https://github.com/openstack-k8s-operators/dataplane-node-exporter/tree/main | 19:15 |
opendevreview | Merged openstack/kolla-ansible stable/2024.1: Change from PERCONA_SCHEMA to mysql for mariabackup https://review.opendev.org/c/openstack/kolla-ansible/+/934338 | 19:57 |
opendevreview | Merged openstack/kolla-ansible master: update openstack_previous_release_name var to 2024.1 https://review.opendev.org/c/openstack/kolla-ansible/+/936169 | 21:57 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!