| *** edebeste7 is now known as edebeste | 06:18 | |
| opendevreview | Maksim Malchuk proposed openstack/kolla-ansible stable/xena: Fix Bash variable expansion issues in openrc file https://review.opendev.org/c/openstack/kolla-ansible/+/883230 | 08:27 |
|---|---|---|
| opendevreview | Maksim Malchuk proposed openstack/kolla-ansible master: Fix the Cyborg service https://review.opendev.org/c/openstack/kolla-ansible/+/883495 | 08:55 |
| mnasiadka | good morning | 08:55 |
| mnasiadka | frickler: https://review.opendev.org/c/openstack/releases/+/883634 - we can update the cycle highlights when doing the GA release, right? | 08:55 |
| frickler | mnasiadka: I honestly have no idea how that works, that's why I was asking, maybe confirm with the release team to be sure | 08:58 |
| frickler | how far are we with branching ansible-collections, too? | 08:58 |
| mnasiadka | frickler: that's what I always did, and they were fine with it | 08:58 |
| frickler | https://review.opendev.org/c/openstack/ansible-collection-kolla/+/883054 can be abandoned now? | 08:59 |
| mnasiadka | done | 09:02 |
| clayjar | Is there a way to run kolla-ansible plays quicker or in parallel for the gather_facts? With increasing number of hosts, 'TASK [Gather facts] ...' is getting quite slow, connecting to each node in turn to gather. | 09:03 |
| mnasiadka | tried fact caching? | 09:03 |
| mnasiadka | and there's this - https://docs.openstack.org/kolla-ansible/latest/user/ansible-tuning.html | 09:04 |
| clayjar | Excellent, thank you. I'll check it out. | 09:05 |
| frickler | mnasiadka: it seems https://review.opendev.org/c/openstack/kolla/+/880809 breaks opensearch, because it now expects the config in /etc/opensearch instead of /usr/share/bla. too bad no CI job ran on it | 13:04 |
| frickler | (at least on ubuntu, didn't check other distros) | 13:05 |
| mnasiadka | frickler: oh, nice find - I can have a look tomorrow - but yes, we should run opensearch jobs on kolla changes | 13:05 |
| SvenKieske | there's even a bugreport, I just noticed: https://bugs.launchpad.net/kolla-ansible/+bug/2020152 | 13:09 |
| frickler | nice | 13:09 |
| mnasiadka | frickler: actually we have those jobs, but in experimental - should we move them out of experimental? | 13:09 |
| frickler | looks like that might be helpful. ppl will still ignore n-v jobs, but at least it may make locating regressions retrospectively easier | 13:10 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: CI: Move useful rocky/ubuntu jobs out of experimental https://review.opendev.org/c/openstack/kolla/+/883937 | 13:11 |
| mnasiadka | We might assess adding voting to more jobs | 13:12 |
| * SvenKieske tries to wrap his head around the concept of non voting ci jobs. | 13:15 | |
| SvenKieske | are these especially flaky and fail a lot for unrelated reasons? | 13:16 |
| frickler | yes, that's the idea | 13:16 |
| frickler | https://zuul.opendev.org/t/openstack/build/4f0d44083d9246d1845996a244fb60a8 now shows the failure in CI | 13:17 |
| SvenKieske | so, how do we fix this dockerfile? | 13:39 |
| mnasiadka | let me have a look | 13:54 |
| SvenKieske | for reference, this is the upstream dockerfile for production releases: https://github.com/opensearch-project/opensearch-build/blob/main/docker/release/dockerfiles/opensearch.al2.dockerfile | 13:57 |
| mnasiadka | but we're installing from the rpm/deb packages now, which might be different | 13:59 |
| mnasiadka | well actually I'm amazed it worked at all in the past | 14:07 |
| mnasiadka | and might be unrelated to the patch | 14:08 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: opensearch-dashboards: Fix permissions https://review.opendev.org/c/openstack/kolla/+/883941 | 14:15 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: opensearch-dashboards: Fix permissions https://review.opendev.org/c/openstack/kolla/+/883941 | 14:17 |
| mnasiadka | that's one thing | 14:17 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: opensearch: alter path after using rpm/deb packaging https://review.opendev.org/c/openstack/kolla-ansible/+/883942 | 14:22 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: opensearch-dashboards: Fix permissions https://review.opendev.org/c/openstack/kolla/+/883941 | 14:27 |
| mnasiadka | ok, let's see how that works, because that's probably a better path than proposed in the bug | 14:31 |
| mnasiadka | but we'll need a reno for the changed log path for dashboards | 14:31 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: opensearch: alter path after using rpm/deb packaging https://review.opendev.org/c/openstack/kolla-ansible/+/883942 | 14:45 |
| mnasiadka | frickler: seems the container engine scripts broke us as well (Invalid container engine in https://07d0a64b8dc56d1f446f-58126eca793a2b87b83ccf22d3c107a6.ssl.cf5.rackcdn.com/883937/1/check/kolla-ansible-rocky9-upgrade-cephadm/58399d3/job-output.txt) | 14:49 |
| mnasiadka | fantastic, no logs, ugh | 15:08 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Revert "Add container engine option to scripts" https://review.opendev.org/c/openstack/kolla-ansible/+/883914 | 15:08 |
| mnasiadka | I'll have a look tomorrow if we can just fix it instead of reverting... | 15:09 |
| grami[m] | Hi all, noob here with git review. Any good cli example to create backports and can I create backports on a review someone else owns? | 15:42 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: fix logs - add container_engine to post https://review.opendev.org/c/openstack/kolla-ansible/+/883948 | 15:44 |
| mnasiadka | grami[m]: sure - you can just use the gerrit UI - click in top right corner (three dots) and choose cherry pick | 15:45 |
| mnasiadka | or use git cherry-pick in command line | 15:45 |
| mnasiadka | make sure you're cherry picking merged changes | 15:45 |
| grami[m] | mnasiadka Ah ok so I should only create once it's merged to master? | 15:47 |
| grami[m] | I'm looking to make sure this review https://review.opendev.org/c/openstack/kolla-ansible/+/876039 gets backported to stable zed | 15:48 |
| opendevreview | Graeme Moss proposed openstack/kolla-ansible stable/zed: the fluentd regexp is wrong, it can not collect log https://review.opendev.org/c/openstack/kolla-ansible/+/883915 | 15:50 |
| opendevreview | Graeme Moss proposed openstack/kolla-ansible master: the fluentd regexp is wrong, it can not collect log https://review.opendev.org/c/openstack/kolla-ansible/+/876039 | 15:56 |
| opendevreview | Graeme Moss proposed openstack/kolla-ansible master: the fluentd regexp is wrong, it can not collect log https://review.opendev.org/c/openstack/kolla-ansible/+/876039 | 16:01 |
| opendevreview | Graeme Moss proposed openstack/kolla-ansible master: the fluentd regexp is wrong, it can not collect log https://review.opendev.org/c/openstack/kolla-ansible/+/876039 | 16:02 |
| opendevreview | Dr. Jens Harbott proposed openstack/kolla-ansible master: Fix log collection https://review.opendev.org/c/openstack/kolla-ansible/+/883950 | 16:14 |
| opendevreview | Graeme Moss proposed openstack/kolla-ansible master: the fluentd regexp is wrong, it can not collect log https://review.opendev.org/c/openstack/kolla-ansible/+/876039 | 16:15 |
| frickler | mnasiadka: ^^ maybe it's as simple as that, will check more once results are in | 16:15 |
| mnasiadka | frickler: yeah, uploaded similar change earlier - but your has a better commit message :) | 16:28 |
| opendevreview | Graeme Moss proposed openstack/kolla-ansible master: Fix the fluentd regexp to collect the logs https://review.opendev.org/c/openstack/kolla-ansible/+/876039 | 17:01 |
| opendevreview | Léo Gillot-Lamure proposed openstack/kolla-ansible master: loadbalancer: support setting nbthread with variable haproxy_threads https://review.opendev.org/c/openstack/kolla-ansible/+/883559 | 21:29 |
| opendevreview | Léo Gillot-Lamure proposed openstack/kolla-ansible master: loadbalancer: support cpu-map for threads https://review.opendev.org/c/openstack/kolla-ansible/+/883560 | 21:29 |
| opendevreview | Léo Gillot-Lamure proposed openstack/kolla-ansible master: loadbalancer: remove support for haproxy_processes https://review.opendev.org/c/openstack/kolla-ansible/+/883982 | 21:29 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!