Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make windows package download behavior consistent with linux #359

Merged
merged 3 commits into from
May 20, 2021

Conversation

camjay
Copy link
Contributor

@camjay camjay commented May 17, 2021

Skip download in check mode on windows as linux installs do.
This will reporting a changed task in check mode when agent version is current.

@camjay camjay requested a review from a team as a code owner May 17, 2021 18:38
@bkabrda
Copy link
Contributor

bkabrda commented May 18, 2021

Hi! Thanks for sending the PR. It looks good, but I think we'll also need https://github.com/DataDog/ansible-datadog/pull/359/files#diff-bef70ee7f6fc30d9bf9be97dd8eaff36eb544b6e51dd36dc3be66f7e4f07610bR105 (the Delete temporary msi task) to have the not ansible_check_mode in when, because otherwise it tries to access the download_msi_result variable which is not available in the check mode.

@camjay
Copy link
Contributor Author

camjay commented May 18, 2021

thanks for catching that @bkabrda, i've added that conditional.

tasks/pkg-windows.yml Outdated Show resolved Hide resolved
Co-authored-by: Slavek Kabrda <slavek.kabrda@gmail.com>
@bkabrda
Copy link
Contributor

bkabrda commented May 20, 2021

LGTM now, merging. Thanks a lot for your contribution!

@bkabrda bkabrda merged commit 9568f25 into DataDog:master May 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants