| kevko | Morning | 07:47 |
|---|---|---|
| SvenKieske | o/ | 07:49 |
| kevko | Sven are u ok now with ^^^ | 07:57 |
| kevko | https://review.opendev.org/c/openstack/kolla-ansible/+/895546 | 07:58 |
| kevko | >> | 07:58 |
| Viii | hi i would like to start vitrage. But I have an error. I am using the latest code 2023.1 | 08:22 |
| Viii | msg": "Could not find or access '/etc/kolla/config/vitrage/prometheus_conf.yaml' on the Ansible Controller.\nIf you are using a module and expect the file to exist on the remote, see the remote_src option | 08:22 |
| Viii | what should be in this configuration file? any example. I can't find anything in the documentation | 08:23 |
| mnasiadka | it seems you have set enable_vitrage_prometheus_datasource to yes, I guess you should read Vitrage documentation to understand what needs to be in this file | 08:24 |
| mnasiadka | sorry, I doubt anybody here uses vitrage ;) | 08:24 |
| Viii | cat /etc/kolla/globals.yml |grep vitrage ||| #enable_horizon_vitrage: "{{ enable_vitrage | bool }}". ||| enable_vitrage: "yes" | 08:26 |
| Viii | ok, out of curiosity I check out various services | 08:27 |
| mnasiadka | seems it's enabled by default when you use prometheus - https://github.com/openstack/kolla-ansible/blob/f6d0ffd61c950416f4e521064772291e65de9623/ansible/group_vars/all.yml#L1269 | 08:27 |
| Viii | Ok, thx | 08:28 |
| kevko | SvenKieske replied | 08:28 |
| kevko | Mnasiadka: letsencrypt Kolla path comments addressed if you have a time ... | 08:29 |
| Viii | and what do you most often use to monitor openstation? | 08:29 |
| Viii | opensatck* | 08:29 |
| Viii | I guess it depends :) but such a good proven way? | 08:31 |
| SvenKieske | Viii: per default kolla-ansible deploys prometheus & grafana | 08:31 |
| opendevreview | Christian Berendt proposed openstack/kolla-ansible master: keystone: ensure bool for two parameters https://review.opendev.org/c/openstack/kolla-ansible/+/895546 | 08:31 |
| kevko | I would like to ask again SvenKieske, mnasiadka - how to deal with https://review.opendev.org/c/openstack/kolla-ansible/+/895546 for example ? It's fixing bug ....but reno missing with 'fixes' block inside ... mnasiadka gave +2 SvenKieske gave +1 | 08:49 |
| mnasiadka | kevko: we agreed we don't require bugfix renos | 08:49 |
| kevko | ok, nice to hear :) | 08:49 |
| mnasiadka | they are nice to have and probably needed on some complicated bugfixes | 08:51 |
| opendevreview | Christian Berendt proposed openstack/kolla-ansible master: keystone: ensure bool for two parameters https://review.opendev.org/c/openstack/kolla-ansible/+/895546 | 08:52 |
| mnasiadka | well, you've got your reno now :) | 08:53 |
| SvenKieske | we're being watched ;) | 08:57 |
| SvenKieske | kevko: about the let's encrypt stuff: I realize some other part writes the cert via rsync to tmp.. I lost track: is that part of the same changeset? if yes, we should alter this path, because hardcoded /tmp/ paths are the bane of my existence and the root of all evil. | 08:59 |
| SvenKieske | either way we should change that, I'll do it myself if I find the corresponding code. | 08:59 |
| kevko | SvenKieske: expand-all -> ctrl + f -> write rsync | 09:01 |
| mnasiadka | yeah, we should handle tmp properly - that's usually the first place security bugs appear ;) | 09:03 |
| SvenKieske | mhm | 09:03 |
| SvenKieske | can't docker do the same stuff, systemd does? private tmp? that would mitigate a log, still we should not write this way to tmp | 09:04 |
| SvenKieske | kevko: thx for the expand all tip, I just was going through all files manually..my coffeine storage is depleted :( | 09:04 |
| opendevreview | Jake Hutchinson proposed openstack/kayobe master: Add NTP parameter configuration https://review.opendev.org/c/openstack/kayobe/+/895199 | 09:06 |
| kevko | SvenKieske: no problem, i will move /tmp/ to /var/lib/haproxy home dir ...so give me few minute s | 09:07 |
| kevko | ansible/roles/mariadb/tasks/recover_cluster.yml: command: "cp {{ docker_runtime_directory or '/var/lib/docker' }}/volumes/kolla_logs/_data/mariadb/mariadb.log /tmp/mariadb_tmp.log" | 09:08 |
| kevko | :D | 09:08 |
| opendevreview | Dr. Jens Harbott proposed openstack/kolla-ansible master: keystone: ensure bool for two parameters https://review.opendev.org/c/openstack/kolla-ansible/+/895546 | 09:10 |
| SvenKieske | maybe I can refactor some of the tmp stuff in other scripts, guilty of this cardinal sin. | 09:13 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Rework letsencrypt https://review.opendev.org/c/openstack/kolla/+/887347 | 09:27 |
| kevko | SvenKieske: done | 09:27 |
| SvenKieske | ty | 09:33 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.1: senlin: add missing cafile parameter https://review.opendev.org/c/openstack/kolla-ansible/+/894432 | 09:48 |
| kevko | SvenKieske: anything else ? | 09:55 |
| SvenKieske | I wanted to check if the httpd binary is really always located in /usr/sbin/ on all distros - I don't know - but haven't got the time just yet :) | 10:04 |
| SvenKieske | chances are, this is correct, but I wanted to check either way, just paranoia | 10:04 |
| opendevreview | Merged openstack/kolla-ansible master: Configure Nova libvirt.num_pcie_ports to 16 by default https://review.opendev.org/c/openstack/kolla-ansible/+/768301 | 10:08 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Add support for LetsEncrypt-managed certs https://review.opendev.org/c/openstack/kolla-ansible/+/741340 | 10:08 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible stable/2023.1: Configure Nova libvirt.num_pcie_ports to 16 by default https://review.opendev.org/c/openstack/kolla-ansible/+/895371 | 10:13 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible stable/zed: Configure Nova libvirt.num_pcie_ports to 16 by default https://review.opendev.org/c/openstack/kolla-ansible/+/895372 | 10:13 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible stable/yoga: Configure Nova libvirt.num_pcie_ports to 16 by default https://review.opendev.org/c/openstack/kolla-ansible/+/895373 | 10:13 |
| opendevreview | Merged openstack/kolla-ansible master: ceilometer: process custom event_pipeline.yaml with merge_yaml https://review.opendev.org/c/openstack/kolla-ansible/+/894182 | 10:18 |
| kevko | SvenKieske: haha, instead of leave a comment you can just check another apache service in k-a ? :D | 10:18 |
| SvenKieske | kevko: right, this might have already been done elsewhere | 10:18 |
| kevko | SvenKieske: for example placement ? https://github.com/openstack/kolla-ansible/blob/master/ansible/roles/placement/templates/placement-api.json.j2 | 10:19 |
| SvenKieske | still not really awoken I guess, it was a long weekend. | 10:19 |
| kevko | :D :D :D | 10:19 |
| kevko | i fell asleep at 5 :D and at 9 i was already working :D | 10:19 |
| kevko | sleep is overrated | 10:20 |
| kevko | :D | 10:20 |
| SvenKieske | it seems to be | 10:22 |
| mnasiadka | and then after a week of that, you go to sleep at 9pm and wake up at 9pm ;-) | 10:30 |
| kevko | haha :D sometimes :D | 10:41 |
| mnasiadka | I have a feeling apache_binary and apache_conf_dir should be moved to group_vars/all.yml instead of multiplying that code everywhere | 11:05 |
| SvenKieske | that seems also like a good point | 12:17 |
| opendevreview | Merged openstack/kolla-ansible master: keystone: ensure bool for two parameters https://review.opendev.org/c/openstack/kolla-ansible/+/895546 | 12:26 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible master: Add option for extra kernel modules in neutron role https://review.opendev.org/c/openstack/kolla-ansible/+/894735 | 12:32 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible master: Add option for extra kernel modules in neutron role https://review.opendev.org/c/openstack/kolla-ansible/+/894735 | 12:32 |
| opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/2023.1: keystone: ensure bool for two parameters https://review.opendev.org/c/openstack/kolla-ansible/+/895379 | 12:54 |
| opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/zed: keystone: ensure bool for two parameters https://review.opendev.org/c/openstack/kolla-ansible/+/895380 | 12:55 |
| opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/yoga: keystone: ensure bool for two parameters https://review.opendev.org/c/openstack/kolla-ansible/+/895661 | 12:55 |
| kevko | mnasiadka: SvenKieske: no problem, but i'm not going to store apache binary value to groups vars in LE patch ...if it is another blocker for LE, ok just let me know ... give me 5 minutes and i will propose another patch for this and rebase onto it | 13:07 |
| kevko | i just wanted to say, focus on real changes which LE patch is proposing, or on functionality , let's not run into other corners which are not relevant and IF we want to change we NEED to change in another patchsets | 13:10 |
| Perdu | I'm trying to figure out what I need to do to get my (very small) patch finally merged (https://review.opendev.org/c/openstack/kolla-ansible/+/894930). Documentation seems to say I need to get attention from a core reviewer to approve merge | 13:21 |
| kevko | Perdu: two +2 and one +w | 13:29 |
| mnasiadka | kevko: no no, just saying about a possible refactor of everything, not doing that now :) | 13:34 |
| jangutter | kevko: want to save a small amount of CI resources (job unlikely to be fixed until C-series)? https://review.opendev.org/c/openstack/kolla-ansible/+/895163 | 13:34 |
| mnasiadka | Perdu: thanks for doing that, completely forgot it when we moved from wiki - it's merging now | 13:35 |
| frickler | Perdu: generally you should not need to do anything, core reviewer will find open patches by themselves and act upon them, but asking here after waiting a bit is fine, too | 13:36 |
| SvenKieske | interesting bug report from the scs lean operator call: https://bugs.launchpad.net/kolla-ansible/+bug/2033980 | 13:40 |
| SvenKieske | sounds like we should relocate that one to neutron though, no? | 13:40 |
| SvenKieske | kevko: no problem regarding LE patch, was just wondering, path is fine for me and you don't need to do anything regarding it :) | 13:41 |
| SvenKieske | Perdu: just pester frickler and mnasiadka here, that's what we do as well ;) | 13:42 |
| frickler | SvenKieske: added neutron to that bug so they can have a look | 13:43 |
| mnasiadka | it's rather radvd in the neutron agent container | 13:44 |
| opendevreview | Célestin Matte proposed openstack/kolla-ansible master: Fix ansible-lint warnings related to jinja spacing https://review.opendev.org/c/openstack/kolla-ansible/+/895684 | 13:46 |
| SvenKieske | the condition under which this happens is funny, they provided 5 dns servers to the neutron ipv6 network, then it died like described in this bug | 13:49 |
| SvenKieske | added that info to the bug | 13:49 |
| SvenKieske | but I have not reproduced myself | 13:49 |
| SvenKieske | frickler: thanks | 13:51 |
| opendevreview | Merged openstack/kolla-ansible master: Support customising Nova config at the individual service level https://review.opendev.org/c/openstack/kolla-ansible/+/894298 | 14:29 |
| opendevreview | Merged openstack/kolla-ansible master: Add genconfig to the Operating Kolla guide https://review.opendev.org/c/openstack/kolla-ansible/+/890397 | 14:29 |
| opendevreview | Merged openstack/kolla-ansible master: README: link to the meetings page directly https://review.opendev.org/c/openstack/kolla-ansible/+/894930 | 14:29 |
| SvenKieske | mnasiadka: should we maybe add the doodle links to the etherpad? | 14:53 |
| SvenKieske | do we have any special pid file handling code in kolla? still wondering about this radvd issue, so far haven't found much. | 15:00 |
| SvenKieske | nvm, it's all over the place :) | 15:11 |
| SvenKieske | okay, neutron invented it's own pid file handling, it's somewhere in this maze: https://github.com/openstack/neutron/blob/c33805919eb89f5ea6d8b54d4142d4c829d9a2a0/neutron/agent/linux/ra.py | 15:36 |
| mnasiadka | SvenKieske: good idea, will add tomorrow | 15:43 |
| SvenKieske | yeah I'm also out for today, need to get some sleep.. o/ | 16:06 |
| frickler | mnasiadka: maybe use https://framadate.org/ instead of doodle as a privacy friendly alternative. also I like the idea of ignoring the prescribed timeslots | 16:12 |
| opendevreview | Merged openstack/kolla master: toolbox: Add sleep script handling signals https://review.opendev.org/c/openstack/kolla/+/891777 | 18:36 |
| opendevreview | Merged openstack/kolla-ansible master: toolbox: use script handling signals https://review.opendev.org/c/openstack/kolla-ansible/+/891779 | 18:44 |
| opendevreview | Célestin Matte proposed openstack/kolla-ansible master: kolla_address: check correct error return value https://review.opendev.org/c/openstack/kolla-ansible/+/895736 | 19:24 |
| opendevreview | Rocky proposed openstack/kolla-ansible master: Set config files to root/horizon https://review.opendev.org/c/openstack/kolla-ansible/+/893307 | 23:22 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!