Wednesday, 2024-09-25

ravlewOne +2 needed for ironic-conductor backports: https://review.opendev.org/c/openstack/kolla/+/93025807:24
ravlewhttps://review.opendev.org/c/openstack/kolla/+/93025907:24
ravlewhttps://review.opendev.org/c/openstack/kolla/+/93026007:24
SvenKieskedone07:26
mnasiadkamorning07:27
mnasiadkaSvenKieske, frickler , bbezak : seems we need to have a look in https://review.opendev.org/q/topic:%22kolla_endpoint_type%22 - octavia is failing due to this at least on Noble07:28
bbezakI think worth a bug for backporting mnasiadka . morning07:29
bbezak(if needed)07:30
SvenKieskeI wanted to ask about backporting indeed, currently reviewing, because frickler -1 backportvoted (probably due to the missing bugreport?)07:30
SvenKieskeas this interface is merged in keystone since 2017 a backport wouldn't hurt, it's not a new feature.07:31
fricklerhmm, I'd be willing to agree to that, yes07:31
fricklermnasiadka: do you have a pointer to the octavia failure?07:31
frickleralso, does this need a reno for people who override their config?07:32
mnasiadkayeah, probably needs a reno07:33
SvenKieskeyou added one instance in https://review.opendev.org/c/openstack/kolla-ansible/+/912359 though, that seems a little off?07:33
SvenKieskecommented on it07:34
SvenKieskewas that missing previously?07:34
mnasiadkafrickler: https://zuul.opendev.org/t/openstack/build/b5c27b94d0a14e3d8ac2d8edaa828bf0/log/primary/logs/kolla/octavia/octavia-api.txt#51307:35
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: octavia: Use valid_interfaces instead of endpoint_type  https://review.opendev.org/c/openstack/kolla-ansible/+/91235907:36
mnasiadkaBut it might be something else - but I guess endpoint_type is not even parsed now looking at keystoneauth code07:37
mnasiadkaI guess we need to treat every change individually and check if it helps in anything ;-)07:38
fricklerthat's a weird failure mode. also I don't see why that would be distro-specific07:38
fricklerbut I'm +2 on merging your whole stack if you rebase it07:39
SvenKieskemhm, honestly the reviews are going a little fast for me, already found one instance where something was added that wasn't there before07:40
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: CI: Create cluster in Magnum jobs  https://review.opendev.org/c/openstack/kolla-ansible/+/90449307:40
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: magnum: Use valid_interfaces instead of endpoint_type  https://review.opendev.org/c/openstack/kolla-ansible/+/91235807:40
SvenKieskecan we not rush this? it feels rushed to me at least.07:41
mnasiadkarebased magnum valid_interfaces on top of CI patch that really creates a Magnum cluster07:42
mnasiadkaif that passes - I guess we're good to merge07:42
mnasiadkathere's no need to rush, I'll action on octavia and magnum today and test if that doesn't break anything07:42
SvenKieskewas ther a new keystone release just now or how did you find this breakage?07:43
fricklerthe stack is 6 months old, not sure that can be called rushing07:43
fricklerI do find this in upstream testing at least, too 07:43
fricklerhttps://opensearch.logs.openstack.org/_dashboards/app/data-explorer/discover#?_a=(discover:(columns:!(_source),isDirty:!f,sort:!()),metadata:(indexPattern:'94869730-aea8-11ec-9e6a-83741af3fdcd',view:discover))&_g=(filters:!(),refreshInterval:(pause:!t,value:0),time:(from:now-7d,to:now))&_q=(filters:!(),query:(language:kuery,query:'message:%22Encountered%20an%20exception%20attempting%20to%20proces07:43
fricklers%20config%20for%20project%22'))07:43
mnasiadkaSvenKieske: I did this stack 6 months ago, when I spotted endpoint_type has been removed, and interface is deprecated :)07:44
mnasiadkaMight be it's not what is breaking anything07:45
mnasiadkabut still we should adapt to changes that have been done long time ago07:45
SvenKieskefrickler: if it has been lying around for 5 months and then suddenly is reviewed in a few hours it's still rushed :P but you got a point, I didn't look at the dates from the initial patch07:45
mnasiadkait seems there's no endpoint_type since 2015, so I'm still amazed how anything works :)07:45
fricklerhmm, this looks broken, though07:48
mnasiadkaLooking at codesearch.opendev.org only heat has entries in it's code for endpoint_type07:48
frickler2024-09-24 17:48:42.786 21 WARNING openstack [None req-fb514446-3a3d-4986-bcd4-09158c847ad3 - 944ff5cef3c3428cbb9a8e28fb00317f - - default default] Disabling service 'block-storage': Encountered an exception attempting to process config for project 'cinder' (service type 'block-storage'): no such option valid_interfaces in group [cinder]: oslo_config.cfg.NoSuchOptError: no such option 07:48
fricklervalid_interfaces in group [cinder]07:48
fricklerthat's from https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_378/912359/6/check/kolla-ansible-ubuntu-octavia/3785e79/primary/logs/kolla/octavia/octavia-api.txt on the octavia change07:48
mnasiadkayeah, I'll work on Octavia today and experiment what we really need, because it seems we need that in more sections?07:48
fricklerhmm, octavia.conf doesn't even contain a [cinder] section.07:50
fricklermight be some regression/weird behavior from recent oslo.config?07:51
mnasiadkamight be07:52
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: magnum: Add k8s_cluster_api_rockylinux to CAPI drivers list  https://review.opendev.org/c/openstack/kolla-ansible/+/93041408:29
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: CI: Create cluster in Magnum jobs  https://review.opendev.org/c/openstack/kolla-ansible/+/90449308:29
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: magnum: Use valid_interfaces instead of endpoint_type  https://review.opendev.org/c/openstack/kolla-ansible/+/91235808:30
kevkomorning08:40
kevkofrickler: you sent some opensearch dashboard  ..shoul I be able to login somehow ? Who is able to login ? :) 09:09
opendevreviewMerged openstack/kolla stable/2024.1: Add missing iso packages to ironic-conductor  https://review.opendev.org/c/openstack/kolla/+/93025809:10
opendevreviewMerged openstack/kolla stable/2023.2: Add missing iso packages to ironic-conductor  https://review.opendev.org/c/openstack/kolla/+/93025909:10
opendevreviewMerged openstack/kolla stable/2023.1: Add missing iso packages to ironic-conductor  https://review.opendev.org/c/openstack/kolla/+/93026009:10
fricklerkevko: login is openstack/openstack , it is just in place because it seems impossible to run it without a login09:12
kevkoah ok 09:12
kevkothanks 09:12
opendevreviewMichal Nasiadka proposed openstack/kolla master: erlang/aarch64: Move to openstack-kolla owned COPR  https://review.opendev.org/c/openstack/kolla/+/93042809:59
opendevreviewMichal Nasiadka proposed openstack/kolla master: erlang/aarch64: Move to openstack-kolla owned COPR  https://review.opendev.org/c/openstack/kolla/+/93042810:13
PrzemekKSomebody using Octavia with OVN deployed by KA ? Documentation is very limited for OVN https://docs.openstack.org/kolla-ansible/2024.1/reference/networking/octavia.html . What tasks must be done in globals and commands ?10:25
opendevreviewGrzegorz Koper proposed openstack/kolla-ansible master: Documentation update, to reflect VPNaaS deployment changes  https://review.opendev.org/c/openstack/kolla-ansible/+/93044211:08
opendevreviewGrzegorz Koper proposed openstack/kolla-ansible master: Documentation update, to reflect VPNaaS(ML2/OVS) deployment changes  https://review.opendev.org/c/openstack/kolla-ansible/+/93044211:12
sylvrHello ! I'm trying to configure nova because of an issue of timeout during instance creation.11:32
sylvrI based myself on https://docs.openstack.org/kayobe/latest/configuration/reference/kolla-ansible.html#control-plane-services and https://docs.openstack.org/newton/config-reference/compute/config-options.html11:34
sylvrto create a file in etc/kayobe/kolla/nova.conf with the parameters I wanted to change (number of retry and delay between retries for volume creation), I ran kayobe overcloud service reconfigure but it didn't change anything11:34
sylvrdid I miss something ?11:34
mnasiadka etc/kayobe/kolla/config/nova.conf would work11:42
sylvrmnasiadka: yuuup that would be the correct path, sorry for disturbing for nothing ^^11:50
opendevreviewPierre Riteau proposed openstack/ansible-collection-kolla master: Add installation of dbus package  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/92261412:30
opendevreviewJakub Darmach proposed openstack/kayobe master: Add support for Ubuntu Noble Numbat (24.04) LTS  https://review.opendev.org/c/openstack/kayobe/+/93002612:35
opendevreviewJakub Darmach proposed openstack/kayobe master: Add support for Ubuntu Noble Numbat (24.04) LTS  https://review.opendev.org/c/openstack/kayobe/+/93002612:37
mnasiadkaSvenKieske: https://review.opendev.org/c/openstack/ansible-collection-kolla/+/922614 - willing to have another look?12:39
kevko+112:39
SvenKieskelooking12:41
kevkocan I also approve things that I am sure that will pass in the future :) ? I am welcoming it 12:47
SvenKieskeah my bad, zuul only reported on patchset 212:48
kevkoyeah, few months old ...12:48
kevkobut it will pass i would say ...12:48
SvenKieskethis could use some GUI improvement to highlight that this is an old result :/12:48
kevkowell, it's on the top ..12:49
kevkoas summary 12:49
SvenKieskeI always look there, but I mean to highlight it there even more, it's only a small number, maybe make the green "success" a little grey if it's not the latest patchset or something?12:50
kevkoctrl+ on chrome :D 12:51
kevko*browser 12:51
kevkonevermind ... SvenKieske have you some time to check some of my patches ? 12:51
SvenKieskeif they are small like the one above, sure.12:54
mnasiadkabbezak frickler kevko SvenKieske mmalchuk gkoper jangutter jsuazo jovial osmanlicilegi mattcrees dougszu darmach - meeting in 4 minutes12:56
kevkohttps://review.opendev.org/c/openstack/kolla-ansible/+/929408  https://review.opendev.org/c/openstack/kolla-ansible/+/922369 < tiny ansible-core dependency13:00
mnasiadka#startmeeting kolla13:00
opendevmeetMeeting started Wed Sep 25 13:00:24 2024 UTC and is due to finish in 60 minutes.  The chair is mnasiadka. Information about MeetBot at http://wiki.debian.org/MeetBot.13:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.13:00
opendevmeetThe meeting name has been set to 'kolla'13:00
mnasiadka#topic rollcall13:00
mnasiadkao/13:00
SvenKieskeo/13:00
jovial\o13:00
bbezako/13:00
kevko\o/13:00
mhinero/13:00
r-krcek0/13:00
mnasiadka#topic agenda... (full message at <https://matrix.org/oftc/media/v1/media/download/AfvCuoxbKukDvfMYa1XqelHiI6JA2oy1yrxh5OX-liMapP23Mk5VHpYGNwZDdFVV4TB1g_TTAeUxVm6vFfY2Ur9CeSKv1GuQAG1hdHJpeC5vcmcvSEVTTFBoWldjanVBSWtJT0xKSWlqV0xS>)13:01
mnasiadka#topic CI status13:01
mnasiadkaGreen? Red? Amber? Watermelon?13:01
fricklero/13:02
fricklerthat agenda topic was mangled in the matrix13:02
SvenKieskewhat about honeydew melon? :D13:02
mnasiadkaYeah, switched to Element, probably it's mangling ;-)13:03
mnasiadkaI'll try better next time13:03
SvenKieskeyeah, you need to post agenda line by line for poor old IRC13:03
mnasiadkaCI looks not bad, except aarch6413:03
mnasiadkafrickler: do you know if there's any outlook on aarch64 improvements?13:04
fricklermnasiadka: none that I'm aware of13:04
mnasiadkaok, it is what it is13:05
mnasiadka#topic Current cycle planning13:05
mnasiadkaSo, Ubuntu 24.04 seems to be closer, some small CI failures to revisit and I guess I'll ask for reviews next week13:06
mnasiadkaanything else major that we should be thinking of?13:06
mnasiadkaThere's RMQ 4.013:06
mnasiadkaAs in no patches, but it got released13:06
mnasiadkaSince 4.0 release - 3.13 community support has ended13:07
SvenKieskeyes, we need to shift everything over from the old queues afaik for that to work, so afaik we need to finish the fanout/reply queue stuff13:07
mnasiadkaSounds fantastic13:07
SvenKieskefor that to work in turn, we need the queue manager which kevko is working on13:07
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Drop support for py38, py39  https://review.opendev.org/c/openstack/kolla-ansible/+/92940813:07
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Add ansible-core as a dependency  https://review.opendev.org/c/openstack/kolla-ansible/+/92236913:07
SvenKieskebut he seems busy :P ;)13:07
kevkowith queue manager it's little bit hard ..we need to bind /dev/shm to containers using oslo.messaging 13:08
mnasiadkais that secure?13:08
SvenKieskejust..bind mount it? :) /dev/shm is actually just a fancy way to say "tmpfs".13:08
kevkowell, i was thinking about the option to create tmpfs and mount as /dev/shm into container13:09
SvenKieskeI hope oslo has guarded it and there are no race conditions with concurrent access :D13:09
kevkobut then I realized we have already some containers which has /dev/shm already mounted 13:09
mnasiadkaIsn't --ipc=host for that?13:09
kevkoand ...moreover ...in classic openstack installations from packages ..there is shared memory also ...13:09
kevkono, ipc turning more than /dev/shm 13:09
kevkobut yes ..if you set ipc_mode ...it's effectively turn on bind /dev/shm into container 13:10
kevkobut we don't need it ...13:10
mnasiadkathere's the alternative of shared ipc for containers13:10
mnasiadkabut that was a bit... tricky, because you start a container pointing to other running containers ipc13:11
fricklermaybe we should stop doing containers if we share everything again anyway in the end? (almost, but not completely non-serious)13:11
SvenKieskeI think the short term conclusion is we need some time before we can truly merge rmq 4?13:11
mnasiadkayeah, that's another point13:11
mnasiadkaif we need shared mem for all containers, why even containers?13:11
mnasiadkaso, is that bloody queue manager completely required for RMQ 4?13:12
SvenKieskewell if we wanna split hairs it's afaik not required for rmq, but for how oslo/openstack uses rmq :)13:13
kevkofrickler: It is still an effective way to deploy OpenStack services and, for example, run two different versions on one system... it's containerized. Anyway... we use also like net=host namespace...it's similar 13:13
mnasiadkathe oslo.messaging patch says it's optional13:14
kevkofrickler: But yes, I understand your comment.. :) 13:14
SvenKieskewhat it does is essentially keeping track of queue names, so they always stay the same "per host" (that's the tricky part with containers). so consumers can just always bind to the same queue13:14
kevkomnasiadka: it's optional yes ... but we also provides ability to override configuration per user ...so if user choose to override and turn on the queue_manager ...he brokes his cloud because we are kolla ..right ? 13:14
SvenKieskexD13:15
kevkoand it's also very nice feature :) 13:15
mnasiadkaok, but it doesn't sound like required13:15
SvenKieskewell for first release we could put big fat warning that queue manager is not yet supported? if it works?13:15
kevkoso should we  add precheck if user override includes queue_manager = true and fail the reconfigure ? :D 13:15
mnasiadkauser overrides mean user knows what he's doing13:16
kevkobut agree that it's not important issue for now 13:16
mnasiadkawe never checked what is in user overrides13:16
kevkodon't forget it's also requirement for streams for fanout 13:16
mnasiadkaso let's not create a drama13:16
kevkoagree ... 13:16
kevkoit's mainly my downstream project which i am (as always) tracking in gerrit ... and I like that feature 13:17
mnasiadkaWell, if we can create a tmpfs volume and mount that as shm, that would probably be a bit better13:18
kevkoThat's most clean i think ...but i need to test if  - when you turn on the ipc if it's not automatically mounted host /dev/shm and overrides your tmpfs mount 13:19
mnasiadkawell, we only run libvirt with ipc=host IIRC13:20
mnasiadkathat doesn't interact with RMQ13:20
kevkohacluster ? 13:20
kevkocinder ? 13:20
kevkomasakari ? 13:21
mnasiadkaah, and nova-compute13:21
kevkomanila ? 13:21
mnasiadkawell then let's drop containers13:21
mnasiadka:)13:21
kevkowe already has bunch of containers host's /dev/shm accessible 13:21
kevkojust saying .. :D 13:22
SvenKieskevarious neutron stuff also has quite some host access (necessary for SRIOV etc) afaik13:22
mnasiadkaSounds like a lot of work13:22
mnasiadkaMaybe it's easier to write a container-friendly queue_manager? :)13:22
SvenKieskeit's just inherent complexity, the k8s deployment tools are basically doing the same thing13:22
mnasiadkaso then it probably means we need to identify which containers need to have "/dev/shm" mounted13:24
kevkoContainerization is still great for delivering code; we don't necessarily have to use everything. Moreover, we also copy config files into the container instead of using binds as it should be... for example.13:24
mnasiadkaand that's it13:24
kevkoI just want to say that we, as Kolla, definitely don't do everything the way it should be done... and with the complexity of OpenStack, I don't think it's even possible.13:25
mnasiadkawell, binds have the problem of permissions13:25
mnasiadkaso probably that's why the project started with this13:26
mnasiadkaanyway, we have probably other priorities this cycle13:26
kevkoAnyway, I will work on it and track it in Gerrit. It's probably not worth discussing it much. I’d rather ask about ProxySQL. I’ve fixed everything, including IPv6, and I think it’s OK to switch.13:27
mnasiadkaMariaDB bump is not happening, unless they release a new version of 11.4 where they fixed SST between old MariaDB and new MariaDB13:27
mnasiadkaWell, I think I commented we need to remove the unused mariadb-clustercheck running container in upgrade tasks13:28
mnasiadkaif we're changing the default13:29
kevkomnasiadka: yeah, I replied that it can be used by external scripts maybe ... and in reno it's specified that it can be removed manually 13:29
kevkomnasiadka: but I agree ... i will write something in followup ..so i will add on the top of relation chain ..13:30
mnasiadkaok then13:30
kevko+113:30
mnasiadkaI've added my +213:30
mnasiadkahttps://review.opendev.org/c/openstack/kolla-ansible/+/91372413:31
mnasiadkaDo we need to backport any of the fixes you had to do?13:31
kevkoyep ..we should one 13:32
kevkothis one https://review.opendev.org/c/openstack/kolla-ansible/+/929848 only 13:32
kevkoit's just ipv6 ...13:32
kevkonothing more ..because that dance with --initial and removal of db file << ..this was only in master ...lower versions still working fine ...13:33
kevkoand maybe this ... https://review.opendev.org/c/openstack/kolla-ansible/+/930077/4 << but this sounds like a feature .... 13:33
kevkomnasiadka: thanks 13:34
opendevreviewMerged openstack/kolla-ansible stable/2024.1: Fix links in docs for unmaintained releases  https://review.opendev.org/c/openstack/kolla-ansible/+/92857913:34
opendevreviewMerged openstack/kolla-ansible stable/2023.2: Fix links in docs for unmaintained releases  https://review.opendev.org/c/openstack/kolla-ansible/+/92858113:34
opendevreviewMerged openstack/kolla-ansible stable/2023.1: Fix links in docs for unmaintained releases  https://review.opendev.org/c/openstack/kolla-ansible/+/92858713:35
mnasiadkafrickler, SvenKieske : around backporting 930077 - it would be useful for my $downstream - are you fine with doing that?13:35
mnasiadkadoesn't include any inventory changes, so won't include any problems hopefully13:35
kevkonice to hear that you also using ProxySQL ;-) 13:36
mnasiadkaNot yet, but planning to.13:36
kevkoah, ok 13:36
SvenKieskewe need an exception for those, no? via ML? but looks okayish I think13:36
mnasiadkaAnd monitoring is an important aspect13:36
mnasiadkakevko: propose the backports please for the ipv6 and the prometheus endpoint via Gerrit and we'll sort it then13:37
kevkoML ? what is the shortcut for ? :D 13:37
mnasiadkamailing list13:37
kevkoah ok 13:37
mnasiadkabut no, never did it that way13:37
kevkomnasiadka: ack 13:37
mnasiadka#topic Additional agenda (from whiteboard)13:37
mnasiadkamhiner - please review: Move to high level client in DockerWorker https://review.opendev.org/c/openstack/kolla-ansible/+/90829513:38
opendevreviewGrzegorz Koper proposed openstack/kolla-ansible stable/2023.1: Fixing typo in etc/kolla/globals.yml  https://review.opendev.org/c/openstack/kolla-ansible/+/93046613:38
mnasiadkamhiner: commented, will also do a proper review13:39
mhinerthanks13:39
opendevreviewGrzegorz Koper proposed openstack/kolla-ansible stable/2023.2: Fixing typo in etc/kolla/globals.yml  https://review.opendev.org/c/openstack/kolla-ansible/+/93046713:39
mnasiadkar-krcek:  https://review.opendev.org/c/openstack/kolla-ansible/+/92311013:39
opendevreviewGrzegorz Koper proposed openstack/kolla-ansible stable/2024.1: Fixing typo in etc/kolla/globals.yml  https://review.opendev.org/c/openstack/kolla-ansible/+/93046813:40
mnasiadkaok, commented - would be nice to get a python based CLI this cycle I guess13:41
mnasiadkaSvenKieske: I've seen you did some reviews - can you help r-krcek to get those over the line?13:42
mnasiadkaAnother one from r-krcek - https://review.opendev.org/c/openstack/kolla-ansible/+/59973513:42
mnasiadkareviewed13:43
SvenKieskeyeah, probably over the next days, still stuck in upgrading stuff, sadly have considerable less time currently during working hours for reviews :-/13:43
SvenKieskewill make some room for reviews in my free time I guess13:44
mnasiadkaLast one is a question if anybody is interested in Swift and would like to help defining tests in https://review.opendev.org/c/openstack/kolla-ansible/+/79749813:44
mnasiadkaIf yes, wuchunyang and r-krcek would be probably grateful13:46
mnasiadka#topic Open discussion13:46
mnasiadkaanything else to cover today?13:46
r-krcekThank you guys for the reviews and comments :) 13:47
kevkoi would like to ask again for some reviews ...but you know which one right :D 13:48
mnasiadkar-krcek: if you are interested in being a more permanent part of Kolla community - given your not small contributions - you could review other people patches and become a core some day13:49
mnasiadkakevko: yeah, will have a look on your patches tomorrow13:49
* frickler will be on holiday for the coming three weeks, just fyi. will be back in time for the ptg13:50
mnasiadkaRemember PTG for Kolla starts on Tue - I already booked the slots 13:50
fricklerack13:50
mnasiadka#link https://etherpad.opendev.org/p/kolla-epoxy-ptg13:50
kevkoack13:50
mnasiadkaPlease try to populate the topics13:50
mnasiadkawe have 2x3 hours13:50
mnasiadka(and there's kayobe as well)13:51
mnasiadkajovial: ^^13:51
mnasiadkaOk then, that's all for me - see you next week (have fun frickler !)13:51
jovialawesome - I'll populate a few topics. Would definitely appreciate any input there.13:51
SvenKieskesee you :)13:51
mmalchuko/13:52
mnasiadka#endmeeting13:52
opendevmeetMeeting ended Wed Sep 25 13:52:20 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)13:52
opendevmeetMinutes:        https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-09-25-13.00.html13:52
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-09-25-13.00.txt13:52
opendevmeetLog:            https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-09-25-13.00.log.html13:52
r-krcekmnasiadka: That would be great. Is there a guide on how to become reviewer? :) Or could I just choose where to help?13:52
mmalchuksorry late13:52
mmalchukmnasiadka thanks13:52
SvenKieskethx for running the meeting mnasiadka, as always13:52
mmalchukcores please review: https://review.opendev.org/c/openstack/kolla-ansible/+/930170 and https://review.opendev.org/c/openstack/kolla-ansible/+/86925213:53
mnasiadkar-krcek: in the Kolla whiteboard (https://etherpad.opendev.org/p/KollaWhiteBoard#L207) there are links to review dashboards - basically review anything that is active and fresh, what we value is sensible comments, +1 without comments are also fine - but more input the better ;-)13:53
kevkothx13:53
mmalchukfrickler please review https://review.opendev.org/c/openstack/kolla-ansible/+/86925213:53
r-krcekmnasiadka: Thank you for the tip, Ill take a look 13:54
opendevreviewGrzegorz Koper proposed openstack/kolla-ansible stable/2023.1: Fixing typo in etc/kolla/globals.yml  https://review.opendev.org/c/openstack/kolla-ansible/+/93046613:54
mnasiadkar-krcek: in moments of doubt - don't hesitate to reach out to me or other cores :)13:54
r-krcekmnasiadka: Thanks, in the beginning Ill definitely have to sometimes :)13:55
opendevreviewGrzegorz Koper proposed openstack/kolla-ansible stable/2023.2: Fixing typo in etc/kolla/globals.yml  https://review.opendev.org/c/openstack/kolla-ansible/+/93046713:55
opendevreviewGrzegorz Koper proposed openstack/kolla-ansible stable/2024.1: Fixing typo in etc/kolla/globals.yml  https://review.opendev.org/c/openstack/kolla-ansible/+/93046813:56
Pcmalih__Hi, kolla ansible team. 14:12
opendevreviewMaksim Malchuk proposed openstack/ansible-collection-kolla master: Drop OpenEuler host OS support (due to no python3.10+)  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/93047214:27
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: Drop OpenEuler support  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/93047514:28
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: Drop OpenEuler support  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/93047514:30
mnasiadkammalchuk: mine had a reno - added you as a co-author ;-)14:30
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: Drop OpenEuler support  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/93047514:43
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: Drop OpenEuler support  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/93047514:43
mnasiadkaSvenKieske, frickler, bbezak : I believe https://review.opendev.org/c/openstack/kolla/+/907589 is ready for reviews14:58
opendevreviewMerged openstack/ansible-collection-kolla master: Add installation of dbus package  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/92261414:58
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla stable/2024.1: Add installation of dbus package  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/93048415:00
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla stable/2023.2: Add installation of dbus package  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/93048515:01
fricklerSvenKieske: mnasiadka: I'm still not convinced about https://review.opendev.org/c/openstack/kolla-ansible/+/869252, added some comments, but since I'll be away for a bit I dropped my -2 and will leave it in your hands15:36
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla stable/2023.2: Add installation of dbus package  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/93048516:41
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/2024.1: fluentd: don't check labels in check_mode  https://review.opendev.org/c/openstack/kolla-ansible/+/93049616:48
opendevreviewMerged openstack/ansible-collection-kolla stable/2024.1: Add installation of dbus package  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/93048418:00
stanI just watched a video about http driver for oslo20:36
stanmessaging and could not be more happy.. Rabbitless Openstack, finally seems possible20:36
kevkostan: something new for me :) 21:05
kevkostan: is it this ? https://opendev.org/openstack/oslo-specs/commit/b440f3dccd2edf6e55a5384b0364be69a291a8a2 << with consul stuff ? 21:05
kevkostan: very interisting if there is a consul stuff around ...i've asked some weeks ago to implement consul into kolla for masakari :D ... not approved :D 21:09

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