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

Public Ansible Project Meeting Agenda - June 2018 #317

Closed
bcoca opened this issue May 31, 2018 · 29 comments
Closed

Public Ansible Project Meeting Agenda - June 2018 #317

bcoca opened this issue May 31, 2018 · 29 comments

Comments

@bcoca
Copy link
Member

bcoca commented May 31, 2018

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.

See https://github.com/ansible/community/blob/master/meetings/README.md for the schedule

Once an item has been addressed it should get strike-though strike-though

When creating new agenda ensure core and meeting_agenda labels are set

@bcoca
Copy link
Member Author

bcoca commented Jun 1, 2018

ansible/ansible#40913

@bcoca
Copy link
Member Author

bcoca commented Jun 4, 2018

ansible/ansible#41078

@tbouvet
Copy link

tbouvet commented Jun 7, 2018

ansible/ansible#38280

@jose-delarosa
Copy link

jose-delarosa commented Jun 11, 2018

ansible/ansible#41297
IRC nick is jdelaros1

@maxamillion
Copy link
Contributor

maxamillion commented Jun 11, 2018

I'd like to review the inclusion of facts gathering arg to the k8s module. Personally I'd like to see a separate module (which has also been submitted ansible/ansible#41128) for this since it's already been stated that the lookup module doesn't provide the desired functionality, but I wanted to get the opinion of the group. The PR owner said they would try and attend the meeting, but it's at 5am local time and they might not make it. (understandably so)
ansible/ansible#41088

@maxamillion
Copy link
Contributor

maxamillion commented Jun 11, 2018

Would like consensus on how best to handle the scenario in which the user wants to provide more information to the user outside msgs. Should this be a warning, new return key int he JSON, $other? https://github.com/ansible/ansible/pull/40723/files

@mattclay
Copy link
Member

mattclay commented Jun 11, 2018

Should ansible/ansible#41171 (Introduce inventory.any_unparsed_is_failed configuration setting) be backported to stable-2.6? Although it does add a new configuration entry, it was done to address two bug reports.

@agaffney
Copy link

agaffney commented Jun 12, 2018

ansible/ansible#41058

Merging of default-derived callback plugin functionality back into default to allow mixing/matching of features from those callback plugins.

@farhan7500
Copy link

farhan7500 commented Jun 13, 2018

We want to discuss including HPE's hpe3par_cpg. The review comments have been addressed and the CI has also executed successfully. Can we merge the module now?

ansible/ansible#39656

@backbord
Copy link

backbord commented Jun 13, 2018

First time contributor here. I'd like to discuss the inclusion of a new strategy "streamlined". The CI executed successfully (a month ago and after today's rebasing) but the PR hasn't received reviews yet. I would like to see this or a similar strategy added to ansible. If possible, how can I help to move this forward?

ansible/ansible#39602

Edit: Renamed from "atomic" to "streamlined"

@Im0
Copy link

Im0 commented Jun 13, 2018

Adding success_msg to assert - ansible/ansible#41048

@sean797
Copy link

sean797 commented Jun 14, 2018

ability to flush handlers specific handlers ansible/ansible#25573

@eikef
Copy link

eikef commented Jun 14, 2018

I'd like to request a core team review/commit of ansible/ansible#35844 as well as ansible/ansible#35558 (keycloak_realm and keycloak_client new modules; one user had already tested them. They are in the line of other keycloak modules already added.)

There are also some minor bugfix PRs for existing keycloak modules in ansible/ansible#37776 and ansible/ansible#39515 which could use a look-over and commit :-)

@Akasurde
Copy link
Member

Akasurde commented Jun 14, 2018

* Ansible Installation procedure documentation for Package - ansible/ansible#41437

@jose-delarosa
Copy link

jose-delarosa commented Jun 18, 2018

ansible/ansible#41656
This is a follow-up to ansible/ansible#41297, where the original module was broken into 3. This submission is for the first module, according to the grouped modules documentation.
IRC nick is jdelaros1

@s-hertel
Copy link
Contributor

s-hertel commented Jun 18, 2018

Discuss expectations for conditionals and undefined variables. Should these tasks have the same outcome? If and where should they fail? ``` --- - hosts: localhost connection: local gather_facts: no vars: assigned_undef: "{{ never_defined }}" tasks: - debug: msg="{{ never_defined }}" # succeeds on devel when: never_defined is defined - debug: msg="{{ assigned_undef }}" when: assigned_undef is defined - debug: msg="{{ assigned_undef }}" when: assigned_undef is not undefined - debug: msg="{{ assigned_undef }}" # is this different than the task above? when: not assigned_undef is undefined ```
---
- hosts: localhost
  connection: local
  gather_facts: no
  vars:
    dict_one: {foo: bar}
    dict_two: "{{ dict_one['baz'] }}"
  tasks:
    - debug: msg="{{ dict_one['baz'] }}"  # succeeds on devel
      when: dict_one['baz'] is defined
    - debug: msg="{{ dict_two }}"
      when: dict_two is defined
    - debug: msg="{{ dict_two }}"
      when: dict_two is not undefined
    - debug: msg="{{ dict_two }}"  # is this different than the task above?
      when: not dict_two is undefined
```</strike>

@caphrim007
Copy link

caphrim007 commented Jun 20, 2018

* would like to discuss #247 (comment) as @Qalthos recommended the request be asked at core meeting. Logs from networking meeting where this was brought up are https://meetbot.fedoraproject.org/ansible-network/2018-06-20/network_working_group.2018-06-20-15.59.log.html

@maxamillion
Copy link
Contributor

maxamillion commented Jun 22, 2018

Currently command and shell only support one-liners, there's a request to support multi-line commands via this issue ticket. I'm curious if the collective community has any interest in supporting this use case or if this sort of thing should be offloaded to creating scripts that are copy'd and then executed?

ansible/ansible#12034

@pilou-
Copy link
Contributor

pilou- commented Jun 26, 2018

A regression was very recently backported to 2.6, it breaks some inventory plugins. This should be fixed before the 2.6 release, see ansible/ansible#41888 ansible/ansible#41913 for details DONE: 2nd PR has been merged and 2.6 backport pull request has been created.

@farhan7500
Copy link

farhan7500 commented Jun 26, 2018

Want to discuss hpe3par_cpg module. Implemented the review requests from last week's meeting. Need to discuss the integration tests.

ansible/ansible#39656

@abadger
Copy link
Contributor

abadger commented Jun 26, 2018

Proposal for how to handle file encodings ansible/proposals#121

@bcoca bcoca closed this as completed Jun 26, 2018
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