Monday, 2024-11-25

opendevreviewMichal Nasiadka proposed openstack/kayobe-config-dev master: Store ironic-inspector data in database  https://review.opendev.org/c/openstack/kayobe-config-dev/+/93612309:18
PrzemekKPlease look at https://www.reddit.com/r/openstack/comments/1gyiobu/kolla_ansible_deployment/09:19
mnasiadkaPrzemekK: Yes, that's the effect of rewriting CLI to Python/Cliff, and it's mentioned in the release notes.09:29
opendevreviewSven Kieske proposed openstack/kolla-ansible stable/2024.2: Give ironic-inspector system scope ``all``  https://review.opendev.org/c/openstack/kolla-ansible/+/93612509:43
opendevreviewMaksim Malchuk proposed openstack/kolla-ansible master: Make certificates role more configurable  https://review.opendev.org/c/openstack/kolla-ansible/+/93451409:44
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: nova: Add support for using uWSGI  https://review.opendev.org/c/openstack/kolla-ansible/+/93597509:47
SvenKieskethere's a bug to fix the docs for this, maybe I got time to actually do this today.09:49
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: nova: Add support for using uWSGI  https://review.opendev.org/c/openstack/kolla-ansible/+/93597509:53
*** olafb` is now known as olafb09:56
kevko\o/09:57
opendevreviewVerification 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/+/93433809:59
SvenKieskeo/ kevko, back from vacation?10:01
opendevreviewRafal Lewandowski proposed openstack/kolla-ansible master: Enable ML2/OVN by default  https://review.opendev.org/c/openstack/kolla-ansible/+/90495910:03
kevkoSvenKieske: yep10:05
kevkosomething new ? 10:05
SvenKieskelot's of stuff in the changes I guess. I didn't review quite some time as well, but today I got some time for it10:08
SvenKieskeI 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
SvenKieskeit's about increased election times for ovs raft, which is an issue during high load.10:09
kevkoSvenKieske: i see10:14
SvenKieskesome of your stuff got merged, e.g. the ability to apply custom patches :)10:16
SvenKieskeand you got a +2 for the ceph stuff from me now: https://review.opendev.org/c/openstack/kolla-ansible/+/90716610:16
kevkoSvenKieske: i've seen ...thanks god 10:16
kevkoSvenKieske: so if ceph is +2 now from you ..you can also https://review.opendev.org/c/openstack/kolla-ansible/+/926408 this 10:17
SvenKieskeah thx for the pointer, there's a lot of open reviews, looking into it10:19
deflatedMorning 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
SvenKieskeI'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
kevkoSvenKieske: why ? :D 10:22
kevkoSvenKieske: some tldr version ? 10:22
SvenKieskekevko: "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 details10:24
SvenKieskequite 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
SvenKieskethat'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
SvenKieskeand 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
kevkoSvenKieske: from user perspective nova is ephemeral ...because it's deleted with VM ...cinder is not ...10:30
kevkoSvenKieske: https://docs.openstack.org/arch-design/design-storage/design-storage-concepts.html << official doc from openstack ? :D 10:30
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: nova: Add support for using uWSGI  https://review.opendev.org/c/openstack/kolla-ansible/+/93597510:30
SvenKieskeyeah, from a user pov this is confusing, so I think I gotta +2 that patch :910:31
mnasiadkaSvenKieske: https://review.opendev.org/c/openstack/kolla-ansible/+/926408 - that is dependent on the Ceph stuff10:32
opendevreviewMerged openstack/kolla-ansible stable/2023.2: Change from PERCONA_SCHEMA to mysql for mariabackup  https://review.opendev.org/c/openstack/kolla-ansible/+/93433910:33
SvenKieskeah sorry10:33
SvenKieskeright10:33
opendevreviewVerification 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/+/92640810:33
opendevreviewVerification of a change to openstack/kolla-ansible master failed: Enhance Ceph Integration for Multiple Clusters  https://review.opendev.org/c/openstack/kolla-ansible/+/90716610:33
opendevreviewMichal Nasiadka proposed openstack/kolla master: WIP: Move pre tasks into roles  https://review.opendev.org/c/openstack/kolla/+/92059010:34
deflated@PrzemekK Here's the results from vvvv: https://paste.openstack.org/show/bkGukBgreiLaw4un3F3o/10:34
SvenKieskeuhm, the ceph stuff failed with a dependency on the libvirt secret stuff? is there some circular dependency?oO10:35
opendevreviewMichal Nasiadka proposed openstack/kolla master: WIP: Move pre tasks into roles  https://review.opendev.org/c/openstack/kolla/+/92059010:35
mnasiadkaSvenKieske: you took back workflow?10:37
SvenKieskeyep, did that trip up zuul now? :(10:37
SvenKieskemnasiadka: 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
opendevreviewJakub Darmach proposed openstack/kolla-ansible master: Quickstart guide - remove Ansible dependency  https://review.opendev.org/c/openstack/kolla-ansible/+/93612710:38
mnasiadkaSvenKieske: I remember asking about cyborg related CI in some patch10:39
SvenKieskeI asked about it in the above patch as well10:40
opendevreviewBartosz Bezak proposed openstack/kayobe master: CI: use ECR registry [1] for alpine due too dockerhub rate limits  https://review.opendev.org/c/openstack/kayobe/+/93613010:43
opendevreviewJakub Darmach proposed openstack/kolla-ansible master: Quickstart guide - remove Ansible dependency  https://review.opendev.org/c/openstack/kolla-ansible/+/93612710:46
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Use more descriptive libvirt secret names corresponding to reality  https://review.opendev.org/c/openstack/kolla-ansible/+/92640810:53
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Enhance Ceph Integration for Multiple Clusters  https://review.opendev.org/c/openstack/kolla-ansible/+/90716610:53
opendevreviewMerged openstack/kolla-ansible stable/2023.2: Fix mariadb backup unable to create history table  https://review.opendev.org/c/openstack/kolla-ansible/+/93433510:58
opendevreviewPierre Riteau proposed openstack/kayobe master: CI: use ECR registry [1] for alpine due to dockerhub rate limits  https://review.opendev.org/c/openstack/kayobe/+/93613011:16
opendevreviewPierre Riteau proposed openstack/kolla-ansible master: Avoid use of quotes in HAProxy configuration  https://review.opendev.org/c/openstack/kolla-ansible/+/93613911:26
opendevreviewMerged openstack/kayobe-config-dev master: Store ironic-inspector data in database  https://review.opendev.org/c/openstack/kayobe-config-dev/+/93612311:34
opendevreviewChristian Schilling proposed openstack/kolla-ansible master: Fix external ceph cinder keyring  https://review.opendev.org/c/openstack/kolla-ansible/+/93586011:37
opendevreviewMerged openstack/kolla-ansible stable/2024.1: Add configuration for NTP server in Ironic Inspector DHCP server  https://review.opendev.org/c/openstack/kolla-ansible/+/93195511:37
opendevreviewMerged 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/+/93575711:37
opendevreviewVerification 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/+/93433812:05
opendevreviewMaksim Malchuk proposed openstack/kolla-ansible master: Make certificates role more configurable  https://review.opendev.org/c/openstack/kolla-ansible/+/93451412:07
opendevreviewMaksim Malchuk proposed openstack/kolla-ansible master: Make certificates role more configurable  https://review.opendev.org/c/openstack/kolla-ansible/+/93451412:07
opendevreviewJakub Darmach proposed openstack/kolla-ansible master: Quickstart guide - remove Ansible dependency  https://review.opendev.org/c/openstack/kolla-ansible/+/93612712:07
opendevreviewSven Kieske proposed openstack/kolla-ansible stable/2024.2: set haproxy for cyborg module  https://review.opendev.org/c/openstack/kolla-ansible/+/93615313:00
SvenKieskemnasiadka, 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
fricklerSvenKieske: 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 up13:37
SvenKieskeyep; I'll upload a patch13:37
fricklermnasiadka: 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 guess13:40
opendevreviewPierre Riteau proposed openstack/kolla-ansible master: Avoid double quotes in HAProxy configuration  https://review.opendev.org/c/openstack/kolla-ansible/+/93613913:58
SvenKieskefrickler: 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/+/92335514:13
SvenKieskeah right, I first assumed we had this still running on master, seems I just need to remove it from 2024.114:15
SvenKieskemhm14:15
SvenKieskebut can we even remove platform support from a stable branch?14:15
SvenKieskethe last email regarding openeuler is from april14:29
opendevreviewMerged openstack/kolla-ansible stable/2024.1: Updating proxysql.yaml.j2 to use mariadb_port for backends  https://review.opendev.org/c/openstack/kolla-ansible/+/93444114:32
opendevreviewVerification 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/+/93433814:37
lansonHello ! 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
opendevreviewVerification 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/+/93613015:08
mnasiadkafrickler: We should check kolla builds if they don't fail15:12
mnasiadkaAnd if they work after all other projects transitioned - then I guess we're good to go15:12
opendevreviewyosef proposed openstack/kolla-ansible master: Add election-timer option to ovn-db role  https://review.opendev.org/c/openstack/kolla-ansible/+/93410615:16
opendevreviewyosef proposed openstack/kolla-ansible master: Add election-timer option to ovn-db role  https://review.opendev.org/c/openstack/kolla-ansible/+/93410615:18
opendevreviewBartosz Bezak proposed openstack/kayobe-config-dev stable/2024.2: Store ironic-inspector data in database  https://review.opendev.org/c/openstack/kayobe-config-dev/+/93616715:20
opendevreviewyosef proposed openstack/kolla-ansible master: Add election-timer option to ovn-db role  https://review.opendev.org/c/openstack/kolla-ansible/+/93410615:20
opendevreviewMerged openstack/kolla-ansible master: Use more descriptive libvirt secret names corresponding to reality  https://review.opendev.org/c/openstack/kolla-ansible/+/92640815:23
opendevreviewBartosz Bezak proposed openstack/kayobe-config-dev stable/2024.1: Store ironic-inspector data in database  https://review.opendev.org/c/openstack/kayobe-config-dev/+/93616815:25
opendevreviewBartosz Bezak proposed openstack/kolla-ansible master: update openstack_previous_release_name var to 2024.1  https://review.opendev.org/c/openstack/kolla-ansible/+/93616915:39
andreykurilinHi 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 release16:06
opendevreviewChristian Schilling proposed openstack/kolla-ansible master: Fix external ceph cinder keyring  https://review.opendev.org/c/openstack/kolla-ansible/+/93586016:22
kevkoandreykurilin: what's the branch you see that error ? 16:28
SvenKieskemhm, second time I got this retry_limit error due to "can't find LABEL=kolla": https://zuul.opendev.org/t/openstack/build/3c5d5810c9f346988299064d907da4e916:32
opendevreviewMichal 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/+/93617416:33
opendevreviewMichal 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/+/93617516:34
opendevreviewMichal 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/+/93617616:36
opendevreviewMichal 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/+/93617616:36
fricklerSvenKieske: known issue, kolla jobs broken on th raxflex cloud, I added it to the meeting agenda already16:36
SvenKieskeah thx, thought as much but didn't want to spam #opendev :)16:37
SvenKieskebut I guess I should hold the rechecks then for now?16:38
andreykurilinkevko: stable/2024.116:39
fricklerSvenKieske: rechecks should have a good chance to land on a different cloud, so I wouldn't advise against them16:41
SvenKieskeok, good to know :)16:42
opendevreviewMerged openstack/kolla-ansible stable/2024.1: Fix IPv6 address format in ProxySQL configuration  https://review.opendev.org/c/openstack/kolla-ansible/+/93444217:53
opendevreviewMerged openstack/kayobe-config-dev stable/2024.2: Store ironic-inspector data in database  https://review.opendev.org/c/openstack/kayobe-config-dev/+/93616717:53
opendevreviewMerged openstack/kayobe-config-dev stable/2024.1: Store ironic-inspector data in database  https://review.opendev.org/c/openstack/kayobe-config-dev/+/93616817:54
opendevreviewMerged openstack/kolla-ansible master: Enhance Ceph Integration for Multiple Clusters  https://review.opendev.org/c/openstack/kolla-ansible/+/90716618:01
opendevreviewFelipe Figueroa Vergara proposed openstack/kolla master: Adds missing nvme package in cinder-backup  https://review.opendev.org/c/openstack/kolla/+/93619118:28
opendevreviewFelipe Figueroa Vergara proposed openstack/kolla master: Adds missing nvme package in cinder-backup  https://review.opendev.org/c/openstack/kolla/+/93619118:37
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Fix TLS settings when letsencrypt turned on  https://review.opendev.org/c/openstack/kolla-ansible/+/92597119:14
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Don't notify handlers during copy-cert  https://review.opendev.org/c/openstack/kolla-ansible/+/92465119:14
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Performance: Don't notify handlers during config  https://review.opendev.org/c/openstack/kolla-ansible/+/74516419:14
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Refactor services' check-containers and optimise  https://review.opendev.org/c/openstack/kolla-ansible/+/77324319:14
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Fix unintentional trigger of ansible handlers  https://review.opendev.org/c/openstack/kolla-ansible/+/92414519:14
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Fix proxysql-config's TLS DB configuration  https://review.opendev.org/c/openstack/kolla-ansible/+/93392219:14
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Add TLS support for MariaDB Connections  https://review.opendev.org/c/openstack/kolla-ansible/+/93392419:14
supamattanyone still working on the ovs and ovn exporters that are wip in the kolla backlog? 19:15
supamattJust asking, because this seems to work really well for just ovs19:15
supamatthttps://github.com/openstack-k8s-operators/dataplane-node-exporter/tree/main19:15
opendevreviewMerged openstack/kolla-ansible stable/2024.1: Change from PERCONA_SCHEMA to mysql for mariabackup  https://review.opendev.org/c/openstack/kolla-ansible/+/93433819:57
opendevreviewMerged openstack/kolla-ansible master: update openstack_previous_release_name var to 2024.1  https://review.opendev.org/c/openstack/kolla-ansible/+/93616921:57

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