Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@AndrewKahr
Copy link
Member

Changes

  • Add tests for Windows images
  • Consolidate testing workflows
  • Fix checking out main instead of latest tag for building images
  • Fix using GITHUB_ENV rather than GITHUB_OUTPUT which broke Windows image builds
  • Improve error checking on Windows editor installations
  • General improvements to error checking on editor/module installations
  • Hide node error spam from Unity Hub by piping stderr to null. This is ok because the errors we care about end up on stdout

Checklist

  • Read the contribution guide and accept the code of conduct
  • Readme (updated or not needed)

@github-actions
Copy link

Cat Gif

&& apt-get -q install -y "unityhub=$hubVersion" \
&& apt-get clean

# Bug in xvfb-run that causes it to redirect stderr to stdout. We want it separate
Copy link
Member

@GabLeRoux GabLeRoux Apr 11, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you have details about that bug? (Just curious). Maybe it's worth linking to the bug and its details

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AndrewKahr AndrewKahr marked this pull request as ready for review April 12, 2023 11:33
Copy link
Member

@GabLeRoux GabLeRoux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it would be worth it at some point to extract some scripts from workflow files for readability (ex; in .github/workflows/test.yml). Other than that, this looks good to me 👍 🤯

@GabLeRoux
Copy link
Member

The following two jobs both failed with a timeout, but everything else is green.

https://github.com/game-ci/docker/actions/runs/4675713024/jobs/8286515438?pr=206
https://github.com/game-ci/docker/actions/runs/4675713024/jobs/8286515018?pr=206

Are we ok with that?

@AndrewKahr
Copy link
Member Author

I checked the timeouts and they are from Unity not exiting despite completing the build successfully in well under 15 minutes. This is a known issue but out of scope for this PR so it is acceptable. The images themselves are building and installing the correct modules which is the main thing that matters for the scope of this PR.

@AndrewKahr AndrewKahr merged commit 2345c23 into main Apr 12, 2023
@AndrewKahr AndrewKahr deleted the additional-testing branch April 12, 2023 18:51
mob-sakai pushed a commit to mob-sakai/docker that referenced this pull request Dec 27, 2023
# Conflicts:
#	.github/workflows/new-ubuntu-base-image-requested.yml
#	.github/workflows/new-ubuntu-hub-image-requested.yml
#	.github/workflows/new-ubuntu-legacy-editor-image-requested.yml
#	.github/workflows/new-ubuntu-post-2019-2-editor-image-requested.yml
#	.github/workflows/new-windows-base-image-requested.yml
#	.github/workflows/new-windows-hub-image-requested.yml
#	.github/workflows/new-windows-legacy-editor-image-requested.yml
#	.github/workflows/new-windows-post-2019-2-editor-image-requested.yml
#	.github/workflows/test.yml
#	images/windows/editor/Dockerfile
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.

3 participants