ravlew | One +2 needed for ironic-conductor backports: https://review.opendev.org/c/openstack/kolla/+/930258 | 07:24 |
---|---|---|
ravlew | https://review.opendev.org/c/openstack/kolla/+/930259 | 07:24 |
ravlew | https://review.opendev.org/c/openstack/kolla/+/930260 | 07:24 |
SvenKieske | done | 07:26 |
mnasiadka | morning | 07:27 |
mnasiadka | SvenKieske, 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 Noble | 07:28 |
bbezak | I think worth a bug for backporting mnasiadka . morning | 07:29 |
bbezak | (if needed) | 07:30 |
SvenKieske | I wanted to ask about backporting indeed, currently reviewing, because frickler -1 backportvoted (probably due to the missing bugreport?) | 07:30 |
SvenKieske | as this interface is merged in keystone since 2017 a backport wouldn't hurt, it's not a new feature. | 07:31 |
frickler | hmm, I'd be willing to agree to that, yes | 07:31 |
frickler | mnasiadka: do you have a pointer to the octavia failure? | 07:31 |
frickler | also, does this need a reno for people who override their config? | 07:32 |
mnasiadka | yeah, probably needs a reno | 07:33 |
SvenKieske | you added one instance in https://review.opendev.org/c/openstack/kolla-ansible/+/912359 though, that seems a little off? | 07:33 |
SvenKieske | commented on it | 07:34 |
SvenKieske | was that missing previously? | 07:34 |
mnasiadka | frickler: https://zuul.opendev.org/t/openstack/build/b5c27b94d0a14e3d8ac2d8edaa828bf0/log/primary/logs/kolla/octavia/octavia-api.txt#513 | 07:35 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: octavia: Use valid_interfaces instead of endpoint_type https://review.opendev.org/c/openstack/kolla-ansible/+/912359 | 07:36 |
mnasiadka | But it might be something else - but I guess endpoint_type is not even parsed now looking at keystoneauth code | 07:37 |
mnasiadka | I guess we need to treat every change individually and check if it helps in anything ;-) | 07:38 |
frickler | that's a weird failure mode. also I don't see why that would be distro-specific | 07:38 |
frickler | but I'm +2 on merging your whole stack if you rebase it | 07:39 |
SvenKieske | mhm, honestly the reviews are going a little fast for me, already found one instance where something was added that wasn't there before | 07:40 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Create cluster in Magnum jobs https://review.opendev.org/c/openstack/kolla-ansible/+/904493 | 07:40 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: magnum: Use valid_interfaces instead of endpoint_type https://review.opendev.org/c/openstack/kolla-ansible/+/912358 | 07:40 |
SvenKieske | can we not rush this? it feels rushed to me at least. | 07:41 |
mnasiadka | rebased magnum valid_interfaces on top of CI patch that really creates a Magnum cluster | 07:42 |
mnasiadka | if that passes - I guess we're good to merge | 07:42 |
mnasiadka | there's no need to rush, I'll action on octavia and magnum today and test if that doesn't break anything | 07:42 |
SvenKieske | was ther a new keystone release just now or how did you find this breakage? | 07:43 |
frickler | the stack is 6 months old, not sure that can be called rushing | 07:43 |
frickler | I do find this in upstream testing at least, too | 07:43 |
frickler | https://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%20proces | 07:43 |
frickler | s%20config%20for%20project%22')) | 07:43 |
mnasiadka | SvenKieske: I did this stack 6 months ago, when I spotted endpoint_type has been removed, and interface is deprecated :) | 07:44 |
mnasiadka | Might be it's not what is breaking anything | 07:45 |
mnasiadka | but still we should adapt to changes that have been done long time ago | 07:45 |
SvenKieske | frickler: 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 patch | 07:45 |
mnasiadka | it seems there's no endpoint_type since 2015, so I'm still amazed how anything works :) | 07:45 |
frickler | hmm, this looks broken, though | 07:48 |
mnasiadka | Looking at codesearch.opendev.org only heat has entries in it's code for endpoint_type | 07:48 |
frickler | 2024-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 |
frickler | valid_interfaces in group [cinder] | 07:48 |
frickler | that'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 change | 07:48 |
mnasiadka | yeah, I'll work on Octavia today and experiment what we really need, because it seems we need that in more sections? | 07:48 |
frickler | hmm, octavia.conf doesn't even contain a [cinder] section. | 07:50 |
frickler | might be some regression/weird behavior from recent oslo.config? | 07:51 |
mnasiadka | might be | 07:52 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: magnum: Add k8s_cluster_api_rockylinux to CAPI drivers list https://review.opendev.org/c/openstack/kolla-ansible/+/930414 | 08:29 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Create cluster in Magnum jobs https://review.opendev.org/c/openstack/kolla-ansible/+/904493 | 08:29 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: magnum: Use valid_interfaces instead of endpoint_type https://review.opendev.org/c/openstack/kolla-ansible/+/912358 | 08:30 |
kevko | morning | 08:40 |
kevko | frickler: you sent some opensearch dashboard ..shoul I be able to login somehow ? Who is able to login ? :) | 09:09 |
opendevreview | Merged openstack/kolla stable/2024.1: Add missing iso packages to ironic-conductor https://review.opendev.org/c/openstack/kolla/+/930258 | 09:10 |
opendevreview | Merged openstack/kolla stable/2023.2: Add missing iso packages to ironic-conductor https://review.opendev.org/c/openstack/kolla/+/930259 | 09:10 |
opendevreview | Merged openstack/kolla stable/2023.1: Add missing iso packages to ironic-conductor https://review.opendev.org/c/openstack/kolla/+/930260 | 09:10 |
frickler | kevko: login is openstack/openstack , it is just in place because it seems impossible to run it without a login | 09:12 |
kevko | ah ok | 09:12 |
kevko | thanks | 09:12 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: erlang/aarch64: Move to openstack-kolla owned COPR https://review.opendev.org/c/openstack/kolla/+/930428 | 09:59 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: erlang/aarch64: Move to openstack-kolla owned COPR https://review.opendev.org/c/openstack/kolla/+/930428 | 10:13 |
PrzemekK | Somebody 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 |
opendevreview | Grzegorz Koper proposed openstack/kolla-ansible master: Documentation update, to reflect VPNaaS deployment changes https://review.opendev.org/c/openstack/kolla-ansible/+/930442 | 11:08 |
opendevreview | Grzegorz Koper proposed openstack/kolla-ansible master: Documentation update, to reflect VPNaaS(ML2/OVS) deployment changes https://review.opendev.org/c/openstack/kolla-ansible/+/930442 | 11:12 |
sylvr | Hello ! I'm trying to configure nova because of an issue of timeout during instance creation. | 11:32 |
sylvr | I 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.html | 11:34 |
sylvr | to 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 anything | 11:34 |
sylvr | did I miss something ? | 11:34 |
mnasiadka | etc/kayobe/kolla/config/nova.conf would work | 11:42 |
sylvr | mnasiadka: yuuup that would be the correct path, sorry for disturbing for nothing ^^ | 11:50 |
opendevreview | Pierre Riteau proposed openstack/ansible-collection-kolla master: Add installation of dbus package https://review.opendev.org/c/openstack/ansible-collection-kolla/+/922614 | 12:30 |
opendevreview | Jakub Darmach proposed openstack/kayobe master: Add support for Ubuntu Noble Numbat (24.04) LTS https://review.opendev.org/c/openstack/kayobe/+/930026 | 12:35 |
opendevreview | Jakub Darmach proposed openstack/kayobe master: Add support for Ubuntu Noble Numbat (24.04) LTS https://review.opendev.org/c/openstack/kayobe/+/930026 | 12:37 |
mnasiadka | SvenKieske: https://review.opendev.org/c/openstack/ansible-collection-kolla/+/922614 - willing to have another look? | 12:39 |
kevko | +1 | 12:39 |
SvenKieske | looking | 12:41 |
kevko | can I also approve things that I am sure that will pass in the future :) ? I am welcoming it | 12:47 |
SvenKieske | ah my bad, zuul only reported on patchset 2 | 12:48 |
kevko | yeah, few months old ... | 12:48 |
kevko | but it will pass i would say ... | 12:48 |
SvenKieske | this could use some GUI improvement to highlight that this is an old result :/ | 12:48 |
kevko | well, it's on the top .. | 12:49 |
kevko | as summary | 12:49 |
SvenKieske | I 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 |
kevko | ctrl+ on chrome :D | 12:51 |
kevko | *browser | 12:51 |
kevko | nevermind ... SvenKieske have you some time to check some of my patches ? | 12:51 |
SvenKieske | if they are small like the one above, sure. | 12:54 |
mnasiadka | bbezak frickler kevko SvenKieske mmalchuk gkoper jangutter jsuazo jovial osmanlicilegi mattcrees dougszu darmach - meeting in 4 minutes | 12:56 |
kevko | https://review.opendev.org/c/openstack/kolla-ansible/+/929408 https://review.opendev.org/c/openstack/kolla-ansible/+/922369 < tiny ansible-core dependency | 13:00 |
mnasiadka | #startmeeting kolla | 13:00 |
opendevmeet | Meeting 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 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 13:00 |
opendevmeet | The meeting name has been set to 'kolla' | 13:00 |
mnasiadka | #topic rollcall | 13:00 |
mnasiadka | o/ | 13:00 |
SvenKieske | o/ | 13:00 |
jovial | \o | 13:00 |
bbezak | o/ | 13:00 |
kevko | \o/ | 13:00 |
mhiner | o/ | 13:00 |
r-krcek | 0/ | 13:00 |
mnasiadka | #topic agenda... (full message at <https://matrix.org/oftc/media/v1/media/download/AfvCuoxbKukDvfMYa1XqelHiI6JA2oy1yrxh5OX-liMapP23Mk5VHpYGNwZDdFVV4TB1g_TTAeUxVm6vFfY2Ur9CeSKv1GuQAG1hdHJpeC5vcmcvSEVTTFBoWldjanVBSWtJT0xKSWlqV0xS>) | 13:01 |
mnasiadka | #topic CI status | 13:01 |
mnasiadka | Green? Red? Amber? Watermelon? | 13:01 |
frickler | o/ | 13:02 |
frickler | that agenda topic was mangled in the matrix | 13:02 |
SvenKieske | what about honeydew melon? :D | 13:02 |
mnasiadka | Yeah, switched to Element, probably it's mangling ;-) | 13:03 |
mnasiadka | I'll try better next time | 13:03 |
SvenKieske | yeah, you need to post agenda line by line for poor old IRC | 13:03 |
mnasiadka | CI looks not bad, except aarch64 | 13:03 |
mnasiadka | frickler: do you know if there's any outlook on aarch64 improvements? | 13:04 |
frickler | mnasiadka: none that I'm aware of | 13:04 |
mnasiadka | ok, it is what it is | 13:05 |
mnasiadka | #topic Current cycle planning | 13:05 |
mnasiadka | So, Ubuntu 24.04 seems to be closer, some small CI failures to revisit and I guess I'll ask for reviews next week | 13:06 |
mnasiadka | anything else major that we should be thinking of? | 13:06 |
mnasiadka | There's RMQ 4.0 | 13:06 |
mnasiadka | As in no patches, but it got released | 13:06 |
mnasiadka | Since 4.0 release - 3.13 community support has ended | 13:07 |
SvenKieske | yes, 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 stuff | 13:07 |
mnasiadka | Sounds fantastic | 13:07 |
SvenKieske | for that to work in turn, we need the queue manager which kevko is working on | 13:07 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Drop support for py38, py39 https://review.opendev.org/c/openstack/kolla-ansible/+/929408 | 13:07 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Add ansible-core as a dependency https://review.opendev.org/c/openstack/kolla-ansible/+/922369 | 13:07 |
SvenKieske | but he seems busy :P ;) | 13:07 |
kevko | with queue manager it's little bit hard ..we need to bind /dev/shm to containers using oslo.messaging | 13:08 |
mnasiadka | is that secure? | 13:08 |
SvenKieske | just..bind mount it? :) /dev/shm is actually just a fancy way to say "tmpfs". | 13:08 |
kevko | well, i was thinking about the option to create tmpfs and mount as /dev/shm into container | 13:09 |
SvenKieske | I hope oslo has guarded it and there are no race conditions with concurrent access :D | 13:09 |
kevko | but then I realized we have already some containers which has /dev/shm already mounted | 13:09 |
mnasiadka | Isn't --ipc=host for that? | 13:09 |
kevko | and ...moreover ...in classic openstack installations from packages ..there is shared memory also ... | 13:09 |
kevko | no, ipc turning more than /dev/shm | 13:09 |
kevko | but yes ..if you set ipc_mode ...it's effectively turn on bind /dev/shm into container | 13:10 |
kevko | but we don't need it ... | 13:10 |
mnasiadka | there's the alternative of shared ipc for containers | 13:10 |
mnasiadka | but that was a bit... tricky, because you start a container pointing to other running containers ipc | 13:11 |
frickler | maybe we should stop doing containers if we share everything again anyway in the end? (almost, but not completely non-serious) | 13:11 |
SvenKieske | I think the short term conclusion is we need some time before we can truly merge rmq 4? | 13:11 |
mnasiadka | yeah, that's another point | 13:11 |
mnasiadka | if we need shared mem for all containers, why even containers? | 13:11 |
mnasiadka | so, is that bloody queue manager completely required for RMQ 4? | 13:12 |
SvenKieske | well if we wanna split hairs it's afaik not required for rmq, but for how oslo/openstack uses rmq :) | 13:13 |
kevko | frickler: 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 |
mnasiadka | the oslo.messaging patch says it's optional | 13:14 |
kevko | frickler: But yes, I understand your comment.. :) | 13:14 |
SvenKieske | what 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 queue | 13:14 |
kevko | mnasiadka: 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 |
SvenKieske | xD | 13:15 |
kevko | and it's also very nice feature :) | 13:15 |
mnasiadka | ok, but it doesn't sound like required | 13:15 |
SvenKieske | well for first release we could put big fat warning that queue manager is not yet supported? if it works? | 13:15 |
kevko | so should we add precheck if user override includes queue_manager = true and fail the reconfigure ? :D | 13:15 |
mnasiadka | user overrides mean user knows what he's doing | 13:16 |
kevko | but agree that it's not important issue for now | 13:16 |
mnasiadka | we never checked what is in user overrides | 13:16 |
kevko | don't forget it's also requirement for streams for fanout | 13:16 |
mnasiadka | so let's not create a drama | 13:16 |
kevko | agree ... | 13:16 |
kevko | it's mainly my downstream project which i am (as always) tracking in gerrit ... and I like that feature | 13:17 |
mnasiadka | Well, if we can create a tmpfs volume and mount that as shm, that would probably be a bit better | 13:18 |
kevko | That'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 |
mnasiadka | well, we only run libvirt with ipc=host IIRC | 13:20 |
mnasiadka | that doesn't interact with RMQ | 13:20 |
kevko | hacluster ? | 13:20 |
kevko | cinder ? | 13:20 |
kevko | masakari ? | 13:21 |
mnasiadka | ah, and nova-compute | 13:21 |
kevko | manila ? | 13:21 |
mnasiadka | well then let's drop containers | 13:21 |
mnasiadka | :) | 13:21 |
kevko | we already has bunch of containers host's /dev/shm accessible | 13:21 |
kevko | just saying .. :D | 13:22 |
SvenKieske | various neutron stuff also has quite some host access (necessary for SRIOV etc) afaik | 13:22 |
mnasiadka | Sounds like a lot of work | 13:22 |
mnasiadka | Maybe it's easier to write a container-friendly queue_manager? :) | 13:22 |
SvenKieske | it's just inherent complexity, the k8s deployment tools are basically doing the same thing | 13:22 |
mnasiadka | so then it probably means we need to identify which containers need to have "/dev/shm" mounted | 13:24 |
kevko | Containerization 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 |
mnasiadka | and that's it | 13:24 |
kevko | I 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 |
mnasiadka | well, binds have the problem of permissions | 13:25 |
mnasiadka | so probably that's why the project started with this | 13:26 |
mnasiadka | anyway, we have probably other priorities this cycle | 13:26 |
kevko | Anyway, 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 |
mnasiadka | MariaDB bump is not happening, unless they release a new version of 11.4 where they fixed SST between old MariaDB and new MariaDB | 13:27 |
mnasiadka | Well, I think I commented we need to remove the unused mariadb-clustercheck running container in upgrade tasks | 13:28 |
mnasiadka | if we're changing the default | 13:29 |
kevko | mnasiadka: 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 |
kevko | mnasiadka: but I agree ... i will write something in followup ..so i will add on the top of relation chain .. | 13:30 |
mnasiadka | ok then | 13:30 |
kevko | +1 | 13:30 |
mnasiadka | I've added my +2 | 13:30 |
mnasiadka | https://review.opendev.org/c/openstack/kolla-ansible/+/913724 | 13:31 |
mnasiadka | Do we need to backport any of the fixes you had to do? | 13:31 |
kevko | yep ..we should one | 13:32 |
kevko | this one https://review.opendev.org/c/openstack/kolla-ansible/+/929848 only | 13:32 |
kevko | it's just ipv6 ... | 13:32 |
kevko | nothing more ..because that dance with --initial and removal of db file << ..this was only in master ...lower versions still working fine ... | 13:33 |
kevko | and maybe this ... https://review.opendev.org/c/openstack/kolla-ansible/+/930077/4 << but this sounds like a feature .... | 13:33 |
kevko | mnasiadka: thanks | 13:34 |
opendevreview | Merged openstack/kolla-ansible stable/2024.1: Fix links in docs for unmaintained releases https://review.opendev.org/c/openstack/kolla-ansible/+/928579 | 13:34 |
opendevreview | Merged openstack/kolla-ansible stable/2023.2: Fix links in docs for unmaintained releases https://review.opendev.org/c/openstack/kolla-ansible/+/928581 | 13:34 |
opendevreview | Merged openstack/kolla-ansible stable/2023.1: Fix links in docs for unmaintained releases https://review.opendev.org/c/openstack/kolla-ansible/+/928587 | 13:35 |
mnasiadka | frickler, SvenKieske : around backporting 930077 - it would be useful for my $downstream - are you fine with doing that? | 13:35 |
mnasiadka | doesn't include any inventory changes, so won't include any problems hopefully | 13:35 |
kevko | nice to hear that you also using ProxySQL ;-) | 13:36 |
mnasiadka | Not yet, but planning to. | 13:36 |
kevko | ah, ok | 13:36 |
SvenKieske | we need an exception for those, no? via ML? but looks okayish I think | 13:36 |
mnasiadka | And monitoring is an important aspect | 13:36 |
mnasiadka | kevko: propose the backports please for the ipv6 and the prometheus endpoint via Gerrit and we'll sort it then | 13:37 |
kevko | ML ? what is the shortcut for ? :D | 13:37 |
mnasiadka | mailing list | 13:37 |
kevko | ah ok | 13:37 |
mnasiadka | but no, never did it that way | 13:37 |
kevko | mnasiadka: ack | 13:37 |
mnasiadka | #topic Additional agenda (from whiteboard) | 13:37 |
mnasiadka | mhiner - please review: Move to high level client in DockerWorker https://review.opendev.org/c/openstack/kolla-ansible/+/908295 | 13:38 |
opendevreview | Grzegorz Koper proposed openstack/kolla-ansible stable/2023.1: Fixing typo in etc/kolla/globals.yml https://review.opendev.org/c/openstack/kolla-ansible/+/930466 | 13:38 |
mnasiadka | mhiner: commented, will also do a proper review | 13:39 |
mhiner | thanks | 13:39 |
opendevreview | Grzegorz Koper proposed openstack/kolla-ansible stable/2023.2: Fixing typo in etc/kolla/globals.yml https://review.opendev.org/c/openstack/kolla-ansible/+/930467 | 13:39 |
mnasiadka | r-krcek: https://review.opendev.org/c/openstack/kolla-ansible/+/923110 | 13:39 |
opendevreview | Grzegorz Koper proposed openstack/kolla-ansible stable/2024.1: Fixing typo in etc/kolla/globals.yml https://review.opendev.org/c/openstack/kolla-ansible/+/930468 | 13:40 |
mnasiadka | ok, commented - would be nice to get a python based CLI this cycle I guess | 13:41 |
mnasiadka | SvenKieske: I've seen you did some reviews - can you help r-krcek to get those over the line? | 13:42 |
mnasiadka | Another one from r-krcek - https://review.opendev.org/c/openstack/kolla-ansible/+/599735 | 13:42 |
mnasiadka | reviewed | 13:43 |
SvenKieske | yeah, probably over the next days, still stuck in upgrading stuff, sadly have considerable less time currently during working hours for reviews :-/ | 13:43 |
SvenKieske | will make some room for reviews in my free time I guess | 13:44 |
mnasiadka | Last 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/+/797498 | 13:44 |
mnasiadka | If yes, wuchunyang and r-krcek would be probably grateful | 13:46 |
mnasiadka | #topic Open discussion | 13:46 |
mnasiadka | anything else to cover today? | 13:46 |
r-krcek | Thank you guys for the reviews and comments :) | 13:47 |
kevko | i would like to ask again for some reviews ...but you know which one right :D | 13:48 |
mnasiadka | r-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 day | 13:49 |
mnasiadka | kevko: yeah, will have a look on your patches tomorrow | 13:49 |
* frickler will be on holiday for the coming three weeks, just fyi. will be back in time for the ptg | 13:50 | |
mnasiadka | Remember PTG for Kolla starts on Tue - I already booked the slots | 13:50 |
frickler | ack | 13:50 |
mnasiadka | #link https://etherpad.opendev.org/p/kolla-epoxy-ptg | 13:50 |
kevko | ack | 13:50 |
mnasiadka | Please try to populate the topics | 13:50 |
mnasiadka | we have 2x3 hours | 13:50 |
mnasiadka | (and there's kayobe as well) | 13:51 |
mnasiadka | jovial: ^^ | 13:51 |
mnasiadka | Ok then, that's all for me - see you next week (have fun frickler !) | 13:51 |
jovial | awesome - I'll populate a few topics. Would definitely appreciate any input there. | 13:51 |
SvenKieske | see you :) | 13:51 |
mmalchuk | o/ | 13:52 |
mnasiadka | #endmeeting | 13:52 |
opendevmeet | Meeting ended Wed Sep 25 13:52:20 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 13:52 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-09-25-13.00.html | 13:52 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-09-25-13.00.txt | 13:52 |
opendevmeet | Log: https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-09-25-13.00.log.html | 13:52 |
r-krcek | mnasiadka: That would be great. Is there a guide on how to become reviewer? :) Or could I just choose where to help? | 13:52 |
mmalchuk | sorry late | 13:52 |
mmalchuk | mnasiadka thanks | 13:52 |
SvenKieske | thx for running the meeting mnasiadka, as always | 13:52 |
mmalchuk | cores please review: https://review.opendev.org/c/openstack/kolla-ansible/+/930170 and https://review.opendev.org/c/openstack/kolla-ansible/+/869252 | 13:53 |
mnasiadka | r-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 |
kevko | thx | 13:53 |
mmalchuk | frickler please review https://review.opendev.org/c/openstack/kolla-ansible/+/869252 | 13:53 |
r-krcek | mnasiadka: Thank you for the tip, Ill take a look | 13:54 |
opendevreview | Grzegorz Koper proposed openstack/kolla-ansible stable/2023.1: Fixing typo in etc/kolla/globals.yml https://review.opendev.org/c/openstack/kolla-ansible/+/930466 | 13:54 |
mnasiadka | r-krcek: in moments of doubt - don't hesitate to reach out to me or other cores :) | 13:54 |
r-krcek | mnasiadka: Thanks, in the beginning Ill definitely have to sometimes :) | 13:55 |
opendevreview | Grzegorz Koper proposed openstack/kolla-ansible stable/2023.2: Fixing typo in etc/kolla/globals.yml https://review.opendev.org/c/openstack/kolla-ansible/+/930467 | 13:55 |
opendevreview | Grzegorz Koper proposed openstack/kolla-ansible stable/2024.1: Fixing typo in etc/kolla/globals.yml https://review.opendev.org/c/openstack/kolla-ansible/+/930468 | 13:56 |
Pcmalih__ | Hi, kolla ansible team. | 14:12 |
opendevreview | Maksim 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/+/930472 | 14:27 |
opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: Drop OpenEuler support https://review.opendev.org/c/openstack/ansible-collection-kolla/+/930475 | 14:28 |
opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: Drop OpenEuler support https://review.opendev.org/c/openstack/ansible-collection-kolla/+/930475 | 14:30 |
mnasiadka | mmalchuk: mine had a reno - added you as a co-author ;-) | 14:30 |
opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: Drop OpenEuler support https://review.opendev.org/c/openstack/ansible-collection-kolla/+/930475 | 14:43 |
opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: Drop OpenEuler support https://review.opendev.org/c/openstack/ansible-collection-kolla/+/930475 | 14:43 |
mnasiadka | SvenKieske, frickler, bbezak : I believe https://review.opendev.org/c/openstack/kolla/+/907589 is ready for reviews | 14:58 |
opendevreview | Merged openstack/ansible-collection-kolla master: Add installation of dbus package https://review.opendev.org/c/openstack/ansible-collection-kolla/+/922614 | 14:58 |
opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla stable/2024.1: Add installation of dbus package https://review.opendev.org/c/openstack/ansible-collection-kolla/+/930484 | 15:00 |
opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla stable/2023.2: Add installation of dbus package https://review.opendev.org/c/openstack/ansible-collection-kolla/+/930485 | 15:01 |
frickler | SvenKieske: 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 hands | 15:36 |
opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla stable/2023.2: Add installation of dbus package https://review.opendev.org/c/openstack/ansible-collection-kolla/+/930485 | 16:41 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible stable/2024.1: fluentd: don't check labels in check_mode https://review.opendev.org/c/openstack/kolla-ansible/+/930496 | 16:48 |
opendevreview | Merged openstack/ansible-collection-kolla stable/2024.1: Add installation of dbus package https://review.opendev.org/c/openstack/ansible-collection-kolla/+/930484 | 18:00 |
stan | I just watched a video about http driver for oslo | 20:36 |
stan | messaging and could not be more happy.. Rabbitless Openstack, finally seems possible | 20:36 |
kevko | stan: something new for me :) | 21:05 |
kevko | stan: is it this ? https://opendev.org/openstack/oslo-specs/commit/b440f3dccd2edf6e55a5384b0364be69a291a8a2 << with consul stuff ? | 21:05 |
kevko | stan: 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/!