Friday, 2023-10-06

mnasiadkagreatgatsby: that should not happen I guess, but thanks for the heads up04:43
opendevreviewMerged openstack/kolla stable/yoga: [yoga-only]: Fix erlang versions  https://review.opendev.org/c/openstack/kolla/+/89735205:27
opendevreviewPierre Riteau proposed openstack/kayobe master: Add timing information to Ansible output  https://review.opendev.org/c/openstack/kayobe/+/89751707:04
opendevreviewPierre Riteau proposed openstack/kayobe master: CI: Add timing information to Ansible output  https://review.opendev.org/c/openstack/kayobe/+/89751707:05
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Remove keystone admin port leftovers  https://review.opendev.org/c/openstack/kolla-ansible/+/89752708:54
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: Remove keystone admin leftovers  https://review.opendev.org/c/openstack/kolla-ansible/+/87565208:56
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: Remove keystone admin leftovers  https://review.opendev.org/c/openstack/kolla-ansible/+/87565208:57
opendevreviewSven Kieske proposed openstack/kolla master: bump td-agent lts from v4 to v5  https://review.opendev.org/c/openstack/kolla/+/89494809:11
SvenKieskedoes anybody in here feel very familiar with fluentd/td-agent?09:38
SvenKieskebecause we install a list of plugins here (sorry for GH link): https://github.com/openstack/kolla/blob/a1261160da74e39b066401e315f19134d5b5acf6/docker/fluentd/Dockerfile.j2#L4809:38
SvenKieskeand some of those are bundled in the new fluentd release, see here for the list: https://github.com/fluent/fluent-package-builder/blob/b4b7017c09d78f2c404db62612733b40f69668a6/CHANGELOG.md#bundled-plugins-and-gems09:39
SvenKieskeso I wondered if: a) this can cause conflicts if we manually install plugins which are "bundled" (whatever that means in ruby land) b) if it's better to manually install so we get newer versions c) if we don't get newer versions by manual installation and d) if we should just omit the bundled plugins?09:40
SvenKieskemnasiadka: I think|hope https://review.opendev.org/c/openstack/kolla/+/894948 is finally ready for review, the centos|aarch64 builds where already succeeding and I think I fixed all the path/user/group issues for debian/ubuntu09:44
mnasiadkaSvenKieske: will have a look later today, but for me we could stop manually installing anything09:53
mnasiadkaand revisit that when it's required09:53
SvenKieskewell yeah, but I guess that's a case of "chesterton's fence" if you're familiar with that: https://en.wikipedia.org/wiki/G._K._Chesterton#Chesterton's_fence10:08
SvenKieskethat page could use an update on the use in software engineering10:08
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: nova: Stop using deprecated api_servers for glance  https://review.opendev.org/c/openstack/kolla-ansible/+/89753510:08
kevkoSvenKieske, mnasiadka: can we start to check distro inside the image instead ENV injected by kolla ? 10:47
kevkoi mean KOLLA_BASE_DISTRO 10:47
kevkoi am not saying that in current fluentd review ...but maybe followup ..and maybe for all images 10:48
kevkowhy ? because sometimes we replace one service with another distro ... (for example we have libvirt from centos ) ...10:49
kevkofor the majority of the images it doesn't depends ...but for some images it depends on KOLLA_BASE_DISTRO ...and it's set globally 10:50
kevkoso we should provide this variable for every role so it can be overriden ...OR which i like more ..don't rely on kolla-ansible 10:50
kevkobecause we can check if it is centos/ubuntu/debian/rocky from the image 10:50
kevkofrom /etc/os-release ..or lsb ..or wharever ...OR we can safely create /etc/kolla-distro or something ...10:51
kevkothen we don't need to rely on that ENV var10:51
SvenKieskemhm, good question :)10:52
SvenKieskeregarding fluent, it already failed again10:52
kevkoSvenKieske: i will check fluentd also when i will have some time 10:58
SvenKieskecould be some mirror issue, it's apparently downloading stuff from heroku.. maybe the gems or something, don't know if these where bundled previously#10:59
kevkobtw, why we are not using fluent-bit ? 11:00
SvenKieskedon't know, what is that?11:01
kevkohttps://logz.io/blog/fluentd-vs-fluent-bit/11:03
kevkoruby absence is quite sexy :) 11:03
SvenKieskelooks nice, also CNCF project, still corporate, but not a single corporation11:04
guesswhat[m]vector is far better than fluentbit, btw13:08
guesswhat[m]mnasiadka: will You take care of https://review.opendev.org/c/openstack/kolla-ansible/+/888496 ? If so .. I can try to test Prometheus integration in Skyline https://bugs.launchpad.net/skyline-apiserver/+bug/2002902/comments/213:08
opendevreviewWill Szumski proposed openstack/kolla-ansible master: Support exposing prometheus_server externally  https://review.opendev.org/c/openstack/kolla-ansible/+/82869513:21
mnasiadkaguesswhat[m]: I thought it's a shame not to merge it ;)13:21
opendevreviewWill Szumski proposed openstack/kolla-ansible master: Support exposing prometheus_server externally  https://review.opendev.org/c/openstack/kolla-ansible/+/82869513:23
opendevreviewWill Szumski proposed openstack/kolla-ansible master: CI: Adds a kolla-variables parent job  https://review.opendev.org/c/openstack/kolla-ansible/+/89754813:23
opendevreviewMichal Arbet proposed openstack/ansible-collection-kolla master: Add Podman support  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/85224013:33
mnasiadkakevko: libvirt from centos?13:40
kevkoyep 13:42
kevkomnasiadka: yep13:42
kevkounfortunatelly :( 13:43
mnasiadkakevko: that's not something we officially support13:43
kevkoand what ? :D 13:43
mnasiadkaJust saying you're on your own :)13:43
mnasiadkaKOLLA_BASE_DISTRO is used in gazilion of places13:43
kevkowait wait 13:43
kevkokolla is big overrideable dict :D 13:43
mnasiadkadoesn't say everything is tested and ,,supported''13:44
kevkoso if you override your images in globals ... you have centos libvirt 13:44
mnasiadkaand all problems are your own :)13:44
kevkowell, KOLLA_BASE_DISTRO is bad by design 13:44
kevkoto be used globally 13:44
mnasiadkajust like running Zed on CentOS 7, because why not :)13:44
mnasiadkakevko: if you start negating everything, then you better provide better solutions and implement them :)13:45
kevkoof course I will 13:45
kevkoimagine that rabbitmq or ovn is newer version in bookworm ..and you have old ubuntu 13:46
kevkoor old centos 13:46
kevkoso, i think kolla should support replace/upgrade service to some other version 13:46
kevkoif bookworm is version i like ...i just upgrade 13:46
kevkothat'is possible also now ..but not for all services ..because of KOLLA_BASE_DISTRO 13:47
kevkoalso, we should remove this dependency in longterm ..if we sometimes in a future just take one distro 13:47
kevkodebian/alpine/whatever_simple_and_minimal13:48
mnasiadkaWe discussed the infra container versions previously13:49
mnasiadkaone distro is not going to fly as long as StackHPC relies on Kolla13:49
mnasiadkawe've had that discussion before, and it would be better if you would be present on the PTG next time when this is discussed instead of bringing in those ideas outside of PTG or even outside of weekly meetings13:50
opendevreviewWill Szumski proposed openstack/kolla-ansible master: Support exposing prometheus_server externally  https://review.opendev.org/c/openstack/kolla-ansible/+/82869514:27
opendevreviewWill Szumski proposed openstack/kolla-ansible master: Support exposing prometheus_server externally  https://review.opendev.org/c/openstack/kolla-ansible/+/82869514:28
opendevreviewWill Szumski proposed openstack/kolla-ansible master: Support exposing prometheus_server externally  https://review.opendev.org/c/openstack/kolla-ansible/+/82869514:39
kevkoMnasiadka: I was on that ptg :) and I remember this discussion 14:52
mnasiadkakevko: so why raising that again? :) we have customers that want to have distros in line with what they have on the host os to pass some certifications14:53
kevkoAnyway, I think several distros is cool 14:53
kevkoBut it would by nice to not rely on injected env 14:54
kevkoBtw, isn't this already fixed ? 14:55
kevkohttps://d11d69999bcdac29c8fb-3dea60a35fb0d38e41c535f13b48e895.ssl.cf5.rackcdn.com/799229/97/check/kolla-ansible-debian-podman/b1299bb/primary/logs/ansible/deploy-prechecks14:55
kevkoNo module found: docker 14:55
mnasiadkaI think that was merged14:59
mnasiadkabut no time to analyse now, it's 5pm on Friday ;)14:59
kevkoYeah, I am just following zuul logs while drinking beer :) 14:59
opendevreviewChing Kuo proposed openstack/kolla-ansible master: Enable 3.12 Required Feature Flags for RabbitMQ  https://review.opendev.org/c/openstack/kolla-ansible/+/89756715:27
SvenKieskekevko: left some notes in https://review.opendev.org/c/openstack/ansible-collection-kolla/+/85224015:34
opendevreviewPierre Riteau proposed openstack/kayobe master: Speed up migrate_rabbitmq_queues  https://review.opendev.org/c/openstack/kayobe/+/89749319:28
opendevreviewPierre Riteau proposed openstack/kayobe master: Speed up migrate_rabbitmq_queues  https://review.opendev.org/c/openstack/kayobe/+/89749319:31
opendevreviewMerged openstack/kayobe master: Add retries to get_url and uri tasks  https://review.opendev.org/c/openstack/kayobe/+/88093121:51
opendevreviewMerged openstack/kayobe master: Use importlib.metadata instead of importlib_metadata  https://review.opendev.org/c/openstack/kayobe/+/89730623:59

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