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

Misc Improvements and Bug Fixes #206

Merged
merged 28 commits into from
Apr 12, 2023
Merged

Misc Improvements and Bug Fixes #206

merged 28 commits into from
Apr 12, 2023

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

@@ -16,8 +16,11 @@ RUN sh -c 'echo "deb https://hub.unity3d.com/linux/repos/deb stable main" > /etc
&& 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

@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.

2 participants