| mnasiadka | noonedeadpunk: https://docs.ansible.com/ansible/latest/playbook_guide/playbooks_reuse_roles.html#role-argument-validation - if you mean this, then it might make sense for OSA or standalone roles - but in Kolla-Ansible case where we have all roles in one repository it's probably some sort of an overkill - we usuallly have everything in defaults with some default values :) | 06:43 |
|---|---|---|
| noonedeadpunk | mnasiadka: we have also all in defaults. and that's the reason why we do import defaults file to the doc and did just some comments in defaults around... | 06:46 |
| noonedeadpunk | but if there would be a sphinx extension that can parse argument_specs and document it in a way of, say, policy files... then both documentation and validation of input is solved | 06:48 |
| mnasiadka | yeah, that could be something we could do | 06:59 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-openstack_hosts master: Apply a tag to the pki role include https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/942343 | 08:54 |
| noonedeadpunk | andrewbonney: about regexp... I think, it should be adequate escaping to what I see in some examples. | 09:13 |
| noonedeadpunk | but then indeed - maybe we wanna play with https://docs.ansible.com/ansible/latest/collections/ansible/builtin/regex_escape_filter.html | 09:13 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Auto-fix yaml rules https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/941903 | 09:15 |
| jrosser | I did put some of the double-slash stuff into regex101 and just ended up confused | 09:16 |
| noonedeadpunk | jrosser: so you need to escape slash when using double quotes | 09:17 |
| noonedeadpunk | then, ansible removes one slash and leaves another one | 09:18 |
| noonedeadpunk | and behaviour with `''` is different then with `""` | 09:18 |
| noonedeadpunk | they do have some examples in https://docs.ansible.com/ansible/latest/collections/ansible/builtin/regex_search_filter.html#examples | 09:19 |
| noonedeadpunk | but the overall thing, is that if you wanna leave `\.` as a result, you need to escape `\` when yousing double quites | 09:20 |
| noonedeadpunk | *quotes | 09:20 |
| noonedeadpunk | *using | 09:20 |
| noonedeadpunk | so you can do either `'127\.0\.0\.1'` or `"127\\.0\\.0\\.1"` | 09:21 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_neutron master: Return defenition for wsgi-required workers https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/942347 | 09:27 |
| noonedeadpunk | damn it, this uwsgi thing is just /o\ | 09:28 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_neutron master: Return defenition for wsgi-required workers https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/942347 | 09:30 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_neutron master: Auto-fix yaml rules https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/941376 | 09:30 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_neutron master: Auto-fix usage of modules via FQCN https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/941378 | 09:30 |
| noonedeadpunk | I really wonder wtf is that: https://zuul.opendev.org/t/openstack/build/7bfe965e949343638c6b06fe5bf45a91 | 10:03 |
| noonedeadpunk | like - ansible-lint does resolve symlink to destination path? | 10:03 |
| noonedeadpunk | as we run `/opt/ansible-runtime/bin/ansible-lint /etc/ansible/roles/httpd/examples/playbook.yml` | 10:03 |
| noonedeadpunk | but then issue happens in `/home/zuul/src/opendev.org/openstack/ansible-role-httpd/examples/playbook.yml:5:7` | 10:03 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-httpd master: Split example and converge playbooks https://review.opendev.org/c/openstack/ansible-role-httpd/+/942349 | 10:31 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-httpd master: Split example and converge playbooks https://review.opendev.org/c/openstack/ansible-role-httpd/+/942349 | 11:46 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-httpd master: Auto-fix yaml rules https://review.opendev.org/c/openstack/ansible-role-httpd/+/941901 | 11:58 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-httpd master: Split example and converge playbooks https://review.opendev.org/c/openstack/ansible-role-httpd/+/942349 | 12:00 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-httpd master: Auto-fix yaml rules https://review.opendev.org/c/openstack/ansible-role-httpd/+/941901 | 12:00 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-httpd master: Auto-fix yaml rules https://review.opendev.org/c/openstack/ansible-role-httpd/+/941901 | 12:04 |
| opendevreview | Merged openstack/openstack-ansible-os_blazar master: Auto-fix yaml rules https://review.opendev.org/c/openstack/openstack-ansible-os_blazar/+/941323 | 13:08 |
| opendevreview | Merged openstack/openstack-ansible-os_heat master: Auto-fix yaml rules https://review.opendev.org/c/openstack/openstack-ansible-os_heat/+/941342 | 18:09 |
| opendevreview | Merged openstack/openstack-ansible-lxc_hosts master: Auto-fix yaml rules https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/941528 | 18:14 |
| sykeben | Good afternoon, was wondering if anyone could help me pinpoint the source of this error | 18:55 |
| sykeben | fatal: [dev-cc-01]: FAILED! => {"msg": "The task includes an option with an undefined variable.. {{ haproxy_nova_spice_console_service | combine(haproxy_nova_spice_console_service_overrides | default({})) }}: 'haproxy_nova_spice_console_service' is undefined\n\nThe error appears to be in '/etc/ansible/roles/haproxy_server/tasks/haproxy_service_config.yml': line 22, column 3, but may\nbe elsewhere in the file depending on the exact syntax p | 18:56 |
| sykeben | I started getting this after deploying openstack-ansible (tag: 30.0.1) upgrade from 29.2.1 | 18:56 |
| sykeben | I haven't changed anything to do with Nova or the spice console in the meantime | 18:57 |
| sykeben | This is the command I'm using (trying to install ceilometer, gnocchi and cloudkitty): "openstack-ansible openstack.osa.setup_openstack --limit metering-infra_hosts --limit metering-compute_hosts --limit metrics_hosts --limit cloudkitty_hosts" | 18:57 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!