| mnasiadka | greatgatsby: that should not happen I guess, but thanks for the heads up | 04:43 |
|---|---|---|
| opendevreview | Merged openstack/kolla stable/yoga: [yoga-only]: Fix erlang versions https://review.opendev.org/c/openstack/kolla/+/897352 | 05:27 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: Add timing information to Ansible output https://review.opendev.org/c/openstack/kayobe/+/897517 | 07:04 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: CI: Add timing information to Ansible output https://review.opendev.org/c/openstack/kayobe/+/897517 | 07:05 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Remove keystone admin port leftovers https://review.opendev.org/c/openstack/kolla-ansible/+/897527 | 08:54 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Remove keystone admin leftovers https://review.opendev.org/c/openstack/kolla-ansible/+/875652 | 08:56 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Remove keystone admin leftovers https://review.opendev.org/c/openstack/kolla-ansible/+/875652 | 08:57 |
| opendevreview | Sven Kieske proposed openstack/kolla master: bump td-agent lts from v4 to v5 https://review.opendev.org/c/openstack/kolla/+/894948 | 09:11 |
| SvenKieske | does anybody in here feel very familiar with fluentd/td-agent? | 09:38 |
| SvenKieske | because we install a list of plugins here (sorry for GH link): https://github.com/openstack/kolla/blob/a1261160da74e39b066401e315f19134d5b5acf6/docker/fluentd/Dockerfile.j2#L48 | 09:38 |
| SvenKieske | and 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-gems | 09:39 |
| SvenKieske | so 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 |
| SvenKieske | mnasiadka: 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/ubuntu | 09:44 |
| mnasiadka | SvenKieske: will have a look later today, but for me we could stop manually installing anything | 09:53 |
| mnasiadka | and revisit that when it's required | 09:53 |
| SvenKieske | well 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_fence | 10:08 |
| SvenKieske | that page could use an update on the use in software engineering | 10:08 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: nova: Stop using deprecated api_servers for glance https://review.opendev.org/c/openstack/kolla-ansible/+/897535 | 10:08 |
| kevko | SvenKieske, mnasiadka: can we start to check distro inside the image instead ENV injected by kolla ? | 10:47 |
| kevko | i mean KOLLA_BASE_DISTRO | 10:47 |
| kevko | i am not saying that in current fluentd review ...but maybe followup ..and maybe for all images | 10:48 |
| kevko | why ? because sometimes we replace one service with another distro ... (for example we have libvirt from centos ) ... | 10:49 |
| kevko | for 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 |
| kevko | so 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 |
| kevko | because we can check if it is centos/ubuntu/debian/rocky from the image | 10:50 |
| kevko | from /etc/os-release ..or lsb ..or wharever ...OR we can safely create /etc/kolla-distro or something ... | 10:51 |
| kevko | then we don't need to rely on that ENV var | 10:51 |
| SvenKieske | mhm, good question :) | 10:52 |
| SvenKieske | regarding fluent, it already failed again | 10:52 |
| kevko | SvenKieske: i will check fluentd also when i will have some time | 10:58 |
| SvenKieske | could 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 |
| kevko | btw, why we are not using fluent-bit ? | 11:00 |
| SvenKieske | don't know, what is that? | 11:01 |
| kevko | https://logz.io/blog/fluentd-vs-fluent-bit/ | 11:03 |
| kevko | ruby absence is quite sexy :) | 11:03 |
| SvenKieske | looks nice, also CNCF project, still corporate, but not a single corporation | 11:04 |
| guesswhat[m] | vector is far better than fluentbit, btw | 13: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/2 | 13:08 |
| opendevreview | Will Szumski proposed openstack/kolla-ansible master: Support exposing prometheus_server externally https://review.opendev.org/c/openstack/kolla-ansible/+/828695 | 13:21 |
| mnasiadka | guesswhat[m]: I thought it's a shame not to merge it ;) | 13:21 |
| opendevreview | Will Szumski proposed openstack/kolla-ansible master: Support exposing prometheus_server externally https://review.opendev.org/c/openstack/kolla-ansible/+/828695 | 13:23 |
| opendevreview | Will Szumski proposed openstack/kolla-ansible master: CI: Adds a kolla-variables parent job https://review.opendev.org/c/openstack/kolla-ansible/+/897548 | 13:23 |
| opendevreview | Michal Arbet proposed openstack/ansible-collection-kolla master: Add Podman support https://review.opendev.org/c/openstack/ansible-collection-kolla/+/852240 | 13:33 |
| mnasiadka | kevko: libvirt from centos? | 13:40 |
| kevko | yep | 13:42 |
| kevko | mnasiadka: yep | 13:42 |
| kevko | unfortunatelly :( | 13:43 |
| mnasiadka | kevko: that's not something we officially support | 13:43 |
| kevko | and what ? :D | 13:43 |
| mnasiadka | Just saying you're on your own :) | 13:43 |
| mnasiadka | KOLLA_BASE_DISTRO is used in gazilion of places | 13:43 |
| kevko | wait wait | 13:43 |
| kevko | kolla is big overrideable dict :D | 13:43 |
| mnasiadka | doesn't say everything is tested and ,,supported'' | 13:44 |
| kevko | so if you override your images in globals ... you have centos libvirt | 13:44 |
| mnasiadka | and all problems are your own :) | 13:44 |
| kevko | well, KOLLA_BASE_DISTRO is bad by design | 13:44 |
| kevko | to be used globally | 13:44 |
| mnasiadka | just like running Zed on CentOS 7, because why not :) | 13:44 |
| mnasiadka | kevko: if you start negating everything, then you better provide better solutions and implement them :) | 13:45 |
| kevko | of course I will | 13:45 |
| kevko | imagine that rabbitmq or ovn is newer version in bookworm ..and you have old ubuntu | 13:46 |
| kevko | or old centos | 13:46 |
| kevko | so, i think kolla should support replace/upgrade service to some other version | 13:46 |
| kevko | if bookworm is version i like ...i just upgrade | 13:46 |
| kevko | that'is possible also now ..but not for all services ..because of KOLLA_BASE_DISTRO | 13:47 |
| kevko | also, we should remove this dependency in longterm ..if we sometimes in a future just take one distro | 13:47 |
| kevko | debian/alpine/whatever_simple_and_minimal | 13:48 |
| mnasiadka | We discussed the infra container versions previously | 13:49 |
| mnasiadka | one distro is not going to fly as long as StackHPC relies on Kolla | 13:49 |
| mnasiadka | we'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 meetings | 13:50 |
| opendevreview | Will Szumski proposed openstack/kolla-ansible master: Support exposing prometheus_server externally https://review.opendev.org/c/openstack/kolla-ansible/+/828695 | 14:27 |
| opendevreview | Will Szumski proposed openstack/kolla-ansible master: Support exposing prometheus_server externally https://review.opendev.org/c/openstack/kolla-ansible/+/828695 | 14:28 |
| opendevreview | Will Szumski proposed openstack/kolla-ansible master: Support exposing prometheus_server externally https://review.opendev.org/c/openstack/kolla-ansible/+/828695 | 14:39 |
| kevko | Mnasiadka: I was on that ptg :) and I remember this discussion | 14:52 |
| mnasiadka | kevko: 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 certifications | 14:53 |
| kevko | Anyway, I think several distros is cool | 14:53 |
| kevko | But it would by nice to not rely on injected env | 14:54 |
| kevko | Btw, isn't this already fixed ? | 14:55 |
| kevko | https://d11d69999bcdac29c8fb-3dea60a35fb0d38e41c535f13b48e895.ssl.cf5.rackcdn.com/799229/97/check/kolla-ansible-debian-podman/b1299bb/primary/logs/ansible/deploy-prechecks | 14:55 |
| kevko | No module found: docker | 14:55 |
| mnasiadka | I think that was merged | 14:59 |
| mnasiadka | but no time to analyse now, it's 5pm on Friday ;) | 14:59 |
| kevko | Yeah, I am just following zuul logs while drinking beer :) | 14:59 |
| opendevreview | Ching Kuo proposed openstack/kolla-ansible master: Enable 3.12 Required Feature Flags for RabbitMQ https://review.opendev.org/c/openstack/kolla-ansible/+/897567 | 15:27 |
| SvenKieske | kevko: left some notes in https://review.opendev.org/c/openstack/ansible-collection-kolla/+/852240 | 15:34 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: Speed up migrate_rabbitmq_queues https://review.opendev.org/c/openstack/kayobe/+/897493 | 19:28 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: Speed up migrate_rabbitmq_queues https://review.opendev.org/c/openstack/kayobe/+/897493 | 19:31 |
| opendevreview | Merged openstack/kayobe master: Add retries to get_url and uri tasks https://review.opendev.org/c/openstack/kayobe/+/880931 | 21:51 |
| opendevreview | Merged openstack/kayobe master: Use importlib.metadata instead of importlib_metadata https://review.opendev.org/c/openstack/kayobe/+/897306 | 23:59 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!