| noonedeadpunk | btw, ansible-lint bumped limit to 160 chars | 08:47 |
|---|---|---|
| noonedeadpunk | they've upgraded their serial consoles I guess :D | 08:54 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-ceph_client master: Fix linters and metadata https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/888216 | 09:01 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-lxc_hosts master: Fix linters issue and metadata https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/888180 | 09:43 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-ceph_client master: Fix linters and metadata https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/888216 | 09:47 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-ceph_client master: Fix linters and metadata https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/888216 | 09:51 |
| noonedeadpunk | should be very carefull with `|-` vs `>-` | 09:53 |
| * noonedeadpunk thinks yaml is too hard | 09:53 | |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_service master: Select subelements only when services are defined https://review.opendev.org/c/openstack/ansible-role-systemd_service/+/888223 | 10:01 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_service master: Fix linters and metadata https://review.opendev.org/c/openstack/ansible-role-systemd_service/+/888223 | 10:03 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_service master: Fix linters and metadata https://review.opendev.org/c/openstack/ansible-role-systemd_service/+/888223 | 10:06 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-uwsgi master: Fix linters and metadata https://review.opendev.org/c/openstack/ansible-role-uwsgi/+/888224 | 10:11 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_mount master: Fix linters and metadata https://review.opendev.org/c/openstack/ansible-role-systemd_mount/+/888225 | 10:29 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_networkd master: Fix linters and metadata https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/888226 | 10:33 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-python_venv_build master: Fix linters and metadata https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/888227 | 10:56 |
| * noonedeadpunk wonders how many bugs he has created with these patches | 10:56 | |
| opendevreview | Danila Balagansky proposed openstack/openstack-ansible-os_neutron master: Configure OVN NB and SB DB Connection probes https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/887192 | 10:57 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-openstack_openrc master: Update role metadata https://review.opendev.org/c/openstack/openstack-ansible-openstack_openrc/+/888231 | 11:14 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-qdrouterd master: Fix linters and metadata https://review.opendev.org/c/openstack/ansible-role-qdrouterd/+/888232 | 11:25 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server master: Fix linters and metadata https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/888233 | 11:32 |
| jrosser | damiandabrowski: do you ever see something like this? https://paste.opendev.org/show/bP1nSSNiSCM45BHEoGel/ | 11:39 |
| damiandabrowski | jrosser: yeah, here's a fix ;) https://review.opendev.org/c/openstack/openstack-ansible/+/888149 | 11:43 |
| damiandabrowski | as a quick workaround you can also run ansible -m setup nova_all | 11:43 |
| jrosser | hmm i have applied that patch already (saw it earlier) | 11:44 |
| jrosser | `'ansible.utils.unsafe_proxy.AnsibleUnsafeText object' has no attribute 'haproxy_service_name'` | 11:45 |
| jrosser | is this really a missing fact? | 11:45 |
| damiandabrowski | hmm, i had similar issue and it was about the fact :| https://paste.openstack.org/show/bI9F1EW4zeEgVr56MYcT/ | 11:47 |
| damiandabrowski | ouh, but in fact it is not exactly the same error O.o | 11:49 |
| jrosser | interesting, mine fails straight away, yours did metadata and api service | 11:49 |
| damiandabrowski | yeah and it complains about different line in haproxy-service-config.yml | 11:51 |
| jrosser | i'll see if i can add some debug | 11:51 |
| damiandabrowski | btw. i just realized that line number in my case is not relevant because it complains about line 43 that doesn't even exist :D probably i had some customizations in that file for debugging purposes | 11:54 |
| damiandabrowski | I'm leaving the house right now but I can help with that during the evening | 11:55 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_service master: Fix linters and metadata https://review.opendev.org/c/openstack/ansible-role-systemd_service/+/888223 | 11:58 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-lxc_hosts master: Fix linters issue and metadata https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/888180 | 12:01 |
| mgariepy | nice to see that they now have a better serial console ! | 12:15 |
| halali | damiandabrowski seems I had the same failure with different number https://paste.openstack.org/show/bSjLLH9jCDKQL26tVylZ/ | 12:17 |
| damiandabrowski | halali: ahh, did the #888149 fix help you? | 12:19 |
| halali | damiandabrowski tbh did not try that, on it.... | 12:21 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-galera_server master: Fix linters issue and metadata https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/888132 | 12:33 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-lxc_hosts master: Fix linters issue and metadata https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/888180 | 12:44 |
| noonedeadpunk | folks, sorry, I need help. How would you read this path? https://paste.openstack.org/show/buufGCb7jDZhnolpyYU6/ | 13:15 |
| noonedeadpunk | as `ls -la /var/cache/lxc/download/` just empty | 13:16 |
| noonedeadpunk | It feels like we're doing smth very-very weird at the moment | 13:16 |
| noonedeadpunk | oh, for fucks sake | 13:16 |
| noonedeadpunk | https://paste.openstack.org/show/bcP9YnVtjHS5PzQaXcKn/ | 13:17 |
| jamesdenton | woops | 13:17 |
| noonedeadpunk | does anybody has the same on their lxc hosts ? :D | 13:18 |
| jamesdenton | let me see | 13:18 |
| jrosser | same what? | 13:18 |
| noonedeadpunk | last paste | 13:18 |
| noonedeadpunk | like using /root/"/ path instead of /var/cache/lxc/download/ | 13:18 |
| noonedeadpunk | I guess that's result of my linters though... | 13:19 |
| noonedeadpunk | *linters patch | 13:19 |
| jrosser | not on my current AIO | 13:19 |
| jamesdenton | same ^^ | 13:19 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-lxc_hosts master: Fix linters issue and metadata https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/888180 | 13:19 |
| noonedeadpunk | yeah, https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/888180/4..5/tasks/lxc_cache.yml that was cause it | 13:20 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Update haproxy healthcheck options https://review.opendev.org/c/openstack/openstack-ansible/+/887285 | 13:21 |
| noonedeadpunk | I just completely not getting what are we doing with lxc image cache | 13:21 |
| noonedeadpunk | So ansible_facts['date_time']['epoch'] is timestamp, right? https://opendev.org/openstack/openstack-ansible-lxc_hosts/src/branch/master/tasks/lxc_cache.yml#L19 | 13:22 |
| noonedeadpunk | Then we compare contents of `expiry` file with ansible_facts['date_time']['epoch'] | 13:23 |
| jrosser | seems that way | 13:23 |
| noonedeadpunk | but to expiry file we're writing output of date -d @{{ timestamp }} https://opendev.org/openstack/openstack-ansible-lxc_hosts/src/branch/master/tasks/lxc_cache_create.yml#L51-L61 | 13:24 |
| noonedeadpunk | and that output is datetime, and not timestamp | 13:24 |
| noonedeadpunk | so how we compare timestamp with datetime.... | 13:25 |
| jrosser | maybe this never worked / always wrong | 13:26 |
| noonedeadpunk | but `/var/cache/lxc/download/ubuntu/22/amd64/default/expiry` in your aio is datetime? | 13:28 |
| noonedeadpunk | or it me having things messed up again? | 13:29 |
| jrosser | `Thu Jul 4 16:38:15 UTC 2024` | 13:29 |
| noonedeadpunk | I can't imagine how that could ever work.... | 13:30 |
| halali | for me I have `Thu Jul 11 08:13:07 UTC 2024` | 13:30 |
| jrosser | it is probably that this expiry has never worked? | 13:31 |
| jrosser | otherwise it would constantly be expired | 13:31 |
| noonedeadpunk | I _think_ we're comparing these as text | 13:39 |
| noonedeadpunk | as when we do `date -d` we have to apply `int` filter to cache_time | 13:40 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-lxc_hosts master: Refactor LXC image expiration https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/888278 | 13:43 |
| noonedeadpunk | this should fix it ^ | 13:43 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-lxc_hosts master: Fix linters issue and metadata https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/888180 | 13:44 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-lxc_hosts master: Fix linters issue and metadata https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/888180 | 13:45 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-python_venv_build master: Fix linters and metadata https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/888227 | 13:48 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-ceph_client master: Fix linters and metadata https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/888216 | 13:54 |
| mgariepy | looks like i can simply add multiple domains to the lets encrypt cert and then bind to specific ip for the frontend, | 14:05 |
| jrosser | this looks wrong for master branch https://github.com/openstack/openstack-ansible/blob/master/ansible-collection-requirements.yml#L15 | 14:07 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-repo_server master: Fix linters and metadata https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/888280 | 14:07 |
| noonedeadpunk | jrosser: I assume you're trying aio and heat? | 14:07 |
| jrosser | yeah | 14:08 |
| noonedeadpunk | but yeah, on master I guess it should be jsut master... | 14:08 |
| noonedeadpunk | we also should make new tag for config-template | 14:08 |
| jrosser | i see a patch for that to 2023.1 | 14:08 |
| noonedeadpunk | but I wanted to finally publicize it first... | 14:08 |
| noonedeadpunk | yup, I haven't push one for master yet | 14:09 |
| noonedeadpunk | so feel free to | 14:09 |
| jrosser | would be nice to have helper script to update collections / roles | 14:09 |
| noonedeadpunk | I actually have that in `osa releases` | 14:10 |
| noonedeadpunk | But it doesn't take care of plugins IIRC | 14:10 |
| jrosser | sometimes rerunning bootstrap-ansible is not wanted, just redo ansible-galaxy | 14:10 |
| jrosser | particularly when hacking on things | 14:10 |
| noonedeadpunk | https://review.opendev.org/c/openstack/openstack-ansible/+/885376/2/doc/source/contributor/periodic-work.rst | 14:11 |
| noonedeadpunk | ah, but that's not what you're talkign about :) | 14:12 |
| jrosser | no, like just fix collection sha/version in a-c-r and update | 14:14 |
| noonedeadpunk | I think we have some ENV vars for that? | 14:14 |
| noonedeadpunk | like `SKIP_OSA_RUNTIME_VENV_BUILD` | 14:15 |
| noonedeadpunk | we can add vars to skip a-c-r/a-r-r as well | 14:16 |
| noonedeadpunk | or well. | 14:17 |
| noonedeadpunk | we have `SKIP_OSA_ROLE_CLONE`, but that would affect both roles and collections | 14:17 |
| noonedeadpunk | so that's kinda bug | 14:18 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Allow to skip roles/collections boostrap separately https://review.opendev.org/c/openstack/openstack-ansible/+/888283 | 14:24 |
| noonedeadpunk | jrosser: ^ | 14:24 |
| noonedeadpunk | I wonder why focal is failing here and if we should care about that... https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/888132 | 14:31 |
| jrosser | urgh no_log | 14:32 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-galera_server master: Fix linters issue and metadata https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/888132 | 14:32 |
| noonedeadpunk | hopefully it was left quote for ansible_python_interpreter definition.... | 14:34 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Allow to skip roles/collections bootstrap separately https://review.opendev.org/c/openstack/openstack-ansible/+/888283 | 14:58 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-haproxy_server master: Fix linters issue and metadata https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/888143 | 15:35 |
| noonedeadpunk | During review of `osa/core-2.15` please pay attention on `|-` vs `>-` vs `>` vs `|`. In 99% usecases there should be `>-` which means result will be on the single line without new line (\n) at the end | 15:37 |
| mgariepy | ok | 15:38 |
| mgariepy | should we also fix the `Use FQCN for builtin module actions` also ? | 15:42 |
| noonedeadpunk | Well, yes, we should. | 15:48 |
| noonedeadpunk | But maybe not now?:) | 15:48 |
| noonedeadpunk | Pull in https://review.opendev.org/c/openstack/openstack-ansible/+/886527/12/.ansible-lint | 15:49 |
| noonedeadpunk | btw, things becoming more and more fun in rhel world: https://www.suse.com/news/SUSE-Preserves-Choice-in-Enterprise-Linux/ | 15:49 |
| noonedeadpunk | does anyone have any idea wtf is that? https://paste.openstack.org/show/byHFwccdIlzLTiqv4z8O/ shouldn't type be preserved? | 16:01 |
| anskiy | noonedeadpunk: from what I've understood, SUSE's just gonna sponsor Rocky, so, they're not shattering RPM-based distros further | 16:01 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-lxc_hosts master: Refactor LXC image expiration https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/888278 | 16:08 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-lxc_hosts master: Fix linters issue and metadata https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/888180 | 16:09 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-lxc_hosts master: Fix linters issue and metadata https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/888180 | 16:09 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-lxc_hosts master: Fix linters issue and metadata https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/888180 | 16:11 |
| mgariepy | noonedeadpunk, ha. | 17:26 |
| mgariepy | the lint config wasn't applied on hte option 1 ;) haha | 17:27 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-haproxy_server master: Add ability to have different backend port. https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/888314 | 17:47 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-haproxy_server master: Add ability to have different backend port. https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/888314 | 17:51 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/2023.1: Pin version of setuptools https://review.opendev.org/c/openstack/openstack-ansible/+/888332 | 20:29 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!