| *** spatel has joined #openstack-ansible-sig | 00:09 | |
| *** spatel has quit IRC | 00:33 | |
| openstackgerrit | Duc Truong proposed openstack/ansible-collections-openstack master: Add properties for os_project https://review.opendev.org/715266 | 00:54 |
|---|---|---|
| *** spatel has joined #openstack-ansible-sig | 01:22 | |
| *** spatel has quit IRC | 03:26 | |
| *** evrardjp has quit IRC | 04:36 | |
| *** evrardjp has joined #openstack-ansible-sig | 04:36 | |
| *** gtema has joined #openstack-ansible-sig | 06:53 | |
| *** priteau has joined #openstack-ansible-sig | 07:00 | |
| *** dtantsur|afk is now known as dtantsur | 08:04 | |
| *** gtema has quit IRC | 08:17 | |
| *** gtema has joined #openstack-ansible-sig | 08:37 | |
| tremble | dtantsur, What do you mean by "one import per line"? | 08:44 |
| dtantsur | tremble: instead of `from x import (y,z)` use `from x import y\nfrom x import z` | 08:44 |
| dtantsur | it will be enforces by a pep8 rule later on, I think mordred is working on it | 08:45 |
| dtantsur | tremble: https://review.opendev.org/#/c/698068/ | 08:45 |
| tremble | I was not aware of that style guide entry | 08:48 |
| tremble | WRT the extra try/catch - I believe it's *possible* for that to be thrown, and I'd rather catch it and explain what I was *trying* to do at the time than let Ansible's default handler's kick in. | 08:51 |
| tremble | I'm possibly too used to AWS throwing wierd errors at me. | 08:52 |
| tremble | I was assuming some form of permissions errror might trigger it. | 08:52 |
| tremble | I do like the list(map(...)) trick though | 08:53 |
| *** gshippey has joined #openstack-ansible-sig | 09:02 | |
| dtantsur | tremble: a permission error won't trigger ResourceNotFound | 09:03 |
| dtantsur | I thought you were handling the situation when keystone is too old | 09:03 |
| dtantsur | but they the error message is confusing.. | 09:03 |
| *** priteau has quit IRC | 09:03 | |
| * tremble cleans it up | 09:09 | |
| *** owalsh has quit IRC | 09:50 | |
| *** owalsh has joined #openstack-ansible-sig | 09:50 | |
| openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: New module: os_keystone_mapping_info https://review.opendev.org/713686 | 09:56 |
| openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_identity_provider https://review.opendev.org/714632 | 09:56 |
| openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_identity_provider_info https://review.opendev.org/714633 | 09:56 |
| openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_federation_protocol https://review.opendev.org/714991 | 09:56 |
| openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_federation_protocol_info https://review.opendev.org/714992 | 09:56 |
| openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: os_keystone_mapping: fixup some minor nits found in followup reviews https://review.opendev.org/716537 | 09:56 |
| tremble | dtantsur, nits fixed up ^ | 09:56 |
| dtantsur | thanks! will attempt another pass today hopefully | 09:56 |
| *** gtema has quit IRC | 11:38 | |
| *** gtema has joined #openstack-ansible-sig | 12:14 | |
| *** dmellado has quit IRC | 12:15 | |
| *** dmellado has joined #openstack-ansible-sig | 12:23 | |
| sshnaidm | mordred, I hope we don't loose our repo on github? http://lists.openstack.org/pipermail/openstack-discuss/2020-April/013739.html | 13:42 |
| sshnaidm | mordred, because it's not "official" | 13:42 |
| mordred | sshnaidm: we don't need our repo on github, it's just a mirror and we have one in opendev | 13:47 |
| mordred | luckily nothing about galaxy requires a github repo to exist anymore (thank god) | 13:47 |
| sshnaidm | mordred, better to have it for "marketing" purposes | 13:47 |
| sshnaidm | people is still searching in github for ansible modules | 13:48 |
| mordred | it's possible they'll still consider it "official" since it's the work of a sig ... but if not, I'm not too worried about it - I imagine more people will get to us via galaxy than via github | 13:48 |
| mordred | once collections take hold | 13:48 |
| sshnaidm | mordred, is it better to use ">0.45" when using with openstacksdk master? https://review.opendev.org/#/c/715266/ | 13:56 |
| sshnaidm | next version should be 0.46, right? | 13:56 |
| mordred | sshnaidm: yes, I agree, next version should be 0.46 | 14:08 |
| mordred | sshnaidm: and I'm torn - >0.45 is probably fine in this case, but if we cut a bugfix of 0.45 as 0.45.2 without the new feature, it would be wrong | 14:09 |
| mordred | (I don't think there's a super awesome answer) | 14:10 |
| sshnaidm | mordred, yeah, that's what I suspect | 14:10 |
| dtantsur | >= 0.46 is a safer assumption and will work even if decide that we're brave enough to go to 1.0 | 14:14 |
| sshnaidm | dropping links here from Ansible community meeting, might be useful: | 15:13 |
| sshnaidm | Ansible Collections Project Status: https://github.com/ansible-collections/overview/blob/master/status.rst | 15:13 |
| sshnaidm | ACD in general: https://gist.github.com/abadger/1f14caded92117fbd3036842c875701c | 15:13 |
| sshnaidm | ACD preview: https://github.com/abadger/misc-work/blob/master/ansible/build_acd/ansible_infra/cli.py | 15:13 |
| sshnaidm | ACD routing: https://github.com/ansible/ansible/pull/67684 | 15:13 |
| sshnaidm | ACD package layout: https://toshio.fedorapeople.org/ansible/acd/ansible-2.10-file-layout.png | 15:13 |
| sshnaidm | collection presentation: https://danielmellado.github.io/ansible/#1 | 15:13 |
| sshnaidm | statistics of collections contributions: https://stats.eng.ansible.com/apps/collections/contributors/ | 15:13 |
| mordred | sshnaidm: ++ | 15:18 |
| dtantsur | thank you sshnaidm | 15:29 |
| *** gtema has quit IRC | 15:53 | |
| openstackgerrit | Duc Truong proposed openstack/ansible-collections-openstack master: Add properties for os_project https://review.opendev.org/715266 | 15:55 |
| *** holser has quit IRC | 16:32 | |
| *** evrardjp has quit IRC | 16:36 | |
| *** evrardjp has joined #openstack-ansible-sig | 16:36 | |
| *** dtantsur is now known as dtantsur|afk | 17:07 | |
| *** weshay is now known as weshay|ruck | 20:00 | |
| *** gshippey has quit IRC | 20:21 | |
| *** spatel has joined #openstack-ansible-sig | 22:19 | |
| *** spatel has quit IRC | 22:44 | |
| openstackgerrit | Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add properties for os_project https://review.opendev.org/715266 | 23:26 |
| *** sshnaidm is now known as sshnaidm|afk | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
