| opendevreview | Sven Kieske proposed openstack/kolla-ansible master: [docs] remove warning about octavia_network_type https://review.opendev.org/c/openstack/kolla-ansible/+/922098 | 05:51 |
|---|---|---|
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: requirements: pin bcrypt to supported version https://review.opendev.org/c/openstack/kolla-ansible/+/962419 | 08:42 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: requirements: pin bcrypt to supported version https://review.opendev.org/c/openstack/kolla-ansible/+/962419 | 08:44 |
| mnasiadka | bbezak, frickler: ^^ I think that’s the only way to fix that in stable branches - and I’ll work on a permanent fix to not use that Ansible function that uses passlib and just use bcrypt directly (until we can bump to an Ansible version that supports libcrypt/libxcrypt) | 08:46 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Switch from passlib to internal bcrypt filter https://review.opendev.org/c/openstack/kolla-ansible/+/969867 | 08:51 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Switch from passlib to internal bcrypt filter https://review.opendev.org/c/openstack/kolla-ansible/+/969867 | 08:52 |
| Vii | Regarding the bcrypt/passlib issue, has anyone considered offloading the hash generation to the kolla-toolbox container? Since we control the python env inside the image, we could guarantee a working bcrypt/passlib combo there, bypassing whatever broken version is on the host's OS. | 09:12 |
| Vii | or somehow generate the hash in kolla-toolbox, there are many ways | 09:13 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Adapt notifying handlers message https://review.opendev.org/c/openstack/kolla-ansible/+/940428 | 09:13 |
| Vii | mkpasswd, htpasswd | 09:15 |
| mnasiadka | In master - maybe, in stable branches? That’s too much of a change | 09:26 |
| frickler | so do we install k-a without upper-constraints.txt? because the latter has bcrypt===4.3.0 pinned. | 09:28 |
| Vii | ansible/roles/prometheus/tasks/config.yml | 09:33 |
| Vii | - name: Generate Prometheus web password hash via kolla_toolbox | 09:33 |
| Vii | kolla_toolbox: | 09:33 |
| Vii | module_name: "shell" | 09:33 |
| Vii | module_args: "python3 -c 'import bcrypt; print(bcrypt.hashpw(b\"{{ prometheus_password }}\", bcrypt.gensalt()).decode())'" | 09:33 |
| Vii | register: prometheus_hash_toolbox | 09:33 |
| Vii | delegate_to: "{{ groups['deployment'][0] }}" | 09:33 |
| Vii | - name: Set hashed password fact | 09:33 |
| Vii | set_fact: | 09:33 |
| Vii | prometheus_password_hash: "{{ prometheus_hash_toolbox.stdout }}" | 09:33 |
| Vii | something like this and it should work :) | 09:33 |
| mnasiadka | frickler: well, people do install without upper-constraints :) | 09:46 |
| mnasiadka | Vii: if only that would not leak more passwords :D | 09:47 |
| Vii | :) | 09:49 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Adapt notifying handlers message https://review.opendev.org/c/openstack/kolla-ansible/+/940428 | 10:00 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Switch from passlib to internal bcrypt filter https://review.opendev.org/c/openstack/kolla-ansible/+/969867 | 10:05 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Switch from passlib to internal bcrypt filter https://review.opendev.org/c/openstack/kolla-ansible/+/969867 | 10:26 |
| opendevreview | Pierre Riteau proposed openstack/kolla-ansible master: Disable deployment of Bifrost internal registry https://review.opendev.org/c/openstack/kolla-ansible/+/973460 | 11:19 |
| opendevreview | Leonie Chamberlin-Medd proposed openstack/kayobe master: Add support for fail2ban in Kayobe https://review.opendev.org/c/openstack/kayobe/+/973090 | 11:50 |
| opendevreview | Leonie Chamberlin-Medd proposed openstack/kayobe master: Add support for fail2ban in Kayobe https://review.opendev.org/c/openstack/kayobe/+/973090 | 11:58 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: valkey: Fix 'valkey_master_host' undefined error during upgrade https://review.opendev.org/c/openstack/kolla-ansible/+/973469 | 12:55 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Switch from passlib to internal bcrypt filter https://review.opendev.org/c/openstack/kolla-ansible/+/969867 | 13:08 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Add restart policy to container check https://review.opendev.org/c/openstack/kolla-ansible/+/928986 | 13:32 |
| opendevreview | Matt Crees proposed openstack/kayobe master: Deprecate kolla-tags and kolla-limit https://review.opendev.org/c/openstack/kayobe/+/935669 | 13:32 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Adapt notifying handlers message https://review.opendev.org/c/openstack/kolla-ansible/+/940428 | 13:37 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: neutron: decouple backend TLS certs from CA cert copying https://review.opendev.org/c/openstack/kolla-ansible/+/973484 | 14:08 |
| opendevreview | Leonie Chamberlin-Medd proposed openstack/kayobe master: Add support for fail2ban in Kayobe https://review.opendev.org/c/openstack/kayobe/+/973090 | 14:20 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Switch from passlib to internal bcrypt filter https://review.opendev.org/c/openstack/kolla-ansible/+/969867 | 14:23 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Switch from passlib to internal bcrypt filter https://review.opendev.org/c/openstack/kolla-ansible/+/969867 | 14:24 |
| opendevreview | Leonie Chamberlin-Medd proposed openstack/kayobe master: Add support for fail2ban in Kayobe https://review.opendev.org/c/openstack/kayobe/+/973090 | 14:24 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Switch from passlib to internal bcrypt filter https://review.opendev.org/c/openstack/kolla-ansible/+/969867 | 14:48 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: Fix Valkey log parsing in Fluentd https://review.opendev.org/c/openstack/kolla-ansible/+/973493 | 15:03 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible stable/2025.1: Fix idempotence on comparing capabilities for podman https://review.opendev.org/c/openstack/kolla-ansible/+/972836 | 16:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible stable/2025.1: Fix podman idempotence on comparing container dimensions https://review.opendev.org/c/openstack/kolla-ansible/+/972837 | 16:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible stable/2025.1: Fix idempotence on podman volume comparison https://review.opendev.org/c/openstack/kolla-ansible/+/972838 | 16:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible stable/2024.2: Fix idempotence on comparing capabilities for podman https://review.opendev.org/c/openstack/kolla-ansible/+/972839 | 16:11 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible stable/2024.2: Fix podman idempotence on comparing container dimensions https://review.opendev.org/c/openstack/kolla-ansible/+/972840 | 16:11 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible stable/2024.2: Fix idempotence on podman volume comparison https://review.opendev.org/c/openstack/kolla-ansible/+/972841 | 16:11 |
| *** jhorstmann is now known as Guest35949 | 16:22 | |
| blanson[m] | quick question, would it make sense in a reconfigure/deploy scenario to wait (for containers that have healthchecks) for the healthcheck to pass before moving on ? We're doing reconfigure and sometimes previous containers will still be starting during the next restart (mainly APIs) | 16:30 |
| *** gmaan is now known as gmaan_afk | 16:33 | |
| opendevreview | Merged openstack/kolla-ansible master: requirements: pin bcrypt to supported version https://review.opendev.org/c/openstack/kolla-ansible/+/962419 | 16:35 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: kolla-ansible: migrate redis password to valkey in mergepwd https://review.opendev.org/c/openstack/kolla-ansible/+/973512 | 16:44 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: kolla-ansible: migrate redis password to valkey in mergepwd https://review.opendev.org/c/openstack/kolla-ansible/+/973512 | 16:50 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in ceph-rgw https://review.opendev.org/c/openstack/kolla-ansible/+/970881 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in cinder https://review.opendev.org/c/openstack/kolla-ansible/+/970882 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in designate https://review.opendev.org/c/openstack/kolla-ansible/+/970883 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in glance https://review.opendev.org/c/openstack/kolla-ansible/+/970928 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in heat https://review.opendev.org/c/openstack/kolla-ansible/+/970929 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in horizon https://review.opendev.org/c/openstack/kolla-ansible/+/970930 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in ironic https://review.opendev.org/c/openstack/kolla-ansible/+/970931 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in keystone https://review.opendev.org/c/openstack/kolla-ansible/+/970932 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in roles https://review.opendev.org/c/openstack/kolla-ansible/+/970933 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in magnum https://review.opendev.org/c/openstack/kolla-ansible/+/970934 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in manila https://review.opendev.org/c/openstack/kolla-ansible/+/970935 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in mariadb https://review.opendev.org/c/openstack/kolla-ansible/+/970936 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in neutron https://review.opendev.org/c/openstack/kolla-ansible/+/970937 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in roles https://review.opendev.org/c/openstack/kolla-ansible/+/970938 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in nova https://review.opendev.org/c/openstack/kolla-ansible/+/970945 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in octavia https://review.opendev.org/c/openstack/kolla-ansible/+/970946 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in opensearch https://review.opendev.org/c/openstack/kolla-ansible/+/970947 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in roles https://review.opendev.org/c/openstack/kolla-ansible/+/970948 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in ovn-db https://review.opendev.org/c/openstack/kolla-ansible/+/970949 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in ovs-dpdk https://review.opendev.org/c/openstack/kolla-ansible/+/970950 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in placement https://review.opendev.org/c/openstack/kolla-ansible/+/970951 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in roles https://review.opendev.org/c/openstack/kolla-ansible/+/970952 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in roles https://review.opendev.org/c/openstack/kolla-ansible/+/970953 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in rabbitmq https://review.opendev.org/c/openstack/kolla-ansible/+/970954 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in skyline https://review.opendev.org/c/openstack/kolla-ansible/+/970955 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in tacker https://review.opendev.org/c/openstack/kolla-ansible/+/970956 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in globals https://review.opendev.org/c/openstack/kolla-ansible/+/970957 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in etcd https://review.opendev.org/c/openstack/kolla-ansible/+/970958 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in trove https://review.opendev.org/c/openstack/kolla-ansible/+/972580 | 17:01 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: ansible-lint: remove line-length skip https://review.opendev.org/c/openstack/kolla-ansible/+/970959 | 17:01 |
| mnasiadka | blanson[m]: I think I added some wait time before checking but it might not be enough | 17:05 |
| *** gmaan_afk is now known as gmaan | 17:46 | |
| blanson[m] | [@_oftc_mnasiadka:matrix.org](https://matrix.to/#/@_oftc_mnasiadka:matrix.org) i'm not seeing that in the worker classes. Do you remember where it is ? I'll toy around with it to see if something can be done | 18:01 |
| mnasiadka | blanson[m]: ah, I thought you’re mentioning the CI, not a real deployment :) | 18:34 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: kolla-ansible: add prune-images-by-tag command https://review.opendev.org/c/openstack/kolla-ansible/+/973542 | 19:12 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: kolla-ansible: add prune-images-by-tag command https://review.opendev.org/c/openstack/kolla-ansible/+/973542 | 19:15 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: kolla-ansible: add prune-images-by-tag command https://review.opendev.org/c/openstack/kolla-ansible/+/973542 | 19:20 |
| opendevreview | Merged openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in ceph-rgw https://review.opendev.org/c/openstack/kolla-ansible/+/970881 | 19:20 |
| opendevreview | Merged openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in cinder https://review.opendev.org/c/openstack/kolla-ansible/+/970882 | 19:27 |
| opendevreview | Merged openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in designate https://review.opendev.org/c/openstack/kolla-ansible/+/970883 | 19:27 |
| blanson[m] | mnasiadka: yeah I was thinking when we called the restart handler, maybe we can wait for healthcheck if there is one ? would that make the run too slow ? I'd argue for it from a HA perspective but idk how bad it would affect deploy time. | 19:57 |
| blanson[m] | s/called/call/ | 19:58 |
| opendevreview | Merged openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in glance https://review.opendev.org/c/openstack/kolla-ansible/+/970928 | 20:26 |
| opendevreview | Merged openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in heat https://review.opendev.org/c/openstack/kolla-ansible/+/970929 | 20:27 |
| opendevreview | Pierre Riteau proposed openstack/kolla-ansible master: docs: fix mariadb client binary https://review.opendev.org/c/openstack/kolla-ansible/+/973556 | 20:32 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: keystone: Setup fernet credentials encryption keys https://review.opendev.org/c/openstack/kolla-ansible/+/970861 | 20:40 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: kolla-ansible: add prune-images-by-tag command https://review.opendev.org/c/openstack/kolla-ansible/+/973542 | 20:50 |
| opendevreview | Merged openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in horizon https://review.opendev.org/c/openstack/kolla-ansible/+/970930 | 21:16 |
| opendevreview | Merged openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in ironic https://review.opendev.org/c/openstack/kolla-ansible/+/970931 | 21:16 |
| opendevreview | Merged openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in keystone https://review.opendev.org/c/openstack/kolla-ansible/+/970932 | 21:16 |
| opendevreview | Merged openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in roles https://review.opendev.org/c/openstack/kolla-ansible/+/970933 | 21:16 |
| opendevreview | Merged openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in magnum https://review.opendev.org/c/openstack/kolla-ansible/+/970934 | 21:16 |
| opendevreview | Merged openstack/kolla-ansible master: ansible-lint: fix yaml[line-length] in manila https://review.opendev.org/c/openstack/kolla-ansible/+/970935 | 21:16 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!