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

fix: Enable global confirmation for installing mingw #11835

Merged
merged 2 commits into from
Sep 19, 2022
Merged

Conversation

sspaink
Copy link
Contributor

@sspaink sspaink commented Sep 19, 2022

The windows tests are timing out due to requiring user input for installing mingw, the script to install Go resolved this issue by calling choco feature enable -n allowGlobalConfirmation so I moved this outside of the script to always run.

e.g. https://app.circleci.com/pipelines/github/influxdata/telegraf/12709/workflows/4d70fb4e-921c-4060-a605-d0d41100afbb/jobs/204886

mingw v11.2.0.07112021 [Approved]
mingw package files install completed. Performing other installation steps.
The package mingw wants to run 'chocolateyinstall.ps1'.
Note: If you don't run this script, the installation will fail.
Note: To confirm automatically next time, use '-y' or consider:
choco feature enable -n allowGlobalConfirmation
Do you want to run the script?([Y]es/[A]ll - yes to all/[N]o/[P]rint): 

@telegraf-tiger telegraf-tiger bot added the fix pr to fix corresponding bug label Sep 19, 2022
@sspaink sspaink requested a review from a team September 19, 2022 15:00
@telegraf-tiger
Copy link
Contributor

@sspaink sspaink merged commit b3fc1b7 into master Sep 19, 2022
@sspaink sspaink deleted the fixwindowstest branch September 19, 2022 15:32
reimda pushed a commit that referenced this pull request Sep 19, 2022
dba-leshop pushed a commit to dba-leshop/telegraf that referenced this pull request Oct 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix pr to fix corresponding bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants