| *** ysandeep|out is now known as ysandeep|ruck | 04:32 | |
| *** ysandeep|ruck is now known as ysandeep|ruck|afk | 04:47 | |
| *** ysandeep|ruck|afk is now known as ysandeep|ruck | 05:35 | |
| *** ysandeep|ruck is now known as ysandeep|ruck|afk | 07:05 | |
| *** ysandeep|ruck|afk is now known as ysandeep|ruck | 08:37 | |
| opendevreview | Merged openstack/openstack-ansible master: Do not install rally on first upgrade run https://review.opendev.org/c/openstack/openstack-ansible/+/848760 | 10:06 |
|---|---|---|
| *** ysandeep|ruck is now known as ysandeep|ruck|brb | 10:06 | |
| *** ysandeep|ruck|brb is now known as ysandeep|ruck | 10:27 | |
| *** ysandeep|ruck is now known as ysandeep|ruck|afk | 11:01 | |
| *** dviroel|out is now known as dviroel | 11:27 | |
| *** ysandeep|ruck|afk is now known as ysandeep|ruck | 11:50 | |
| *** ysandeep|ruck is now known as ysandeep|out | 16:28 | |
| *** dviroel is now known as dviroel|lunch | 17:13 | |
| noonedeadpunk | y | 17:49 |
| *** dviroel|lunch is now known as dviroel | 18:11 | |
| jrosser | why do we not run_once this? https://opendev.org/openstack/ansible-role-python_venv_build/src/branch/master/tasks/main.yml#L32 | 19:02 |
| jrosser | noonedeadpunk: ^ does that run ansible_play_hosts * group['repo_all'] times? | 19:04 |
| noonedeadpunk | as we need arch and distro of all repo containers? | 19:12 |
| jrosser | but the task loops over them all? | 19:12 |
| noonedeadpunk | oh, ok | 19:13 |
| jrosser | i just tried asking about our module failure stuff in #ansible and basically got chewed up that our task is wrong | 19:13 |
| noonedeadpunk | now I see what you mean | 19:13 |
| jrosser | but thats kind of not the point because in CI there is only one host in the target group anyway | 19:13 |
| jrosser | which is probably why we never noticed that | 19:14 |
| noonedeadpunk | I wonder if we dropped run_once from there one day | 19:14 |
| jrosser | i was wondering that too | 19:14 |
| jrosser | also they just say we should do that as a separate play | 19:15 |
| noonedeadpunk | yeah, it was inside python_venv_wheel_build previously, which had run_once | 19:15 |
| jrosser | but i don't think that is really relevant | 19:15 |
| noonedeadpunk | and I think we moved out of there for $reason | 19:16 |
| noonedeadpunk | but yes, we likely just forgot to add run_once | 19:17 |
| noonedeadpunk | as we moved out as we needed facts for that https://opendev.org/openstack/ansible-role-python_venv_build/src/commit/b2a006d630784b1fd037a3dd59f727d6559020dd/tasks/main.yml#L28 | 19:17 |
| noonedeadpunk | but likely no reason not to run it once | 19:18 |
| noonedeadpunk | so you're right | 19:18 |
| noonedeadpunk | but yeah, it's not relevant :) | 19:18 |
| opendevreview | Jonathan Rosser proposed openstack/ansible-role-python_venv_build master: Gather build hosts facts once. https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/849035 | 19:27 |
| spatel | my ovn-bgp lab half way completed :) evpn part is ready now deploying openstack with frr | 19:43 |
| *** dviroel is now known as dviroel|out | 20:42 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!