#ansible-meeting: Ansible Core Meeting

Meeting summary

1. 1. Agenda https://github.com/ansible/community/issues/150 (gundalow, 15:01:11)
   
   
2. Discuss extending the module "shipit" workflow to non-modules (gundalow, 15:01:41)
   1. https://github.com/ansible/community/issues/150#issuecomment-277025899 (gundalow, 15:02:01)
   2. https://docs.google.com/spreadsheets/d/1z-BfOkAaY60nYir6qx6L9yHMqcwT31UtrgaAFKnjH9w (bcoca, 15:02:52)
   3. spreadsheet exists (bcoca, 15:03:15)
   4. votes being cast (bcoca, 15:03:19)
   5. ACTION: bcoca to organise vote for inventory plugins (gundalow, 15:06:30)
   6. ACTION: bcoca to speak with abadger2000 to find out howe we add python metadata to config files (gundalow, 15:08:55)
   
   
3. ansible/ansible#19283 hosts module (for /etc/hosts) (gundalow, 15:09:11)
   1. https://github.com/ansible/community/issues/150#issuecomment-277028142 (gundalow, 15:09:41)
   2. https://github.com/ansible/ansible/pull/19283 (gundalow, 15:09:54)
   3. Imports have been moved to match "house style" (gundalow, 15:10:40)
   4. https://github.com/ansible/ansible/pull/19283#issuecomment-277007481 (allanice001, 15:12:28)
   5. New module policy requires two(+) shipits (gundalow, 15:12:32)
   6. Although we have modules that are aren't as good we are continually learning from what we've done in the past and trying to raise the bar (gundalow, 15:28:58)
   7. [15:28] <@bcoca> jtyr: you can distribute it through galaxy in role for now, i'm working on making it easier to install/share custom plugins w/o having to put in ansible/ansible (gundalow, 15:29:15)
   
   
4. ansible/ansible#19297 Fix for wildcards inside of a path for fileglob lookup (ie: with_fileglob: "/tmp/*/some.conf") (gundalow, 15:30:15)
   1. https://github.com/ansible/community/issues/150#issuecomment-277029768 (gundalow, 15:30:27)
   2. https://github.com/ansible/ansible/pull/19297 (gundalow, 15:30:38)
   3. ACTION: bcoca to review https://github.com/ansible/ansible/pull/19297 (gundalow, 15:31:23)
   
   
5. ansible/ansible#20440 valid directives on include (for execution or inheritance) (gundalow, 15:31:56)
   1. https://github.com/ansible/community/issues/150#issuecomment-277030715 (gundalow, 15:32:07)
   2. https://github.com/ansible/ansible/issues/20440 (gundalow, 15:32:14)
   3. issue is not as much the problem as the implication of what is inhertited and what isnt (gundalow, 15:34:03)
   4. we dont have clear rules or docs (gundalow, 15:34:17)
   5. http://docs.ansible.com/ansible/playbooks_directives.html <= fullest list i have (bcoca, 15:41:51)
   
   
6. ansible/ansible#20058 Add systemd-nspawn connection driver (gundalow, 15:51:29)
   1. (gundalow, 15:51:51)
   2. https://github.com/ansible/ansible/pull/20058 (gundalow, 15:51:58)
   3. ACTION: abadger2000 to review https://github.com/ansible/ansible/pull/20058 (gundalow, 15:53:19)
   
   
7. ansible/ansible#19264. "Added in bullet of Python 2.4+ support discontinuation" (gundalow, 15:54:02)
   1. https://github.com/ansible/ansible/pull/19264 (gundalow, 15:54:24)
   2. Merged (gundalow, 15:57:06)
   
   
8. ansible/ansible#20834 Add swupd plugin (gundalow, 15:57:22)
   1. https://github.com/ansible/ansible/pull/20834 (gundalow, 15:57:38)
   2. https://github.com/ansible/ansible/blob/devel/lib/ansible/module_utils/facts.py#L159 (albertom, 16:02:37)
   3. ACTION: bcoca (if he has time) to review the code. albertom knows we are close to the 2.3 cut off, so we can't guarantee it will make it in for 2.3 (gundalow, 16:03:04)
   4. ACTION: albertom to review feedback from meeting, continue discussion in #ansible-devel (gundalow, 16:18:22)
   
   
9. ansible/ansible#20703 Fixing broken bind mount on CentOS (gundalow, 16:18:57)
   1. believe most issues have been resolved. Revisit next week to ensure it's merged (gundalow, 16:20:10)
   
   
10. Open Floor (gundalow, 16:20:45)
    
11. ansible/ansible#13206 Allow specifying Vault password from the environment (gundalow, 16:21:54)
    1. Main question is do we allow or not (gundalow, 16:27:35)
    2. Discussion that a runtime warning is needed if this feature is used (most seem to agreed, needs vote) (gundalow, 16:28:15)
    3. If implemented maybe make it impossible for the warning to be disabled (gundalow, 16:29:03)
    4. < allanice001> Perhaps also add a step :: os.environ["ANSIBLE_VAULT_PASSWORD"] = "". This would may help avoid teh password being leaked (gundalow, 16:30:29)
    5. [16:32] < abadger2000> If they're only sticking it into the environment for running ansible, that's fine... if it's in a login script or something then we can't clear the password from every other program. (gundalow, 16:32:42)
    6. https://docs.travis-ci.com/user/private-dependencies/ (allanice001, 16:36:15)
    7. AGREED: Rejected, use work around (gundalow, 16:39:06)
    
    
12. Open Floor (gundalow, 16:39:24)
    1. https://github.com/ansible/community/issues/152 (bcoca, 16:39:43)

Action items

1. bcoca to organise vote for inventory plugins
2. bcoca to speak with abadger2000 to find out howe we add python metadata to config files
3. bcoca to review https://github.com/ansible/ansible/pull/19297
4. abadger2000 to review https://github.com/ansible/ansible/pull/20058
5. bcoca (if he has time) to review the code. albertom knows we are close to the 2.3 cut off, so we can't guarantee it will make it in for 2.3
6. albertom to review feedback from meeting, continue discussion in #ansible-devel

Action items, by person

1. abadger2000
   1. bcoca to speak with abadger2000 to find out howe we add python metadata to config files
   2. abadger2000 to review https://github.com/ansible/ansible/pull/20058
2. albertom
   1. bcoca (if he has time) to review the code. albertom knows we are close to the 2.3 cut off, so we can't guarantee it will make it in for 2.3
   2. albertom to review feedback from meeting, continue discussion in #ansible-devel
3. bcoca
   1. bcoca to organise vote for inventory plugins
   2. bcoca to speak with abadger2000 to find out howe we add python metadata to config files
   3. bcoca to review https://github.com/ansible/ansible/pull/19297
   4. bcoca (if he has time) to review the code. albertom knows we are close to the 2.3 cut off, so we can't guarantee it will make it in for 2.3

People present (lines said)

1. gundalow (137)
2. bcoca (119)
3. albertom (43)
4. abadger2000 (32)
5. jtyr (27)
6. allanice001 (22)
7. jimi|ansible (21)
8. mattclay (11)
9. ryansb (10)
10. nitzmahone (7)
11. jtanner (7)
12. akasivel (6)
13. alikins (6)
14. zodbot (4)
15. rcarrillocruz (4)
16. shertel (3)
17. thaumos (3)