Monday, 2020-05-11

*** evrardjp has quit IRC04:36
*** evrardjp has joined #openstack-ansible-sig04:36
*** rpittau|afk is now known as rpittau07:36
*** dtantsur|afk is now known as dtantsur08:04
*** EmilienM has quit IRC08:15
*** EmilienM has joined #openstack-ansible-sig08:17
*** gtema has joined #openstack-ansible-sig08:21
*** holser has joined #openstack-ansible-sig08:25
*** gtema has quit IRC08:28
*** gtema has joined #openstack-ansible-sig09:09
*** gtema has quit IRC09:14
*** sshnaidm|off is now known as sshnaidm09:15
*** gshippey has joined #openstack-ansible-sig09:19
*** holser has quit IRC09:38
*** gtema has joined #openstack-ansible-sig09:43
*** priteau has joined #openstack-ansible-sig09:44
*** gtema has quit IRC09:48
*** holser has joined #openstack-ansible-sig10:01
*** holser has quit IRC10:01
*** holser has joined #openstack-ansible-sig10:02
*** rpittau is now known as rpittau|bbl10:15
*** gtema has joined #openstack-ansible-sig11:25
*** gtema has quit IRC11:30
*** iurygregory has quit IRC11:37
*** rpittau|bbl is now known as rpittau11:58
*** iurygregory has joined #openstack-ansible-sig11:58
*** priteau has quit IRC12:16
*** gtema has joined #openstack-ansible-sig12:42
*** gtema has quit IRC12:47
openstackgerritMerged openstack/ansible-collections-openstack master: Ensure modules work with queens devstack  https://review.opendev.org/72642113:01
*** iurygregory has quit IRC13:01
*** iurygregory has joined #openstack-ansible-sig13:02
*** gtema has joined #openstack-ansible-sig13:17
*** gtema has quit IRC13:21
*** owalsh has quit IRC13:30
*** owalsh has joined #openstack-ansible-sig13:31
*** dtantsur is now known as dtantsur|brb14:17
mordredsshnaidm: how do I tell ansible-test to ignore a file?14:35
mordredsshnaidm: (it's finding a helper utility script in tools and I really don't need it to try to compile it under python 2.714:35
sshnaidmmordred, something like that: https://github.com/sshnaidm/ansible-podman-collections/blob/master/tests/sanity/ignore-2.10.txt#L11-L1514:40
sshnaidmmordred, https://github.com/openstack/ansible-collections-openstack/blob/master/tests/sanity/14:41
mordredsshnaidm: cool thanks14:42
sshnaidmmordred, possible to skip one error or entire test14:42
mordredsshnaidm: it would be great if you could configure it globally to ignore files ... but this will do just fine :)14:42
sshnaidmmordred, yeah, there is an issue about wildcards, still waiting I suppose :)14:43
openstackgerritMonty Taylor proposed openstack/ansible-collections-openstack master: Remove all of the os_ prefixes  https://review.opendev.org/71305614:45
openstackgerritMonty Taylor proposed openstack/ansible-collections-openstack master: Rename all of the modules  https://review.opendev.org/72653814:45
mordredsshnaidm: ok. I think that should pass now ^^14:45
mordredI had to manually edit one file - but hopefully it's all ok14:45
sshnaidmgreat, will look14:45
*** sshnaidm is now known as sshnaidm|afk14:57
*** dtantsur|brb is now known as dtantsur14:59
-openstackstatus- NOTICE: Our CI mirrors in OVH BHS1 and GRA1 regions were offline between 12:55 and 14:35 UTC, any failures there due to unreachable mirrors can safely be rechecked15:07
openstackgerritMonty Taylor proposed openstack/ansible-collections-openstack master: Rename all of the modules  https://review.opendev.org/72653815:44
*** rpittau is now known as rpittau|afk16:09
*** dtantsur is now known as dtantsur|afk16:19
*** evrardjp has quit IRC16:36
*** evrardjp has joined #openstack-ansible-sig16:36
*** sshnaidm|afk is now known as sshnaidm17:12
mordredsshnaidm: why does the linter not like that documentation update?17:24
sshnaidmmordred, yeah, looking at this right now17:24
mordredit didn't liek openstack.cloud.foo either :)17:24
mordredOH17:25
mordredit's the symlinks17:25
mordredsshnaidm: the module name doesn't match the filename when we symlink them17:25
mordredsshnaidm: maybe we should just rely on the routing file and not make the symlinks17:25
sshnaidmhmm.. yeah, we can try17:26
sshnaidmeither to set ignores of documentation check for every module..17:26
openstackgerritMonty Taylor proposed openstack/ansible-collections-openstack master: Rename all of the modules  https://review.opendev.org/72653817:27
mordredsshnaidm: yeah. I think routing.yml is supposed to be the future anyway17:27
sshnaidmis it merged? and is it working with 2.917:28
mordreddunno17:29
mordredsshnaidm: I mean - ultimatley though - the os_server -> openstack.cloud.server backwards compat is mostly via ansible/ansible anyway17:31
mordredfor someone using collection with 2.9 they really are going to have to update consumption anyway17:31
sshnaidmstill not merged as I see: https://github.com/ansible/ansible/pull/6768417:32
sshnaidmmordred, maybe worth to try symlink anyway, we can always remove them17:33
sshnaidmhttp://paste.openstack.org/show/793395/ :)17:36
openstackgerritSagi Shnaidman proposed openstack/ansible-collections-openstack master: Rename all of the modules  https://review.opendev.org/72653818:17
sshnaidmmordred, 4 files have different name and module name ^18:18
openstackgerritSagi Shnaidman proposed openstack/ansible-collections-openstack master: Remove all of the os_ prefixes  https://review.opendev.org/71305618:18
openstackgerritSagi Shnaidman proposed openstack/ansible-collections-openstack master: Rename all of the modules  https://review.opendev.org/72653818:23
*** iurygregory has quit IRC18:25
*** iurygregory has joined #openstack-ansible-sig18:38
*** gshippey has quit IRC19:28
mordredsshnaidm: cool. I'm asking fokls about the symlinks in #ansible-devel fwiw20:32
*** holser has quit IRC20:54
openstackgerritMonty Taylor proposed openstack/ansible-collections-openstack master: Rename all of the modules  https://review.opendev.org/72653820:55
openstackgerritMonty Taylor proposed openstack/ansible-collections-openstack master: Remove all of the os_ prefixes  https://review.opendev.org/71305621:11
openstackgerritMonty Taylor proposed openstack/ansible-collections-openstack master: Rename all of the modules  https://review.opendev.org/72653821:11
openstackgerritMonty Taylor proposed openstack/ansible-collections-openstack master: Run linters in a temp dir instead of an install  https://review.opendev.org/72695221:11
mordredsshnaidm: ^^ I am not thrilled with that, but it'll do for now. what I think I'd prefer is to write another wrapper script that tox calls that will use mktemp - and will put in a trap so that it'll be sure to clean up the dir when it's done21:11
sshnaidmmordred, ack, maybe worth to leave build-install in the end, just for build test21:16
sshnaidmactually install only, we build before linters21:17
openstackgerritMonty Taylor proposed openstack/ansible-collections-openstack master: Run linters in a temp dir instead of an install  https://review.opendev.org/72695221:18
openstackgerritMonty Taylor proposed openstack/ansible-collections-openstack master: Remove all of the os_ prefixes  https://review.opendev.org/71305621:18
openstackgerritMonty Taylor proposed openstack/ansible-collections-openstack master: Rename all of the modules  https://review.opendev.org/72653821:18
sshnaidmwell, we can skip install, build is enough21:18
mordredsshnaidm: yeah - don't need install21:18
mordredsshnaidm: I extracted into a script for easier reading - and so we can use a trap21:18
sshnaidmmordred, why do we need to remove temporary dir..?21:21
mordredsshnaidm: so that people who are running on their laptop don't wind up with a billion copies in temp dir21:26
sshnaidmmordred, we may do it constant path21:27
mordredyeah - but then cruft can linger locally21:27
mordredI mean - I donm't have a strong opinion on it21:27
mordredthat was just my motivation21:27
mordredI normally let zuul run it :)21:27
sshnaidmnp, just curios :)21:27
openstackgerritSagi Shnaidman proposed openstack/ansible-collections-openstack master: Redesign OpenstackModule class  https://review.opendev.org/72228321:54
openstackgerritSagi Shnaidman proposed openstack/ansible-collections-openstack master: Add OpenstackModule to os_server_action  https://review.opendev.org/72174622:05
openstackgerritSagi Shnaidman proposed openstack/ansible-collections-openstack master: Redesign OpenstackModule class  https://review.opendev.org/72228322:11
*** sshnaidm is now known as sshnaidm|afk22:11
openstackgerritSagi Shnaidman proposed openstack/ansible-collections-openstack master: Add OpenstackModule to os_server_action  https://review.opendev.org/72174622:27
openstackgerritSagi Shnaidman proposed openstack/ansible-collections-openstack master: Run linters in a temp dir instead of an install  https://review.opendev.org/72695222:34
openstackgerritSagi Shnaidman proposed openstack/ansible-collections-openstack master: Run linters in a temp dir instead of an install  https://review.opendev.org/72695222:51
openstackgerritSagi Shnaidman proposed openstack/ansible-collections-openstack master: Remove all of the os_ prefixes  https://review.opendev.org/71305623:13
openstackgerritSagi Shnaidman proposed openstack/ansible-collections-openstack master: Rename all of the modules  https://review.opendev.org/72653823:13
*** sshnaidm|afk has quit IRC23:14
*** sshnaidm has joined #openstack-ansible-sig23:15
*** sshnaidm is now known as sshnaidm|afk23:16

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