| *** ptoal is now known as Sage905 | 02:18 | |
| noonedeadpunk | folks, can we kindly land leftovers for 2023.1? https://review.opendev.org/q/parentproject:openstack/openstack-ansible+branch:%255Estable/2023.1+status:open+ | 08:20 |
|---|---|---|
| noonedeadpunk | especially for the integrated repo | 08:20 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/zed: Include proper vars_file for rally https://review.opendev.org/c/openstack/openstack-ansible/+/888436 | 08:28 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_rally stable/zed: Include proper commit in rally_upper_constraints_url https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/887680 | 08:28 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/zed: Re-enable CI jobs after rally is fixed https://review.opendev.org/c/openstack/openstack-ansible/+/888437 | 08:29 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/zed: Pin version of setuptools https://review.opendev.org/c/openstack/openstack-ansible/+/887157 | 08:30 |
| jrosser | done | 08:31 |
| noonedeadpunk | thanks! | 08:33 |
| noonedeadpunk | will try to fix other stable branches today as well | 08:33 |
| jrosser | do we really mean br-octavia here? https://github.com/openstack/openstack-ansible/commit/d35e3e0daef67546eded457a92186eb043575d69 | 09:51 |
| noonedeadpunk | likely this works for OVN... | 09:52 |
| noonedeadpunk | not sure about ovs though | 09:52 |
| noonedeadpunk | (or anything else) | 09:53 |
| jrosser | it's different to this https://opendev.org/openstack/openstack-ansible/src/branch/master/etc/openstack_deploy/openstack_user_config.yml.aio#L72 | 09:53 |
| jrosser | and not what AIO bootstrap creates | 09:53 |
| noonedeadpunk | yeah, jsut found that as well | 09:53 |
| noonedeadpunk | we need either to revert one or update another as well... | 09:54 |
| jrosser | i just found octavia in an LXC AIO is broken from this | 09:54 |
| noonedeadpunk | well.... we both were reviewing that change... | 09:54 |
| noonedeadpunk | nobody else to blame :D | 09:55 |
| jrosser | indeed | 09:55 |
| jrosser | oh well | 10:00 |
| jrosser | maybe there is br-lbaas on the controllers for containers | 10:00 |
| jrosser | and OVN managed br-octavia on computes | 10:01 |
| jrosser | which are actually the same flat network? | 10:01 |
| noonedeadpunk | well... then there should be 2 records in provider_networks | 10:02 |
| noonedeadpunk | as currently we're trying to build br-octavia to octavia-worker | 10:02 |
| noonedeadpunk | and not only to neutron_ovn_gateway | 10:02 |
| noonedeadpunk | and well, not sure if neutron_linuxbridge_agent/ovs_agent removed | 10:05 |
| jrosser | it needs also to work for AIO scenario | 10:07 |
| jrosser | when controller == compute | 10:07 |
| opendevreview | Merged openstack/openstack-ansible-plugins stable/2023.1: Skip updating service password by default https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/888152 | 10:27 |
| opendevreview | Merged openstack/openstack-ansible-os_rally stable/2023.1: Include proper commit in rally_upper_constraints_url https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/887679 | 10:27 |
| opendevreview | Merged openstack/openstack-ansible-os_keystone stable/2023.1: Fix SSL logic in keystone-httpd.conf.j2 https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/888158 | 10:28 |
| opendevreview | Merged openstack/openstack-ansible-os_nova stable/2023.1: Apply always tag to nova_virt_detect.yml https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/887878 | 10:31 |
| opendevreview | Merged openstack/openstack-ansible-galera_server stable/2023.1: Add optional compression to mariabackup https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/887139 | 10:43 |
| opendevreview | Merged openstack/openstack-ansible stable/2023.1: Adjust default value for *_backend_ssl https://review.opendev.org/c/openstack/openstack-ansible/+/887683 | 11:01 |
| opendevreview | Merged openstack/openstack-ansible stable/2023.1: Pin version of setuptools https://review.opendev.org/c/openstack/openstack-ansible/+/888332 | 11:01 |
| noonedeadpunk | ok, zed also seems "fixed" with https://review.opendev.org/c/openstack/openstack-ansible/+/888436 and https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/887680 | 11:06 |
| 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 | 11:44 |
| 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 | 11:48 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-openstack_hosts master: Fix linters issue and metadata https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/888455 | 11:58 |
| 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:03 |
| noonedeadpunk | huh, looking at what takes time in our CI jobs, and that looks like taking quite a lot for such simple task? https://paste.openstack.org/show/b5uS80Y5DBfPjLhOMtqN/ | 12:12 |
| noonedeadpunk | like 6sec for pasting a file | 12:12 |
| jrosser | thats surprising | 12:13 |
| jrosser | just copy module i think | 12:13 |
| noonedeadpunk | but then same job for rally is fast | 12:14 |
| noonedeadpunk | https://paste.openstack.org/show/bQ9njmsVGIzjzDEZfaGE/ | 12:14 |
| jrosser | i guess we have no idea what is happening with the storage at that moment | 12:15 |
| jrosser | could be anything | 12:15 |
| noonedeadpunk | yeah, true | 12:18 |
| noonedeadpunk | I'm not sure dstat shows iostat (I bet it's not) | 12:18 |
| noonedeadpunk | ah, it does, but we indeed don't know what is on layer below | 12:19 |
| noonedeadpunk | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_906/886527/12/check/openstack-ansible-deploy-aio_metal-rockylinux-9/90602d2/logs/openstack/instance-info/dstat.html | 12:19 |
| jrosser | ansible should be using real tmpfs for all intermediate file operations though | 12:23 |
| jrosser | perhaps we have some case where that doesnt happen and we use real disk | 12:23 |
| noonedeadpunk | I assume we're disabling swap in our jobs | 12:28 |
| noonedeadpunk | anyway. | 12:29 |
| noonedeadpunk | damiandabrowski: fwiw, this new approach with haproxy_ackends configured per playbook acting really crazy for metal deployments - it would configure everything when running anything | 12:31 |
| noonedeadpunk | as metal host is part of all API groups basically, I guess | 12:32 |
| damiandabrowski | ahh :/ will look into that | 12:34 |
| noonedeadpunk | but it could be my weird sandbox with ansible 2.15 as well :D | 12:34 |
| noonedeadpunk | actually, looking at current logs - it seems to work as expected | 12:43 |
| noonedeadpunk | So indeed it might got broken jsut with ansible-core 2.15.1 | 12:44 |
| noonedeadpunk | (logs in CI) | 12:44 |
| mgariepy | hnn | 12:49 |
| mgariepy | i think i miss a part for my haproxy patch. | 12:50 |
| mgariepy | the haproxy_check_port probably need to match the other one | 12:50 |
| mgariepy | https://opendev.org/openstack/openstack-ansible-haproxy_server/src/branch/master/templates/service.j2#L9-L13 this won't do it since it won't monitor the backend correclty | 12:54 |
| mgariepy | https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/888314 | 12:54 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_service master: Reduce output by leveraging loop labels https://review.opendev.org/c/openstack/ansible-role-systemd_service/+/876302 | 12:55 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_glance master: Apply tags to systemd_service include https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/888458 | 12:58 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-ceph_client master: Apply tags to included tasks https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/888461 | 13:02 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-ceph_client master: Apply tags to included tasks https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/888461 | 13:02 |
| 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:04 |
| jamesdenton | as a git newb, how does one resolve these types of failures when running bootstrap-ansible.sh? https://paste.opendev.org/show/bWo13GvOVpbeUJIJ3Iyt/ -- "Failed to download remote objects and refs: fatal: error in object: unshallow xxxx" | 13:18 |
| opendevreview | Merged openstack/openstack-ansible-openstack_openrc master: Update role metadata https://review.opendev.org/c/openstack/openstack-ansible-openstack_openrc/+/888231 | 13:22 |
| Sage905 | jamesdenton: What OS / version are you running it on? | 13:23 |
| jamesdenton | 22.04.2 | 13:23 |
| jrosser | you can perhaps try to make this larger https://github.com/openstack/openstack-ansible/blob/ea0bbd2c582b313fc7174cc714a52e17e27e6e14/scripts/get-ansible-role-requirements.yml#L167 | 13:23 |
| mgariepy | jamesdenton, https://github.com/openstack/openstack-ansible/blob/master/ansible-role-requirements.yml it's a configuration from there: | 13:23 |
| mgariepy | you try to upgrade an existing env ? | 13:24 |
| jamesdenton | no, this was a 2023.1 install from the beginning about a week ago | 13:24 |
| jrosser | it might be that when the number of commits in a repo beyond the shallow-since date is more than `role_clone_default_depth`, then the SHA is question is not available to check out | 13:25 |
| jamesdenton | i will bump to 40 and see if that makes a difference, thank you | 13:26 |
| noonedeadpunk | I think that depth should be ignored by parallel clone when shallow_since is provided | 13:30 |
| noonedeadpunk | And legacy (non-parallel) ignores shallow_since | 13:30 |
| noonedeadpunk | jamesdenton: I think it's due to pacemaker and etcd actually | 13:31 |
| jamesdenton | parallel is failing, too. | 13:31 |
| jamesdenton | yes, pacemaker, etcd, neutron andf tempest | 13:31 |
| noonedeadpunk | as they're master. We have a bug report for that, and it will be fixed in 27.0.1 - already merged to stable/2023.1 | 13:31 |
| jamesdenton | hmm, well lemme check i just did a pull on stable/2023.1 | 13:32 |
| noonedeadpunk | https://bugs.launchpad.net/openstack-ansible/+bug/2025513 | 13:32 |
| jamesdenton | i ended up recloning it to get the latest bits - changes seem to be there now, thanks | 13:37 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_adjutant master: Fix linters and metadata https://review.opendev.org/c/openstack/openstack-ansible-os_adjutant/+/888469 | 13:55 |
| opendevreview | Merged openstack/openstack-ansible-rabbitmq_server master: Fix linters and metadata https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/888233 | 13:56 |
| opendevreview | Merged openstack/openstack-ansible-memcached_server master: Fix linters issue and metadata https://review.opendev.org/c/openstack/openstack-ansible-memcached_server/+/888146 | 13:57 |
| opendevreview | Merged openstack/openstack-ansible-lxc_container_create master: Fix linters issue and metadata https://review.opendev.org/c/openstack/openstack-ansible-lxc_container_create/+/888179 | 13:59 |
| opendevreview | Merged openstack/openstack-ansible master: Allow to skip roles/collections bootstrap separately https://review.opendev.org/c/openstack/openstack-ansible/+/888283 | 14:00 |
| opendevreview | Merged openstack/openstack-ansible master: Remove haproxy_accept_both_protocols from repo_all https://review.opendev.org/c/openstack/openstack-ansible/+/886586 | 14:04 |
| hamburgler | damiandabrowski: ty :) re:ssl | 14:05 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_aodh master: Fix linters and metadata https://review.opendev.org/c/openstack/openstack-ansible-os_aodh/+/888471 | 14:11 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_barbican master: Fix linters and metadata https://review.opendev.org/c/openstack/openstack-ansible-os_barbican/+/888472 | 14:25 |
| 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:28 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-openstack_hosts master: Fix linters issue and metadata https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/888455 | 14:32 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_blazar master: Fix linters and metadata https://review.opendev.org/c/openstack/openstack-ansible-os_blazar/+/888473 | 14:41 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_ceilometer master: Fix linters and metadata https://review.opendev.org/c/openstack/openstack-ansible-os_ceilometer/+/888476 | 15:01 |
| *** dviroel_ is now known as dviroel | 15:22 | |
| noonedeadpunk | Tbh, I start thinking how wise is that to set "0640" for local ansible facts... | 15:40 |
| noonedeadpunk | Like I do.... | 15:41 |
| noonedeadpunk | Maybe worth reverting that back to 0644 until it's not tooooooo late | 15:41 |
| opendevreview | Merged openstack/openstack-ansible-os_zun master: Switch jobs to Jammy https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/884362 | 16:18 |
| damiandabrowski | noonedeadpunk: I spawned metal AIO on master branch and ubuntu jammy, executed gate-check-commit.sh and everything looks fine | 16:39 |
| damiandabrowski | each playbook configures only its own haproxy services | 16:39 |
| damiandabrowski | ex. https://paste.openstack.org/show/b6Pzag6prypbiqiZhtZt/ | 16:39 |
| 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:18 |
| mgariepy | hmm that's not quite flexible enough :/ https://opendev.org/openstack/openstack-ansible-haproxy_server/src/branch/master/templates/service.j2#L55 | 18:20 |
| mgariepy | or i'm not doing something supported.. :/ haha | 18:21 |
| mgariepy | https://opendev.org/openstack/openstack-ansible-haproxy_server/src/branch/master/templates/letsencrypt_renew_certbot_distro.j2#L5-L7 | 18:21 |
| mgariepy | I added both domains to this list: haproxy_ssl_letsencrypt_domains | 18:22 |
| mgariepy | added another ip via : keepalived_instances_overrides | 18:23 |
| mgariepy | then i add haproxy_extra_services and bind to the other domain. | 18:24 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-haproxy_server master: Add possibility to override haproxy_ssl_path https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/888498 | 19:12 |
| mgariepy | this fixes my issue.. | 19:14 |
| mgariepy | could be better tho. we could generate the path the same way in the letsencrypt renewal hook and the haproxy template :D | 19:14 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!