| opendevreview | Merged openstack/openstack-ansible stable/xena: Mark OSA repository as safe in git.config in CI https://review.opendev.org/c/openstack/openstack-ansible/+/837686 | 00:11 |
|---|---|---|
| opendevreview | Merged openstack/openstack-ansible stable/victoria: Mark OSA repository as safe in git.config in CI https://review.opendev.org/c/openstack/openstack-ansible/+/837688 | 00:31 |
| opendevreview | Merged openstack/openstack-ansible stable/wallaby: Mark OSA repository as safe in git.config in CI https://review.opendev.org/c/openstack/openstack-ansible/+/837687 | 00:50 |
| opendevreview | Merged openstack/openstack-ansible-rsyslog_server master: Updated from OpenStack Ansible Tests https://review.opendev.org/c/openstack/openstack-ansible-rsyslog_server/+/835731 | 01:19 |
| opendevreview | Merged openstack/openstack-ansible-os_tempest master: Do not redefine variables with their default values in tempest.conf https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/825157 | 02:08 |
| opendevreview | Merged openstack/openstack-ansible-os_tempest master: Implement variable: tempest_endpoint_type https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/825156 | 02:43 |
| opendevreview | Merged openstack/openstack-ansible-os_tempest master: Rename [orchestration] section to [heat_plugin] in tempest.conf https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/825163 | 02:52 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-plugins master: Add role for creating simple glusterfs clients/servers https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/837582 | 07:34 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-plugins master: Add role for creating simple glusterfs clients/servers https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/837582 | 07:36 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Do not encrypt SSL for CentOS distro path https://review.opendev.org/c/openstack/openstack-ansible/+/837571 | 07:38 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-repo_server master: Add facility to store repo contents on a remote mount https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/837587 | 07:41 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use glusterfs to synchronise repo server contents https://review.opendev.org/c/openstack/openstack-ansible/+/837589 | 07:45 |
| noonedeadpunk | should actually ^ depend on 837588 ? | 07:48 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use glusterfs to synchronise repo server contents https://review.opendev.org/c/openstack/openstack-ansible/+/837589 | 07:59 |
| jrosser | noonedeadpunk: well - i added a temporary var in 837588 `repo_server_enable_sync_manager` in order to be able to manage a transition from lsync->glusterfs | 08:03 |
| jrosser | i'm trying to figure out how to merge this stuff now as i can't break the openstack-ansible deploy with changes to the repo_server role as it's all circular | 08:09 |
| noonedeadpunk | ah, I see.... | 08:22 |
| noonedeadpunk | well, maybe we should then create some dnm change on top of 837588 that would test things for real? | 08:22 |
| noonedeadpunk | * on top of 837589 | 08:27 |
| jrosser | i was thinking that 837589 was that patch to test it all together | 08:46 |
| jrosser | that now sets the flag to disable lsyncd and it defines the mount to pass to systemd_mount in inventory/group_vars/repo_all.yml | 08:46 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use glusterfs to synchronise repo server contents https://review.opendev.org/c/openstack/openstack-ansible/+/837589 | 08:55 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Cleanup distro vars https://review.opendev.org/c/openstack/openstack-ansible/+/837845 | 09:24 |
| opendevreview | Merged openstack/openstack-ansible master: [doc] Add doc how to pass SR-IOV inside containers https://review.opendev.org/c/openstack/openstack-ansible/+/833775 | 10:35 |
| opendevreview | Merged openstack/openstack-ansible-plugins master: Pass ssh_keypairs role in check mode https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/837713 | 10:43 |
| *** dviroel|afk is now known as dviroel | 11:24 | |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Allow mq_setup role to run in check mode https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/837856 | 11:38 |
| opendevreview | Merged openstack/openstack-ansible-openstack_hosts master: Switch codename to Yoga https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/837702 | 11:38 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-plugins master: Add role for creating simple glusterfs clients/servers https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/837582 | 11:44 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-plugins master: Add role for creating simple glusterfs clients/servers https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/837582 | 11:44 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-plugins master: Add role for creating simple glusterfs servers https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/837582 | 11:45 |
| opendevreview | Merged openstack/ansible-role-pki master: Allow to run PKI role in check mode https://review.opendev.org/c/openstack/ansible-role-pki/+/837722 | 11:49 |
| opendevreview | Merged openstack/openstack-ansible-rabbitmq_server master: Pass rabbitmq role in check mode https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/837718 | 11:57 |
| opendevreview | Merged openstack/openstack-ansible-rabbitmq_server master: Replace shell with simple command for version verification https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/837719 | 11:57 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-repo_server master: Remove all code for lsync, rsync and ssh https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/837588 | 12:09 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-repo_server master: Clean up legacy lsycnd, rsync and ssh key config https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/837859 | 12:09 |
| *** dviroel is now known as dviroel|brb | 12:19 | |
| mgariepy | noonedeadpunk, did i miss your post or it never got posted by the moderator ? | 12:19 |
| noonedeadpunk | I haven't posted it yet :( | 12:24 |
| noonedeadpunk | or you regarding PTG? | 12:24 |
| mgariepy | was about damiandabrowski[m] :)\ | 12:25 |
| noonedeadpunk | yeah... I failed to send it at all, and now it's easter... | 12:26 |
| noonedeadpunk | I recalled I have to just couple of hours ago | 12:27 |
| mgariepy | ok no worries. | 12:30 |
| mgariepy | is there any reviews that need attention this morning ? | 12:30 |
| opendevreview | Merged openstack/openstack-ansible-os_glance master: Allow os_glance to run in check mode https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/837739 | 12:41 |
| opendevreview | Merged openstack/openstack-ansible master: Allow to run utility playbook in check mode https://review.opendev.org/c/openstack/openstack-ansible/+/837724 | 12:51 |
| *** dviroel|brb is now known as dviroel | 12:51 | |
| jrosser | noonedeadpunk: do you have a centos lxc aio around? | 12:51 |
| noonedeadpunk | yep I do | 12:54 |
| jrosser | could you see if systemd-udev package is present in a container? | 12:54 |
| noonedeadpunk | nope | 12:55 |
| noonedeadpunk | in terms - it's not there :) | 12:55 |
| jrosser | if you install it, do you get systemd-tmpfiles-setup-dev.service running? | 12:55 |
| noonedeadpunk | after installation it's inactive (dead) | 12:56 |
| jrosser | so that would need enabled:yes and state: restarted | 12:57 |
| noonedeadpunk | yeah,I think it needs that - decided to paste status:) https://paste.openstack.org/show/bxlyHYk6ZSVAThBEWgat/ | 12:58 |
| jrosser | ok cool thanks | 12:58 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-lxc_hosts master: Ensure systemd-udev is present in centos containers https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/837865 | 13:00 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-plugins master: Add role for creating simple glusterfs servers https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/837582 | 13:21 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-repo_server master: Clean up legacy lsycnd, rsync and ssh key config https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/837859 | 13:29 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-repo_server master: Remove all code for lsync, rsync and ssh https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/837588 | 13:30 |
| opendevreview | Merged openstack/openstack-ansible-os_neutron master: Add parameters to limit the number of DHCP or L3 agents https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/833769 | 13:34 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-lxc_hosts master: Ensure systemd-udev is present in centos containers https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/837865 | 13:36 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-repo_server master: Clean up legacy lsycnd, rsync and ssh key config https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/837859 | 13:53 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-repo_server master: Clean up legacy lsycnd, rsync and ssh key config https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/837859 | 14:20 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-repo_server master: Clean up legacy lsycnd, rsync and ssh key config https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/837859 | 14:21 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-repo_server master: Remove all code for lsync, rsync and ssh https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/837588 | 14:22 |
| *** chkumar|ruck is now known as raukadah | 14:31 | |
| *** dviroel is now known as dviroel|lunch | 15:12 | |
| jrosser | noonedeadpunk: do you remember if we have a neat way of making only a systemd drop-in? | 15:33 |
| jrosser | ah yes https://opendev.org/openstack/openstack-ansible-rabbitmq_server/src/branch/master/tasks/rabbitmq_post_install.yml#L48-L57 | 15:37 |
| noonedeadpunk | yup :) | 15:40 |
| * jrosser has to make hacks for centos / glusterfs | 15:47 | |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-config_template master: Pass context to copy module https://review.opendev.org/c/openstack/ansible-config_template/+/837897 | 15:51 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_cinder master: Allow to run cinder role in check mode https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/837903 | 16:06 |
| jrosser | oh thats interesting - we don't reload systemd when placing a drop-in https://github.com/openstack/ansible-role-systemd_service/blob/master/tasks/main.yml#L141 | 16:12 |
| noonedeadpunk | question if it's required? | 16:20 |
| noonedeadpunk | As Iguess it's not? | 16:20 |
| *** dviroel|lunch is now known as dviroel | 16:20 | |
| jrosser | well - when i use systemd_service to install one it doesnt het picked up | 16:20 |
| noonedeadpunk | huh | 16:20 |
| noonedeadpunk | I just thought that daemon reload was needed only when changing system provided ones, and overrides were picked anyway | 16:21 |
| jrosser | certinaly on centos dropins are showing up in systemctl status like this https://paste.opendev.org/show/bjmCKXBNeQZGSrdd93KK/ | 16:22 |
| noonedeadpunk | yeah, you're right | 16:22 |
| jrosser | but only after i do daemon-reload on it | 16:22 |
| * jrosser makes patch | 16:22 | |
| noonedeadpunk | I was soooo sure I either patched that or that was done somewhere else... | 16:22 |
| jrosser | right - i wonder how its working for rabbitmq | 16:23 |
| noonedeadpunk | I think we reload somewhere in handlers as welll... | 16:23 |
| jrosser | theres an ordering issue too becasue if the drop in is placed before the service unit exists it doesnt like that either | 16:23 |
| jrosser | so that makes it harder to have include_tasks: gross_fixups_for_centos.yml, instead those tasks get spread around the others | 16:24 |
| noonedeadpunk | for rabbit it works though somehow | 16:24 |
| jrosser | it does | 16:25 |
| noonedeadpunk | as I checked aio I just spawned hours ago and rabbit has override loaded in service status | 16:25 |
| noonedeadpunk | yeah... well... | 16:26 |
| opendevreview | Jonathan Rosser proposed openstack/ansible-role-systemd_service master: Reload systemd after installing a service unit override https://review.opendev.org/c/openstack/ansible-role-systemd_service/+/837930 | 16:28 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-plugins master: Add role for creating simple glusterfs servers https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/837582 | 16:30 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use glusterfs to synchronise repo server contents https://review.opendev.org/c/openstack/openstack-ansible/+/837589 | 16:31 |
| noonedeadpunk | jrosser: in systemd patch you messed thing a bit | 16:34 |
| jrosser | doh | 16:34 |
| opendevreview | Jonathan Rosser proposed openstack/ansible-role-systemd_service master: Reload systemd after installing a service unit override https://review.opendev.org/c/openstack/ansible-role-systemd_service/+/837930 | 16:35 |
| jrosser | now this is what happens when i *dont* use the vscode remote stuff and copy/paste things out of my vm | 16:35 |
| jrosser | certainly feels like getting 80% of this gluster stuff working took 20% of the time | 16:36 |
| noonedeadpunk | yeah, well, that's always like that:) | 16:42 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_neutron master: Allow os_neutron to run in check mode https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/837932 | 16:48 |
| jrosser | theres not much left i think - perhaps deciding if we always make the /var/lib/glusterd bind mount here https://review.opendev.org/c/openstack/openstack-ansible/+/837589/9/playbooks/repo-install.yml#37 | 16:49 |
| jrosser | or if that should be conditional | 16:49 |
| jrosser | and then there is the delete or not delete the contents of /openstack/glusterd/<container> with lxc_destroy | 16:49 |
| jrosser | the cluster uuid is in there and if that gets re-created then things don't work when delete/replace a container | 16:50 |
| jrosser | so it's good to persist it for that use case of rebuilding a repo container | 16:51 |
| jrosser | but i'm not sure yet how to deal with a complete node replacement, there would need to be an additional step somewhere | 16:51 |
| jrosser | there is a UUID in /var/lib/glusterd/glusterd.info which needs to be set on a replaced node | 16:54 |
| noonedeadpunk | maybe fetch it and put on deploy host at some step? | 16:56 |
| noonedeadpunk | or even put in user_secrets? | 16:57 |
| noonedeadpunk | and let it be generated? | 16:57 |
| noonedeadpunk | regarding bind mount - I guess it can be jsut a variable | 16:57 |
| noonedeadpunk | like https://opendev.org/openstack/openstack-ansible/src/branch/master/playbooks/galera-install.yml#L35 | 16:58 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-plugins master: Add role for creating simple glusterfs servers https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/837582 | 17:50 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-plugins master: Add role for creating simple glusterfs servers https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/837582 | 18:54 |
| jrosser | NeilHanlon: would be interested to know about the correct approach for gluster server installation is for rocky https://zuul.opendev.org/t/openstack/build/f99b7cc2410246a28ad9a5cb658f9ab7/log/job-output.txt#9634 | 19:56 |
| jrosser | it's currently doing this for centos https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/837582/15/roles/glusterfs/vars/redhat.yml | 19:57 |
| opendevreview | Merged openstack/ansible-role-systemd_service master: Pass systemd_service role in check mode https://review.opendev.org/c/openstack/ansible-role-systemd_service/+/837716 | 19:58 |
| opendevreview | Damian Dąbrowski proposed openstack/openstack-ansible-os_tempest master: do not include [*-feature-enabled] sections in tempest.conf https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/825164 | 20:52 |
| *** dviroel is now known as dviroel|out | 21:24 | |
| opendevreview | Merged openstack/openstack-ansible-os_cinder master: Add the ability to disable send_actions option in cinder-volume-usage-audit service. https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/837570 | 23:44 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!