noonedeadpunk | Would be awesome if someone could take a look at https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/898749 | 06:48 |
---|---|---|
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/2023.1: Define install_method default when hosts resolution depend on it https://review.opendev.org/c/openstack/openstack-ansible/+/898072 | 07:43 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_designate master: Fix example playbook linters https://review.opendev.org/c/openstack/openstack-ansible-os_designate/+/898906 | 08:47 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_cloudkitty master: Fix example playbook linters https://review.opendev.org/c/openstack/openstack-ansible-os_cloudkitty/+/898907 | 08:49 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_cloudkitty master: Add quorum queues support for service https://review.opendev.org/c/openstack/openstack-ansible-os_cloudkitty/+/898816 | 08:49 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_heat master: Add quorum queues support for service https://review.opendev.org/c/openstack/openstack-ansible-os_heat/+/898908 | 08:59 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_ironic master: Add quorum queues support for service https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/898909 | 09:04 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_magnum master: Add quorum queues support for service https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/898911 | 09:11 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_masakari master: Add quorum queues support for service https://review.opendev.org/c/openstack/openstack-ansible-os_masakari/+/898912 | 09:17 |
opendevreview | Merged openstack/openstack-ansible-os_nova master: Fix logic of discovering hosts by service https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/898749 | 09:17 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_nova stable/2023.1: Fix logic of discovering hosts by service https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/898779 | 09:19 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_nova stable/2023.1: Fix logic of discovering hosts by service https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/898779 | 09:20 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_nova stable/zed: Fix logic of discovering hosts by service https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/898780 | 09:20 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_mistral master: Add quorum queues support for service https://review.opendev.org/c/openstack/openstack-ansible-os_mistral/+/898913 | 09:26 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_manila master: Add quorum queues support for service https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/898914 | 09:31 |
anskiy | noonedeadpunk: hey! so is this: https://review.opendev.org/c/openstack/openstack-ansible-os_ceilometer/+/888032 broken by some ansible-lint things or is it something in openstack.osa.service_setup role? | 10:29 |
anskiy | also, what else should be done to finally push this thing: https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/858271? Maybe you (or jamesdenton) could lay out some questions for me in the Change so I could answer them?.. | 10:30 |
noonedeadpunk | anskiy: I think it's both | 10:30 |
noonedeadpunk | anskiy: regarding AZ-s - what I currently "suspect" is that added variables are not reflecting possibility for AZ setup of OVS/LXB | 10:31 |
noonedeadpunk | As we do have AZs there, and they're having quite different logic then with OVS | 10:31 |
noonedeadpunk | *OVN | 10:31 |
noonedeadpunk | So to implement these new vars we need to ensure they make sense outside of ovn as well | 10:33 |
noonedeadpunk | Or make them more OVN specific in their names, and then add another set of vars for non-ovn, which also sucks | 10:33 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_ceilometer master: Enable Ceilometer resource cache https://review.opendev.org/c/openstack/openstack-ansible-os_ceilometer/+/888032 | 10:34 |
anskiy | okay, got it, thank you :) | 10:34 |
noonedeadpunk | And I really didn't find time yet to come up with smth that will cover both, despite having that in my backlog... | 10:35 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_murano master: Add quorum queues support for service https://review.opendev.org/c/openstack/openstack-ansible-os_murano/+/898922 | 10:42 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_octavia master: Drop Neutron oslomsg configuration https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/898923 | 10:48 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_octavia master: Drop Neutron oslomsg configuration https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/898923 | 10:48 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_octavia master: Add quorum queues support for service https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/898924 | 10:53 |
opendevreview | Merged openstack/openstack-ansible-os_nova stable/2023.1: Run nova_db_post_setup from playbook directly https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/898495 | 10:59 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_senlin master: Add quorum queues support for service https://review.opendev.org/c/openstack/openstack-ansible-os_senlin/+/898926 | 11:05 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_sahara master: Add quorum queues support for service https://review.opendev.org/c/openstack/openstack-ansible-os_sahara/+/898929 | 11:45 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_cloudkitty master: Add quorum queues support for service https://review.opendev.org/c/openstack/openstack-ansible-os_cloudkitty/+/898816 | 11:53 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_aodh master: Add quorum support for service https://review.opendev.org/c/openstack/openstack-ansible-os_aodh/+/895690 | 11:54 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_heat master: Add quorum queues support for service https://review.opendev.org/c/openstack/openstack-ansible-os_heat/+/898908 | 11:56 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_tacker master: Add quorum queues support for service https://review.opendev.org/c/openstack/openstack-ansible-os_tacker/+/898930 | 11:57 |
andrewbonney | jonher: I heard via jrosser that you had found OIDC issues in newer apache mod_auth_oidc | 12:00 |
andrewbonney | In case you haven't found it yet, it's caused by commit ac5686495a51bc93e257e42bfdc9c9c46252feb1 | 12:00 |
andrewbonney | I'm just looking into whether the issue can be resolved via config tweaks | 12:00 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_trove master: Add quorum queues support for service https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/898931 | 12:00 |
jonher | correct! i had that tab open to try and compile the module pre and post that commit but then it sounds like i don't have to :) | 12:01 |
jonher | initial diagnosis was it works on focal but not on jammy. apache mod_auth_oidc v2.4.8.4 works, v2.4.9 and newer does not. One of the last lines in the log was the debug log of it comparing the OS-FEDERATION url paarm and then it didn't seem to do much after that as if the request was more or less just dropped | 12:03 |
andrewbonney | Yes, that commit adds an explicit return where originally I believe it passed on to the ProxyPass set up for keystone | 12:04 |
jonher | yes, even though the Keystone API docs does not even mention POST being a valid request at all on that endpoint (which also confused me during the initial troubleshooting). https://docs.openstack.org/api-ref/identity/v3-ext/#apis | 12:06 |
jonher | thanks for checking on it as well and updating me on the status here, saves me one step of troubleshooting | 12:06 |
opendevreview | Merged openstack/openstack-ansible master: Stop ignoring hostnames without underscores https://review.opendev.org/c/openstack/openstack-ansible/+/898002 | 12:08 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_zun master: Add quorum queues support for service https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/898934 | 12:17 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_ironic master: Add quorum queues support for service https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/898909 | 12:19 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_masakari master: Fix example playbook linters https://review.opendev.org/c/openstack/openstack-ansible-os_masakari/+/898936 | 12:31 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_masakari master: Add quorum queues support for service https://review.opendev.org/c/openstack/openstack-ansible-os_masakari/+/898912 | 12:31 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_masakari master: Add quorum queues support for service https://review.opendev.org/c/openstack/openstack-ansible-os_masakari/+/898912 | 12:32 |
opendevreview | Stuart Grace proposed openstack/openstack-ansible-os_nova master: Use internal endpoint for barbican API https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/898937 | 12:33 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_sahara master: Add quorum queues support for service https://review.opendev.org/c/openstack/openstack-ansible-os_sahara/+/898929 | 12:33 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_senlin master: Add quorum queues support for service https://review.opendev.org/c/openstack/openstack-ansible-os_senlin/+/898926 | 12:34 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_octavia master: Add quorum queues support for service https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/898924 | 12:34 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_manila master: Add quorum queues support for service https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/898914 | 12:35 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_murano master: Add quorum queues support for service https://review.opendev.org/c/openstack/openstack-ansible-os_murano/+/898922 | 12:35 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_mistral master: Add quorum queues support for service https://review.opendev.org/c/openstack/openstack-ansible-os_mistral/+/898913 | 12:35 |
opendevreview | Andrew Bonney proposed openstack/openstack-ansible-os_keystone master: oidc: fix overloading of redirect_uri for cli client https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/898962 | 13:05 |
andrewbonney | jonher: could you see if https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/898962 fixes it for you? | 13:05 |
jonher | andrewbonney: that works for me on Yoga AIO running 22.04 with libapache2-mod-auth-openidc 2.4.11-1 | 13:34 |
andrewbonney | Great, thanks for confirming | 13:35 |
noonedeadpunk | let's try to merge that fast then | 13:46 |
opendevreview | Merged openstack/openstack-ansible-os_nova stable/2023.1: Fix logic of discovering hosts by service https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/898779 | 13:54 |
noonedeadpunk | andrewbonney: do you think we should keep compatability with previous iodc version? or it's supposed to work in both cases? | 13:57 |
noonedeadpunk | as I guess I'm mainly thinking about how to backport that | 13:58 |
opendevreview | Merged openstack/openstack-ansible-os_cloudkitty master: Fix example playbook linters https://review.opendev.org/c/openstack/openstack-ansible-os_cloudkitty/+/898907 | 13:58 |
andrewbonney | I believe it would work with both, but I haven't tested that explicitly. I think it was working more as a happy accident up to now | 13:58 |
noonedeadpunk | aha, I see | 13:58 |
andrewbonney | I'll roll back the module in my test environment to make sure | 13:59 |
andrewbonney | Yes, it works with the focal version too | 14:03 |
noonedeadpunk | awesome | 14:03 |
opendevreview | Merged openstack/openstack-ansible-openstack_hosts stable/2023.1: Add ability to define a config for journald https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/898769 | 14:19 |
admin1 | i get this "fatal: [c1_repo_container-83f07d5e -> localhost]: FAILED! => {"changed": false, "dest": "/etc/openstack_deploy/upper-constraints/upper_constraints_2710414b4663b85b48ecbbba21f4f0ea42dc3df1.txt", "elapsed": 10, "gid": 0, "group": "root", "mode": "0644", "msg": "Request failed: <urlopen error [Errno -3] Temporary failure in name | 15:05 |
admin1 | resolution>", "owner": "root", "size": 12136, "state": "file", "uid": 0, "url": "https://releases.openstack.org/constraints/upper/2710414b4663b85b48ecbbba21f4f0ea42dc3df1"} " but there is nothing wrong with resolving/dns in the container ? | 15:05 |
noonedeadpunk | I'm not sure what exactly you're doing, but that task was delegated to the `localhost` | 15:16 |
noonedeadpunk | so it's localhost which failed to fetch the url | 15:16 |
admin1 | oh .. the deploy host | 15:17 |
admin1 | thanks for the pointer .. i will check further .. | 15:17 |
admin1 | i am getting this error on glance .. https://gist.githubusercontent.com/a1git/df375d2607f9057f4c057aa226c0771f/raw/32d75f740e6f6f90fea85d40d702fda391c46a74/gistfile1.txt doing an upgrade from 26.2.0 -> 27.1.0 .. i don't see any issues on the nfs mount | 16:01 |
admin1 | 'mount_var' is undefined -- maybe there is a new syntax in 27.1.0 .. | 16:03 |
admin1 | grep -ri mount_var /opt/openstack-ansible/etc/openstack_deploy/* returns null, so not able to see examples on it | 16:04 |
noonedeadpunk | admin1: is it using nfs? | 16:07 |
noonedeadpunk | as it;s basically about format of `glance_mount_points` | 16:07 |
admin1 | yeah .. using nfs for this one | 16:08 |
noonedeadpunk | which indeed did changed syntax a bit | 16:08 |
noonedeadpunk | admin1: I think it's this bug you;re hitting https://bugs.launchpad.net/openstack-ansible/+bug/2032771 | 16:10 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_glance master: Use correct default for glance_mount_points https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/892489 | 16:10 |
noonedeadpunk | and that's the fix ^ | 16:10 |
noonedeadpunk | but I would suggest updating glance_mount_points if possible | 16:12 |
admin1 | thanks .. changed and re-running | 16:13 |
admin1 | i removed all lxc except galera, so it think i hit this now and not before | 16:14 |
opendevreview | Merged openstack/openstack-ansible-os_keystone master: oidc: fix overloading of redirect_uri for cli client https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/898962 | 17:44 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_keystone stable/2023.1: oidc: fix overloading of redirect_uri for cli client https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/898947 | 18:14 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_keystone stable/2023.1: oidc: fix overloading of redirect_uri for cli client https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/898947 | 18:15 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_keystone stable/zed: oidc: fix overloading of redirect_uri for cli client https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/898948 | 18:15 |
opendevreview | Merged openstack/openstack-ansible-os_nova master: Use internal endpoint for barbican API https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/898937 | 21:33 |
jrosser | looks like we might need to bump zk role sha for 2023.1 | 21:41 |
opendevreview | Merged openstack/openstack-ansible-os_keystone stable/zed: oidc: fix overloading of redirect_uri for cli client https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/898948 | 22:16 |
noonedeadpunk | yup, right | 22:53 |
noonedeadpunk | and we need to make a new release of 2023.1 finally | 22:53 |
noonedeadpunk | though that's one more bug backport to fix: https://review.opendev.org/c/openstack/openstack-ansible/+/898072 | 22:55 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/2023.1: Bump SHAs for 2023.1 https://review.opendev.org/c/openstack/openstack-ansible/+/898993 | 23:08 |
noonedeadpunk | and that's sha bump^ | 23:08 |
noonedeadpunk | damn, missed https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/898947 - it's merged on Zed and I thought it's 2023.1 | 23:09 |
noonedeadpunk | but other then that I don't see really outstanding things for https://review.opendev.org/q/parentproject:openstack/openstack-ansible+branch:%255Estable/2023.1+status:open+ | 23:11 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!