Skip to content
This repository has been archived by the owner on Jan 5, 2024. It is now read-only.

Agenda for Public Core Meeting - 2016-06-09 15:00 UTC #109

Closed
nitzmahone opened this issue Jun 7, 2016 · 4 comments
Closed

Agenda for Public Core Meeting - 2016-06-09 15:00 UTC #109

nitzmahone opened this issue Jun 7, 2016 · 4 comments

Comments

@nitzmahone
Copy link
Member

Please leave a comment regarding any agenda item you wish to discuss. If you don't show up for the meeting, your item will be skipped.

If your IRC nick is different from your Github username, leave that as well.

@nitzmahone
Copy link
Member Author

(bumped from Tuesday's meeting)

Discuss ansible/ansible-modules-core#3256 What should be done about the name parameter in the cron module. Requested by @Leibniz137 It seems like name probably should be required and Ansible only manage cron jobs that have the name/comment string. However currently the code doesn't require the name and it's unclear why that changed.

@underyx
Copy link

underyx commented Jun 7, 2016

I'd like to discuss whether it makes sense to merge ansible/ansible-modules-core#901 (was briefly merged and then reverted last weekend).

@robinro
Copy link

robinro commented Jun 8, 2016

I have two topics to discuss:

  1. To review/merge: In 2.2 currently git with depth but without version specified does not update repos. This is the documentation-example usage case for ansible-pull
    Fix: Fix git shallow update ansible-modules-core#3794
    Test: Add git shallow fetch test ansible#16055

    The git module is a bit of a mess and there are still some holes in the support of depth, see also git - in Ansible 2.1 switching a shallow clone from tag to branch fails ansible-modules-core#3817 (not yet fixed). Maybe one can do a git refactor sprint? I'd be happy to take part if time allows, but am pretty busy the next weeks/months.

  2. To discuss: Add check_mode: yes/no to tasks. Replaces always_run (then check_mode: no, see also https://github.com/ansible/proposals/blob/master/rename_always_run.md by @resmo) and introduces check_mode: yes to run a single task in check mode even if this is not requested on the command line.
    PR: add check_mode option for tasks ansible#16056
    Issue: Allow running task or play in check_mode from playbook ansible#12498
    Historical Discussion: Implement a per-task check_mode setting ansible#7439

@quinot
Copy link

quinot commented Jun 9, 2016

I'd like to touch base on ansible/ansible#11810 (multiple interpreter support): I would like to make sure that there is no pending issue left that prevents merging it.

@jimi-c jimi-c closed this as completed Jul 7, 2016
@dagwieers dagwieers added core and removed core labels Jun 24, 2017
@ansible ansible locked and limited conversation to collaborators Jul 19, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

7 participants