ansible / community

This repository is being archived. See https://github.com/ansible-community/presentations and https://github.com/ansible-community/meetings for the new locations
Apache License 2.0
489 stars 144 forks source link

Windows Working Group agenda - 2021 #581

Closed jborean93 closed 2 years ago

jborean93 commented 3 years ago

Github windows issues Github windows PRs Windows agenda Windows pinboard

The Windows Working Group is focused on improving Ansible's management of Windows hosts. We meet every week on Tuesdays at 20:00 UTC (see ical)

More information related to the Windows Working Group is available at: https://github.com/ansible/community/wiki/Windows

Feel free to add your questions to this ticket to be discussed during the upcoming meeting.

After being discussed, raised items are being updated, and after a month resolved items are being hidden from view (but remain accessible).

(migrated from 2020 agenda #517)

jborean93 commented 3 years ago

How should win_package handle check mode when the path specified does not exist. Currently it fails like it would if it ran normally but it may have a previous step that copied/downloaded the file. Should we just ignore this error and report a change if possible or is a failure a desired outcome https://github.com/ansible-collections/ansible.windows/issues/208.

Edit: It was decided the current behaviour is what it should be. The module should act like it would if run in normal mode and it's up to the caller to ensure the playbooks work in check mode by ignoring errors if necessary. The alternative is going to be confusing behaviour as to why things fail/don't fail in certain situations rather than the current cut and dry setup.

nitzmahone commented 2 years ago

NO MEETING SEP 28 DUE TO ANSIBLEFEST

MKletz commented 2 years ago

I'd like to meet and discuss https://github.com/ansible-collections/ansible.windows/issues/15 if possible

briantist commented 2 years ago

The text in https://github.com/ansible/community/wiki/Windows needs updating, and that link is where the IRC channel topic points.

UPDATE: @nitzmahone will take an update pass over it

jborean93 commented 2 years ago

Talk about https://github.com/ansible/ansible/pull/76256, in particular

UPDATE: @jborean93 will enable rules above with line length at 160, add git config for ignore-rev-file on the final reformatting commit

MKletz commented 2 years ago

hey all sorry for the delays but I'd be interested in discussing https://github.com/ansible-collections/ansible.windows/issues/15 on Nov 16 if there isn't anything else.

briantist commented 2 years ago

@MKletz sounds good! take note the meeting time is in UTC, so if you're in a location that observes DST, the meeting time may be different than last time you checked/attended. For example for me in US Eastern, it used to be 4pm, now it's 3pm.

MKletz commented 2 years ago

If we're still meeting on 11/23 I'd like to discuss the missing package the CI is looking for on https://github.com/ansible-collections/ansible.windows/pull/313 it's asking for a missing OMI package after adding the example composite resource to the test files

ERROR: tests/integration/targets/win_dsc/files/xTestCompositeDsc/1.0.0/DSCResources/xTestComposite/xTestComposite.schema.psm1:1:15: ParserError: System.IO.DirectoryNotFoundException: Unable to find DSC schema store at /etc/opt/omi/conf/dsc/configuration. Please ensure PS DSC for Linux is installed.

https://docs.microsoft.com/en-us/powershell/scripting/dsc/getting-started/lnxgettingstarted?view=powershell-7.2

Edit: we skipped the sanity check for this particular file to avoid adding more dependencies to the test container