| jrosser | moving the playbooks to a collection is pretty involved | 07:22 |
|---|---|---|
| jrosser | i think it's quite some work compared to just copy/paste the files across | 07:22 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use openstack.osa.install_defaults role instead of vars_files https://review.opendev.org/c/openstack/openstack-ansible/+/923358 | 07:47 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use openstack.osa.install_defaults role instead of vars_files https://review.opendev.org/c/openstack/openstack-ansible/+/923358 | 07:51 |
| opendevreview | Andrew Bonney proposed openstack/openstack-ansible-openstack_openrc master: Add openrc_clouds_yml_file_default_scope variable https://review.opendev.org/c/openstack/openstack-ansible-openstack_openrc/+/923359 | 08:05 |
| opendevreview | Andrew Bonney proposed openstack/openstack-ansible-openstack_openrc master: Prevent conflicts between openrc files https://review.opendev.org/c/openstack/openstack-ansible-openstack_openrc/+/923360 | 08:05 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-plugins master: Add haproxy_endpoint_manage role https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/923363 | 08:35 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Remove remove_container_journal common task file https://review.opendev.org/c/openstack/openstack-ansible/+/923366 | 08:58 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Remove dynamic-grouping common task file https://review.opendev.org/c/openstack/openstack-ansible/+/923367 | 08:58 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use haproxy_endpoint_manage role from osa collection rather than common-tasks https://review.opendev.org/c/openstack/openstack-ansible/+/923368 | 09:03 |
| jrosser | noonedeadpunk: if you are around today would be interested in opinion on this https://github.com/openstack/openstack-ansible/blame/master/playbooks/common-tasks/os-log-dir-setup.yml | 09:10 |
| jrosser | it looks almost unused, and some special case only for utility host in a metal deploy | 09:10 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use openstack.osa.install_defaults role instead of vars_files https://review.opendev.org/c/openstack/openstack-ansible/+/923358 | 11:04 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Remove remove_container_journal common task file https://review.opendev.org/c/openstack/openstack-ansible/+/923366 | 11:04 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Remove dynamic-grouping common task file https://review.opendev.org/c/openstack/openstack-ansible/+/923367 | 11:04 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use openstack.osa.install_defaults role instead of vars_files https://review.opendev.org/c/openstack/openstack-ansible/+/923358 | 11:17 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Remove remove_container_journal common task file https://review.opendev.org/c/openstack/openstack-ansible/+/923366 | 11:17 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Remove dynamic-grouping common task file https://review.opendev.org/c/openstack/openstack-ansible/+/923367 | 11:17 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use haproxy_endpoint_manage role from osa collection rather than common-tasks https://review.opendev.org/c/openstack/openstack-ansible/+/923368 | 11:17 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use openstack.osa.install_defaults role instead of vars_files https://review.opendev.org/c/openstack/openstack-ansible/+/923358 | 11:18 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Remove remove_container_journal common task file https://review.opendev.org/c/openstack/openstack-ansible/+/923366 | 11:19 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Remove dynamic-grouping common task file https://review.opendev.org/c/openstack/openstack-ansible/+/923367 | 11:19 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use haproxy_endpoint_manage role from osa collection rather than common-tasks https://review.opendev.org/c/openstack/openstack-ansible/+/923368 | 11:19 |
| opendevreview | Merged openstack/openstack-ansible-galera_server master: Remove xinetd clean-up tasks https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/922819 | 11:58 |
| noonedeadpunk | jrosser: yeah, that pretty much looks not very important and I think we can drop safely | 12:07 |
| noonedeadpunk | jrosser: there's actually a quite big consideration about install_defaults role - that there's no good way (except extra_vars) on how to override vars there | 12:11 |
| jrosser | hmm - i wonder if this will work https://review.opendev.org/c/openstack/openstack-ansible/+/923358/5/playbooks/healthcheck-infrastructure.yml | 12:12 |
| noonedeadpunk | I was thinking... Maybe we can change the role a bit, and instead do defaults_from? | 12:13 |
| jrosser | sorry this https://review.opendev.org/c/openstack/openstack-ansible/+/923358/5/playbooks/healthcheck-infrastructure.yml#96 | 12:13 |
| jrosser | note that it is `import_role` too, this is the only way to make it work | 12:14 |
| jrosser | `include_role` does not leave the role vars in scope past where it is included | 12:14 |
| noonedeadpunk | import has `defaults_from` as well | 12:14 |
| noonedeadpunk | hm | 12:15 |
| noonedeadpunk | I can recall you had a hussle with that... | 12:15 |
| jrosser | yes i already had to put this in the plugins repo for the mcapi stuff and it was pretty awkward | 12:15 |
| jrosser | so the remaining work is to tidy up / convert the rest of the usage of it | 12:15 |
| jrosser | but having said that, the mcapi use case is only source install so there could be issues there | 12:16 |
| noonedeadpunk | I _think_ include_role works now as well, just in case | 12:26 |
| noonedeadpunk | so... this would work as well from what I can tell but would fix variable scoping... | 12:30 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Modify install_defaults to be more overridable https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/923390 | 12:35 |
| noonedeadpunk | this ^ | 12:35 |
| noonedeadpunk | and then: https://paste.openstack.org/show/bYGhgVk0hlHHjcKuK73Q/ | 12:35 |
| noonedeadpunk | (sorry, drop `_new` in role name - was testing without messing up too much | 12:35 |
| noonedeadpunk | I was checking for `openstack_db_setup_host` inside executed role and it was there as long as role was imported/includd | 12:36 |
| noonedeadpunk | if that makes any sense? | 12:40 |
| noonedeadpunk | interestingly, such import is not even renderred in logs | 12:41 |
| noonedeadpunk | and, I can confirm that in current setup group_vars are ignored | 12:44 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-ops master: Update format of install_defaults https://review.opendev.org/c/openstack/openstack-ansible-ops/+/923392 | 12:49 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-ops master: Update format of install_defaults https://review.opendev.org/c/openstack/openstack-ansible-ops/+/923392 | 12:50 |
| noonedeadpunk | or we can probably test with that ^ | 12:50 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use haproxy_endpoint_manage role from osa collection rather than common-tasks https://review.opendev.org/c/openstack/openstack-ansible/+/923368 | 13:22 |
| jrosser | noonedeadpunk: ^ on this one your advice on correct way to do tags would be good - i always forget which way to do it :) | 13:24 |
| mgariepy | hmm anyone has seen issue with placement allocation ? i have a gpu passthrough and it seems that one vm that have 2 gpus got allocated 3 gpus in placement.. | 13:35 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Remove os-log-dir-setup common playbook https://review.opendev.org/c/openstack/openstack-ansible/+/923402 | 13:37 |
| andrewbonney | mgariepy: I think we saw an issue with the legacy nova tracking method once, but I couldn't replicate it. We haven't had chance to try the placement-based pci tracking yet | 13:38 |
| mgariepy | that's kinda weird.. | 13:41 |
| mgariepy | i think i see what happened. the server got resized from 1 gpus to 2. i guess it didn't cleaned up the allocation on placement afeter the migraiton/resize. | 13:44 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-plugins master: Add gather_extra_facts role https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/923403 | 13:46 |
| *** gaudenz_ is now known as gaudenz | 13:49 | |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use gather_extra_facts role from plugins collection https://review.opendev.org/c/openstack/openstack-ansible/+/923405 | 13:51 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-plugins master: Add unbound_clients role https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/923407 | 14:08 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use unbound_clients role from plugins collection https://review.opendev.org/c/openstack/openstack-ansible/+/923409 | 14:11 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-plugins master: Add dynamic_address_fact role https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/923410 | 14:21 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use dynamic_address_fact role from plugins collection https://review.opendev.org/c/openstack/openstack-ansible/+/923411 | 14:25 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Remove remove_container_journal common task file https://review.opendev.org/c/openstack/openstack-ansible/+/923366 | 14:30 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Remove dynamic-grouping common task file https://review.opendev.org/c/openstack/openstack-ansible/+/923367 | 14:31 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use haproxy_endpoint_manage role from osa collection rather than common-tasks https://review.opendev.org/c/openstack/openstack-ansible/+/923368 | 14:31 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use haproxy_endpoint_manage role from osa collection rather than common-tasks https://review.opendev.org/c/openstack/openstack-ansible/+/923368 | 14:36 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use unbound_clients role from plugins collection https://review.opendev.org/c/openstack/openstack-ansible/+/923409 | 14:42 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Remove remove_container_journal common task file https://review.opendev.org/c/openstack/openstack-ansible/+/923366 | 14:59 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Remove dynamic-grouping common task file https://review.opendev.org/c/openstack/openstack-ansible/+/923367 | 14:59 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use haproxy_endpoint_manage role from osa collection rather than common-tasks https://review.opendev.org/c/openstack/openstack-ansible/+/923368 | 14:59 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_magnum master: Add test for high-availability mcapi control plane https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/923174 | 15:37 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use unbound_clients role from plugins collection https://review.opendev.org/c/openstack/openstack-ansible/+/923409 | 15:51 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Remove remove_container_journal common task file https://review.opendev.org/c/openstack/openstack-ansible/+/923366 | 15:58 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Remove remove_container_journal common task file https://review.opendev.org/c/openstack/openstack-ansible/+/923366 | 16:04 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Remove dynamic-grouping common task file https://review.opendev.org/c/openstack/openstack-ansible/+/923367 | 16:04 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use haproxy_endpoint_manage role from osa collection rather than common-tasks https://review.opendev.org/c/openstack/openstack-ansible/+/923368 | 16:04 |
| opendevreview | Merged openstack/openstack-ansible-openstack_openrc master: Add openrc_clouds_yml_file_default_scope variable https://review.opendev.org/c/openstack/openstack-ansible-openstack_openrc/+/923359 | 17:29 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_magnum master: Add test for high-availability mcapi control plane https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/923174 | 18:59 |
| jrosser | NeilHanlon: quick question - do you know what the kernel headers package is called on rocky so i can make this work there too? https://github.com/openstack/openstack-ansible-ops/blob/master/mcapi_vexxhost/playbooks/k8s_install.yml#L60-L65 | 19:21 |
| jrosser | needs to be the kernel headers that match the running kernel | 19:21 |
| NeilHanlon | jrosser: `kernel-headers-${version}` should do it | 19:23 |
| NeilHanlon | https://rockylinux.pkgs.org/9/rockylinux-appstream-x86_64/kernel-headers-5.14.0-427.22.1.el9_4.x86_64.rpm.html | 19:24 |
| jrosser | do the older versions hang around a bit too? | 19:24 |
| jrosser | as i might have $age-of-ci-image kernel installed | 19:25 |
| jrosser | ah well actually it is that exact version you mention currently `No package linux-headers-5.14.0-427.22.1.el9_4.x86_64 available.` | 19:28 |
| jrosser | so i just need to manipulate the package name between debians and rhel-alikes | 19:28 |
| Balsa | anyone got a good list of examples for flavors? | 20:21 |
| jrosser | Balsa: we made a very simple set like cpu1 cpu2 cpu4.... with 1:1 cpu cores/ram GB, and also std1, std2, std4... with 1:4 cpu cores/ram GB and that has covered most things for most users | 20:35 |
| jrosser | but everyone has different requirements here i think | 20:36 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: Add support for deploying mcapi control plane k8s on rocky linux https://review.opendev.org/c/openstack/openstack-ansible-ops/+/923447 | 20:53 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_magnum master: Add test for high-availability mcapi control plane https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/923174 | 20:54 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: Add support for deploying mcapi control plane k8s on rocky linux https://review.opendev.org/c/openstack/openstack-ansible-ops/+/923447 | 21:27 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!