| jrosser | good morning | 09:46 |
|---|---|---|
| jrosser | mnasiadka: have you seen anything like this before? https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_0f2/901185/8/check/openstack-ansible-upgrade-aio_metal_tls-ubuntu-jammy/0f27395/logs/host/utility/stestr_results.html | 10:26 |
| jrosser | Details: [{'request_id': '', 'code': 'client', 'status': 400, 'title': 'Cluster type (vm, Unset, kubernetes) not supported', 'detail': 'Cluster type (vm, Unset, kubernetes) not supported.', 'links': []}] | 10:26 |
| mnasiadka | Unset/None - that's a bit weird | 10:27 |
| jrosser | it is | 10:27 |
| jrosser | all the other jobs on that change are passing | 10:27 |
| jrosser | this one is an upgrade (install N-1 then upgrade to N) and enable TLS on the service backends during the upgrade | 10:28 |
| jrosser | but theres no obvious error or exception around magnum or glance | 10:28 |
| jrosser | just not really knowing where to start to debug it :) | 10:28 |
| mnasiadka | if that uses Magnum master - it got broken by oslo.db bump - we're fixing it in https://review.opendev.org/c/openstack/magnum/+/910512 (and other patches in that stack) - so it might be related, but it looks weird ;-) | 10:29 |
| noonedeadpunk | well, it's like that for couple of month already, I guess at least since January or so | 10:42 |
| jrosser | noonedeadpunk: did you manage to reproduce this, or is it OK locally? | 10:43 |
| noonedeadpunk | I was not back in the days and forgot to spawn aio last week. Can do so now... | 10:43 |
| *** tosky_ is now known as tosky | 11:36 | |
| opendevreview | Merged openstack/openstack-ansible-ops master: Add hook playbook install and test magnum capi driver https://review.opendev.org/c/openstack/openstack-ansible-ops/+/906363 | 14:03 |
| mgariepy | is there a way to undefine a var for some hosts ? | 15:36 |
| mgariepy | i did try to set the var in host_var but it's not quite working. | 15:36 |
| jrosser | what happens? | 15:38 |
| mgariepy | it does pick up the user_var | 15:38 |
| jrosser | usual ansible precedence rules should apply | 15:38 |
| jrosser | oh well user vars is an ansible "extra var" so wins all the time | 15:39 |
| jrosser | move things from user_variables to some group_vars/blah is probably the way to approach this | 15:39 |
| mgariepy | ho. | 15:39 |
| mgariepy | good i'll do that. | 15:39 |
| jrosser | then your host_var should win vs. the group_var | 15:39 |
| mgariepy | thanks a lot | 15:40 |
| mgariepy | config override does it have a way to remove a line ? | 15:40 |
| jrosser | https://docs.ansible.com/ansible/latest/playbook_guide/playbooks_variables.html#understanding-variable-precedence | 15:40 |
| jrosser | user vars are the last one on the list, most powerful setting | 15:40 |
| mgariepy | in my head host would be more specific but yeah makes sense since it's extra one. | 15:41 |
| jrosser | i thought that it was possible to drop a line with config override | 15:43 |
| jrosser | perhaps here https://github.com/openstack/ansible-config_template/blob/ca25191181ef1d1269afe95ad0911bac8d4d0610/tests/test.yml#L262 | 15:47 |
| opendevreview | Merged openstack/ansible-role-python_venv_build master: Workaround ansible unsafe text templating bug https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/910477 | 16:03 |
| mgariepy | the moving to group_vars will probably be better for my future-self :D | 16:19 |
| noonedeadpunk | fwiw, it's not possible to undefine var in ansible... | 16:29 |
| mgariepy | yeah found some 2017 issue not closed. | 16:30 |
| jrosser | what does `"{{ undef() }}"` do? | 16:50 |
| jrosser | docs say `The Jinja2 undef() function returns a Python AnsibleUndefined object, derived from jinja2.StrictUndefined. Use undef() to undefine variables of lesser precedence` | 16:50 |
| mgariepy | anyway for my case it wouldn't had worked since the precedences was not quite right :D | 16:56 |
| frickler | noonedeadpunk: fyi reno build now passing, added some comments https://review.opendev.org/c/openstack/openstack-ansible/+/908499 | 17:06 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: reno: Update master for unmaintained/yoga https://review.opendev.org/c/openstack/openstack-ansible/+/908499 | 17:08 |
| noonedeadpunk | oh, thanks | 17:08 |
| noonedeadpunk | eventually, I think with the patch in topic, there's no need anymore to update branches to unmaintained. Or well ,depends I guess | 17:08 |
| noonedeadpunk | at least, it's supposed to detect what replaces the branch (eom tag) | 17:09 |
| frickler | noonedeadpunk: yes, it depends on whether people want to actually publish renos from the unmaintained branches, seems there are mixed opinions on that | 17:11 |
| * noonedeadpunk need to also handle access to unmaintained branches... | 17:12 | |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: Clean up zuul jobs https://review.opendev.org/c/openstack/openstack-ansible-ops/+/910972 | 17:46 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-ops master: Clean up zuul jobs https://review.opendev.org/c/openstack/openstack-ansible-ops/+/910972 | 17:50 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_neutron master: Add support for ovn-bgp-agent deployment https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/909780 | 18:07 |
| opendevreview | Merged openstack/openstack-ansible-os_neutron master: Run neutron OVN agents as neutron user https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/909525 | 18:20 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_neutron master: [doc] Briefly describe VPNaaS plugin changes with OVN https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/910981 | 18:32 |
| ThiagoCMC | When VPNaaS with Wireguard? lol | 18:48 |
| mgariepy | as soon as you implement it in ovn ;p ThiagoCMC haha | 18:54 |
| ThiagoCMC | LOL | 18:56 |
| opendevreview | Merged openstack/ansible-role-uwsgi stable/2023.1: Remove undefined bionic linters job https://review.opendev.org/c/openstack/ansible-role-uwsgi/+/910190 | 20:02 |
| opendevreview | Merged openstack/openstack-ansible-ops master: Clean up zuul jobs https://review.opendev.org/c/openstack/openstack-ansible-ops/+/910972 | 20:10 |
| opendevreview | Jimmy McCrory proposed openstack/openstack-ansible-galera_server master: Additional TLS configuration options https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/911009 | 22:54 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!