*** ysandeep|out is now known as ysandeep | 02:49 | |
*** ysandeep is now known as ysandeep|afk | 05:13 | |
*** ysandeep|afk is now known as ysandeep | 06:25 | |
opendevreview | Ebbex proposed openstack/openstack-ansible-galera_server master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/857812 | 07:07 |
---|---|---|
ebbex | Good morning! Continuing the spam from yesterday :P | 07:08 |
opendevreview | Ebbex proposed openstack/openstack-ansible-ceph_client master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/857813 | 07:10 |
noonedeadpunk | morning | 07:47 |
noonedeadpunk | I will review all at once if no objections | 07:48 |
opendevreview | Ebbex proposed openstack/openstack-ansible-haproxy_server master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/857821 | 07:54 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-haproxy_server master: Add variable for setting certbot `domains` option https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/857719 | 08:03 |
*** ysandeep is now known as ysandeep|afk | 08:07 | |
opendevreview | Ebbex proposed openstack/openstack-ansible-lxc_hosts master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/857822 | 08:15 |
opendevreview | Ebbex proposed openstack/openstack-ansible-memcached_server master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-memcached_server/+/857823 | 08:18 |
opendevreview | Ebbex proposed openstack/openstack-ansible-openstack_hosts master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/857825 | 08:21 |
anskiy | noonedeadpunk: hello, like I've said earlier, I can split that change in two: one for fix and one for new things, if you want. | 08:31 |
opendevreview | Ebbex proposed openstack/openstack-ansible-os_adjutant master: Refactor use of include_vars https://review.opendev.org/c/openstack/openstack-ansible-os_adjutant/+/857827 | 08:32 |
noonedeadpunk | anskiy: doesn't really matter. I just fixed "typo" issue in the patch | 08:40 |
opendevreview | Ebbex proposed openstack/openstack-ansible-os_aodh master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_aodh/+/857828 | 08:40 |
anskiy | okay, I hope it would work :) | 08:41 |
opendevreview | Ebbex proposed openstack/openstack-ansible-os_blazar master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_blazar/+/857829 | 08:47 |
opendevreview | Ebbex proposed openstack/openstack-ansible-os_ceilometer master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_ceilometer/+/857830 | 08:51 |
opendevreview | Ebbex proposed openstack/openstack-ansible-os_gnocchi master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_gnocchi/+/857831 | 08:53 |
*** ysandeep|afk is now known as ysandeep | 08:53 | |
opendevreview | Ebbex proposed openstack/openstack-ansible-os_tacker master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_tacker/+/857833 | 08:57 |
opendevreview | Ebbex proposed openstack/openstack-ansible-rsyslog_client master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-rsyslog_client/+/857837 | 09:13 |
opendevreview | Ebbex proposed openstack/openstack-ansible-rsyslog_server master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-rsyslog_server/+/857846 | 09:16 |
*** ysandeep is now known as ysandeep|brb | 09:51 | |
*** ysandeep|brb is now known as ysandeep | 10:03 | |
opendevreview | Ebbex proposed openstack/ansible-role-systemd_mount master: Remove redundant vars line https://review.opendev.org/c/openstack/ansible-role-systemd_mount/+/857861 | 10:14 |
opendevreview | Ebbex proposed openstack/ansible-role-systemd_networkd master: Remove redundant vars line https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/857863 | 10:27 |
*** dviroel|afk is now known as dviroel | 11:31 | |
*** ysandeep is now known as ysandeep|afk | 12:26 | |
opendevreview | Ebbex proposed openstack/ansible-role-python_venv_build master: Remove redundant vars line https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/857875 | 12:41 |
opendevreview | Ebbex proposed openstack/ansible-role-uwsgi master: Remove redundant vars line https://review.opendev.org/c/openstack/ansible-role-uwsgi/+/857876 | 13:02 |
*** ysandeep|afk is now known as ysandeep | 13:24 | |
noonedeadpunk | jamesdenton: around? I have simple/straightforward and smart complicated solution to the bug with venv_wheel_build_enable | 15:08 |
noonedeadpunk | simple one is just to check for the length of ansible_play_hosts and enable wheels build or not based on that | 15:09 |
noonedeadpunk | complicated is to build another structure that will sort ansible_play_hosts based on their arch/distro, and check the length of that group when running against each host | 15:10 |
noonedeadpunk | But for testing complicated one I would need to spawn some cross-distro mnaio | 15:11 |
noonedeadpunk | The idea is - if you have 3 hosts running focal, and 1 host running bionic (as example), then wheel will be build only for focal and won't for bionic | 15:12 |
noonedeadpunk | though I can imagine it adds quite a lot of runtime overhead | 15:13 |
noonedeadpunk | Ok, I guess I found how to reduce overhead | 15:27 |
noonedeadpunk | Or not... | 15:27 |
anskiy | noonedeadpunk: are you talking about https://bugs.launchpad.net/openstack-ansible/+bug/1989506? | 15:29 |
noonedeadpunk | yep | 15:34 |
opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-python_venv_build master: Change default value for venv_wheel_build_enable https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/857897 | 15:44 |
noonedeadpunk | feel free to test this out ^ | 15:45 |
*** dviroel is now known as dviroel|lunch | 15:59 | |
*** ysandeep is now known as ysandeep|out | 16:03 | |
spatel | Does anyone using encryption with cinder? or any kind of storage safety solution? | 16:24 |
spatel | I would like to understand how does it work :) | 16:25 |
*** dviroel|lunch is now known as dviroel | 17:09 | |
jamesdenton | noonedeadpunk i am around now, thanks for working on that. i will likely test it tomorrow | 21:10 |
*** dviroel is now known as dviroel|out | 21:45 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!