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 Windows & MacOS GH CI testing #144

Closed

Conversation

detjensrobert
Copy link
Contributor

@detjensrobert detjensrobert commented Aug 3, 2021

Description

Since these VMs don't need to reboot, run tests directly on the runner instead of using a nested VM.

This should fix the testing failures with the Windows and MacOS testing from #143.

Signed-off-by: Robert Detjens [email protected]

Check List

  • All tests pass. See TESTING.md for details.
  • New functionality includes testing.
  • New functionality has been documented in the README if applicable.

@detjensrobert detjensrobert requested a review from a team August 3, 2021 22:49
@detjensrobert detjensrobert self-assigned this Aug 3, 2021
@detjensrobert detjensrobert force-pushed the detjensrobert/gh-actions-fix branch from 0f2ae04 to cbccc11 Compare August 3, 2021 23:19
@detjensrobert detjensrobert changed the title Detjensrobert/gh actions fix Fix Windows & MacOS GH CI testing Aug 3, 2021
@detjensrobert detjensrobert added the Release: Patch Release to Chef Supermarket as a version patch when merged label Aug 3, 2021
@detjensrobert detjensrobert force-pushed the detjensrobert/gh-actions-fix branch from f1d857b to 5687864 Compare August 4, 2021 00:05
.github/workflows/ci.yml Outdated Show resolved Hide resolved
@detjensrobert
Copy link
Contributor Author

detjensrobert commented Aug 5, 2021

Looks like the Windows CI test is failing due to https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands before even getting to the integration converge.
Fix here: actionshub/chef-install#15

@ramereth
Copy link
Contributor

ramereth commented Aug 5, 2021

Looks like the Windows CI test is failing due to https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands before even getting to the integration converge.
Fix here: actionshub/chef-install#15

Would you mind temporarily using your branch here so we can see the fix in action?

@xorima
Copy link
Contributor

xorima commented Aug 6, 2021

Actions fix merged.

detjensrobert and others added 5 commits August 31, 2021 22:50
Since these VMs don't need to reboot, run directly on the runner instead of
using a nested VM.

Signed-off-by: Robert Detjens <[email protected]>
Signed-off-by: Robert Detjens <[email protected]>
@ramereth ramereth force-pushed the detjensrobert/gh-actions-fix branch from 348d31c to 9738cbd Compare September 1, 2021 05:51
@damacus damacus changed the base branch from main to automated/standardfiles February 14, 2022 09:24
@damacus damacus removed the Release: Patch Release to Chef Supermarket as a version patch when merged label Feb 14, 2022
@ramereth
Copy link
Contributor

I'm working on resolving this in #146

@ramereth ramereth closed this Feb 16, 2022
@ramereth ramereth deleted the detjensrobert/gh-actions-fix branch February 16, 2022 00:48
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.

4 participants