| opendevreview | Merged openstack/openstack-ansible-os_octavia master: Use PKI role for certificate generation https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/839068 | 01:24 |
|---|---|---|
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Freeze Ansible Roles for Beta release https://review.opendev.org/c/openstack/openstack-ansible/+/844874 | 01:27 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Freeze Ansible Roles for Beta release https://review.opendev.org/c/openstack/openstack-ansible/+/844874 | 01:27 |
| noonedeadpunk | please check that on first possibility ^ :) | 02:00 |
| noonedeadpunk | and I will push rollback for it first time in the morning | 02:00 |
| *** ysandeep|out is now known as ysandeep | 06:15 | |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Unfreeze roles after milestone https://review.opendev.org/c/openstack/openstack-ansible/+/845092 | 07:28 |
| noonedeadpunk | jrosser_: should we discuss https://review.opendev.org/c/openstack/openstack-ansible/+/837551 ? | 07:31 |
| noonedeadpunk | as likely I didn't catch proposal | 07:32 |
| jrosser_ | so its really checking if an override exists from userspace vars | 07:33 |
| jrosser_ | it was just a bit confusing for it seeming to use a role scoped var, but eventually thats not really what the logic is doing | 07:34 |
| noonedeadpunk | basically yes. and assuming at same time that we don't change default in the role, which is nasty... | 07:35 |
| jrosser_ | still trying to wrap my head around why we have is_metal on this too | 07:40 |
| noonedeadpunk | um.... that is interesting indeed | 07:45 |
| jrosser_ | all these conditions kind of look simple | 07:45 |
| jrosser_ | but actually it's not :) | 07:45 |
| jrosser_ | because we have a combination of dirs on the host bind mounted into the container | 07:45 |
| jrosser_ | or alternatively systemd_mount used inside the container | 07:46 |
| noonedeadpunk | eventually we always should run os-lxc-container-setup only against lxc host | 07:46 |
| jrosser_ | and the (nfs) / (non-nfs) task names are kind of wrong now | 07:46 |
| noonedeadpunk | which is kind of metal I guess | 07:46 |
| jrosser_ | and then glance_container_bind_mounts can be made custom for whatever else, but is ignored for some cases | 07:47 |
| * jrosser_ head explodes :) | 07:47 | |
| noonedeadpunk | but we have not metal everywhere | 07:47 |
| noonedeadpunk | but I don;t see any delegate or anything... | 07:48 |
| jrosser_ | somehow we have this being very complicated behaviour, which is why i was hesitating with reviewing it | 07:48 |
| noonedeadpunk | And at same time I was pretty sure these bind mounts made are working quite properly | 07:49 |
| noonedeadpunk | as I bet we use that at least for galera and it;s same there https://opendev.org/openstack/openstack-ansible/src/branch/master/playbooks/galera-install.yml#L33 | 07:50 |
| jrosser_ | right, becasue for both galera and glance we want to persist the storage when re-creating the container, and the backing is some local disk | 07:51 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Replace glance_nfs_client https://review.opendev.org/c/openstack/openstack-ansible/+/837551 | 07:56 |
| jrosser_ | i think it's OK, theres just a lot going on with what the mount is (or isnt, in the case of rbd for example) | 07:56 |
| jrosser_ | i fixed the task names to make it a bit more obvious | 07:57 |
| *** ysandeep is now known as ysandeep|lunch | 07:58 | |
| noonedeadpunk | yeah, looks way more clear, thanks | 08:06 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-lxc_container_create master: Add ovs support for lxc containers. https://review.opendev.org/c/openstack/openstack-ansible-lxc_container_create/+/837734 | 08:10 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Add ability to define bridge type for containers https://review.opendev.org/c/openstack/openstack-ansible/+/837742 | 08:10 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Add ability to define bridge type for containers https://review.opendev.org/c/openstack/openstack-ansible/+/837742 | 08:10 |
| *** anbanerj is now known as frenzy_friday | 08:30 | |
| *** ysandeep|lunch is now known as ysandeep | 08:44 | |
| jrosser_ | noonedeadpunk: i wonder if this condition is still needed https://github.com/openstack/openstack-ansible/blob/master/tests/roles/bootstrap-host/tasks/prepare_aio_config.yml#L80-L81 | 09:53 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use nodepool centos and epel mirrors for CI jobs on centos-9 https://review.opendev.org/c/openstack/openstack-ansible/+/845112 | 10:00 |
| noonedeadpunk | I think they are not. | 10:13 |
| noonedeadpunk | And also I guess that we can also install lxc for centos 9 from epel now | 10:13 |
| jrosser_ | did you have a patch for that? | 10:37 |
| noonedeadpunk | nah, I don't | 10:48 |
| opendevreview | shahab taee proposed openstack/openstack-ansible-os_neutron master: custom configuration for vpnaas https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/844041 | 11:10 |
| opendevreview | Merged openstack/openstack-ansible-os_neutron master: Implement uWSGI for neutron-api https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/486156 | 11:19 |
| *** dviroel|afk is now known as dviroel | 11:19 | |
| spatel | trying to hunt down noonedeadpunk | 11:23 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Freeze Ansible Roles for Beta release https://review.opendev.org/c/openstack/openstack-ansible/+/844874 | 12:39 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Unfreeze roles after milestone https://review.opendev.org/c/openstack/openstack-ansible/+/845092 | 12:39 |
| opendevreview | shahab taee proposed openstack/openstack-ansible-os_neutron master: custom configuration for vpnaas https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/844041 | 14:16 |
| *** ysandeep is now known as ysandeep|out | 14:31 | |
| *** dviroel is now known as dviroel|lunch | 14:47 | |
| *** dviroel|lunch is now known as dviroel | 16:05 | |
| opendevreview | Merged openstack/openstack-ansible master: Add mistra-extra repo https://review.opendev.org/c/openstack/openstack-ansible/+/844048 | 16:28 |
| lowercase | Anyone use mnesia with rabbitmq? | 17:43 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Freeze Ansible Roles for Beta release https://review.opendev.org/c/openstack/openstack-ansible/+/844874 | 17:47 |
| jrosser_ | noonedeadpunk: ^ shastest fails on os_neutron "reference is not a tree" so i tried to bump it forward to todays SHA | 17:48 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Unfreeze roles after milestone https://review.opendev.org/c/openstack/openstack-ansible/+/845092 | 17:48 |
| *** dviroel is now known as dviroel|afk | 20:27 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!