Thursday, 2022-09-15

*** ysandeep|out is now known as ysandeep02:49
*** ysandeep is now known as ysandeep|afk05:13
*** ysandeep|afk is now known as ysandeep06:25
opendevreviewEbbex proposed openstack/openstack-ansible-galera_server master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/85781207:07
ebbexGood morning! Continuing the spam from yesterday :P07:08
opendevreviewEbbex proposed openstack/openstack-ansible-ceph_client master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/85781307:10
noonedeadpunkmorning07:47
noonedeadpunkI will review all at once if no objections 07:48
opendevreviewEbbex proposed openstack/openstack-ansible-haproxy_server master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/85782107:54
opendevreviewDmitriy 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/+/85771908:03
*** ysandeep is now known as ysandeep|afk08:07
opendevreviewEbbex proposed openstack/openstack-ansible-lxc_hosts master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/85782208:15
opendevreviewEbbex proposed openstack/openstack-ansible-memcached_server master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-memcached_server/+/85782308:18
opendevreviewEbbex proposed openstack/openstack-ansible-openstack_hosts master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/85782508:21
anskiynoonedeadpunk: 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
opendevreviewEbbex proposed openstack/openstack-ansible-os_adjutant master: Refactor use of include_vars  https://review.opendev.org/c/openstack/openstack-ansible-os_adjutant/+/85782708:32
noonedeadpunkanskiy: doesn't really matter. I just fixed "typo" issue in the patch08:40
opendevreviewEbbex proposed openstack/openstack-ansible-os_aodh master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_aodh/+/85782808:40
anskiyokay, I hope it would work :)08:41
opendevreviewEbbex proposed openstack/openstack-ansible-os_blazar master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_blazar/+/85782908:47
opendevreviewEbbex proposed openstack/openstack-ansible-os_ceilometer master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_ceilometer/+/85783008:51
opendevreviewEbbex proposed openstack/openstack-ansible-os_gnocchi master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_gnocchi/+/85783108:53
*** ysandeep|afk is now known as ysandeep08:53
opendevreviewEbbex proposed openstack/openstack-ansible-os_tacker master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_tacker/+/85783308:57
opendevreviewEbbex proposed openstack/openstack-ansible-rsyslog_client master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-rsyslog_client/+/85783709:13
opendevreviewEbbex proposed openstack/openstack-ansible-rsyslog_server master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-rsyslog_server/+/85784609:16
*** ysandeep is now known as ysandeep|brb09:51
*** ysandeep|brb is now known as ysandeep10:03
opendevreviewEbbex proposed openstack/ansible-role-systemd_mount master: Remove redundant vars line  https://review.opendev.org/c/openstack/ansible-role-systemd_mount/+/85786110:14
opendevreviewEbbex proposed openstack/ansible-role-systemd_networkd master: Remove redundant vars line  https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/85786310:27
*** dviroel|afk is now known as dviroel11:31
*** ysandeep is now known as ysandeep|afk12:26
opendevreviewEbbex proposed openstack/ansible-role-python_venv_build master: Remove redundant vars line  https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/85787512:41
opendevreviewEbbex proposed openstack/ansible-role-uwsgi master: Remove redundant vars line  https://review.opendev.org/c/openstack/ansible-role-uwsgi/+/85787613:02
*** ysandeep|afk is now known as ysandeep13:24
noonedeadpunkjamesdenton: around? I have simple/straightforward and smart complicated solution to the bug with venv_wheel_build_enable15:08
noonedeadpunksimple one is just to check for the length of ansible_play_hosts and enable wheels build or not based on that15:09
noonedeadpunkcomplicated 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 host15:10
noonedeadpunkBut for testing complicated one I would need to spawn some cross-distro mnaio15:11
noonedeadpunkThe 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 bionic15:12
noonedeadpunkthough I can imagine it adds quite a lot of runtime overhead15:13
noonedeadpunkOk, I guess I found how to reduce overhead15:27
noonedeadpunkOr not...15:27
anskiynoonedeadpunk: are you talking about https://bugs.launchpad.net/openstack-ansible/+bug/1989506?15:29
noonedeadpunkyep15:34
opendevreviewDmitriy 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/+/85789715:44
noonedeadpunkfeel free to test this out ^15:45
*** dviroel is now known as dviroel|lunch15:59
*** ysandeep is now known as ysandeep|out16:03
spatelDoes anyone using encryption with cinder? or any kind of storage safety solution?16:24
spatelI would like to understand how does it work :)16:25
*** dviroel|lunch is now known as dviroel17:09
jamesdentonnoonedeadpunk i am around now, thanks for working on that. i will likely test it tomorrow21:10
*** dviroel is now known as dviroel|out21:45

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!