Skip to content

Conversation

@sunmou99
Copy link
Contributor

@sunmou99 sunmou99 commented Nov 29, 2022

Description

Current unity installation process on newer ubuntu runners has an issue: https://github.com/firebase/firebase-unity-sdk/actions/runs/3567833638/jobs/5996016337#step:8:74

Updated Unity Installation process on ubuntu runners based on: https://docs.unity3d.com/hub/manual/InstallHub.html#install-hub-linux


Testing

https://github.com/firebase/firebase-unity-sdk/actions/runs/3578235575/jobs/6018161702

Also see the comment below


Type of Change

Place an x the applicable box:

  • Bug fix. Add the issue # below if applicable.
  • New feature. A non-breaking change which adds functionality.
  • Other, such as a build process or documentation change.

@sunmou99 sunmou99 added the tests-requested: quick Trigger a quick set of integration tests. label Nov 29, 2022
@sunmou99 sunmou99 added tests-requested: quick Trigger a quick set of integration tests. and removed tests-requested: quick Trigger a quick set of integration tests. labels Nov 29, 2022
@sunmou99 sunmou99 changed the title Fix Unity installation error on Linux: install libfuse2 Fix Unity installation error on Linux Nov 29, 2022
@sunmou99 sunmou99 added tests-requested: quick Trigger a quick set of integration tests. and removed tests-requested: quick Trigger a quick set of integration tests. labels Nov 29, 2022
@sunmou99 sunmou99 added tests-requested: quick Trigger a quick set of integration tests. and removed tests-requested: quick Trigger a quick set of integration tests. labels Nov 29, 2022
@github-actions github-actions bot added tests: in-progress This PR's integration tests are in progress. and removed tests-requested: quick Trigger a quick set of integration tests. labels Nov 29, 2022
@github-actions
Copy link

github-actions bot commented Nov 29, 2022

❌  Integration test FAILED

Requested by @firebase-workflow-trigger[bot] on commit f19dc22
Last updated: Wed Dec 14 19:02 PST 2022
View integration test log & download artifacts

Failures Configs
auth [TEST] [FAILURE] [2020] [1/2 Build OS(s): macos] [1/6 Platform(s): iOS] [1/3 Test Device(s): ios_target]
firestore [TEST] [FAILURE] [2020] [1/2 Build OS(s): windows] [1/6 Platform(s): Windows] [1/3 Test Device(s): github_runner]
messaging [TEST] [ERROR] [2020] [1/2 Build OS(s): macos] [1/6 Platform(s): iOS] [1/3 Test Device(s): ios_target]

@sunmou99 sunmou99 added tests-requested: quick Trigger a quick set of integration tests. and removed tests: in-progress This PR's integration tests are in progress. labels Nov 29, 2022
@sunmou99 sunmou99 marked this pull request as ready for review November 29, 2022 22:14
@sunmou99 sunmou99 requested a review from a-maurice November 29, 2022 22:17
@github-actions github-actions bot added the tests: failed This PR's integration tests failed. label Nov 29, 2022
build_desktop:
name: build-linux-unity${{inputs.unity_version}}-CPP${{ inputs.firebase_cpp_sdk_version }}
runs-on: ubuntu-20.04
runs-on: ubuntu-latest
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this change intentional? Remember we need to ping to ubuntu 20

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes.
I believe github will remove ubuntu 20 runner in the future. Then, we still need to solve the issue.
This PR is to fix the installation issue on newer ubuntu runners, so that we don't have to ping to ubuntu 20.

Copy link
Contributor

@StephenHodgson StephenHodgson Dec 1, 2022

Choose a reason for hiding this comment

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

Unity does not support Ubuntu 22.04. This should be reverted until the next Unity Hub release that does support it.

Copy link
Contributor

Choose a reason for hiding this comment

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

See forum post for details https://forum.unity.com/threads/running-unity-on-ubuntu-22-04.1284083/

I've already tried the workaround in my installation scripts but cannot get around the fuse error

Copy link
Contributor Author

@sunmou99 sunmou99 Dec 1, 2022

Choose a reason for hiding this comment

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

Thanks for the info.
Yeah, I also tried the workaround and it didn't work.
But, we are building SDK here and only installing (not launching) Editor.
So, it worked in my CI:

I1129 21:42:55.199143 140596922395520 unity_installer.py:343] run_with_retry: xvfb-run unityhub --headless install --version 2020.3.40f1 --changeset ba48d4efcef1 (attempt 1 of 3)
I1129 21:46:01.864848 140596922395520 unity_installer.py:346] cmd stdout: This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). The promise rejected with the reason:
Error: Unknown or unsupported transport “disabled” for address “disabled:”
This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). The promise rejected with the reason:
...
Progress:
[Unity 2020.3.40f1] downloading 98.77%
Progress:
[Unity 2020.3.40f1] downloading 100.00%
Progress:
[Unity 2020.3.40f1] finished downloading.
Progress:
[Unity 2020.3.40f1] finished downloading.
Progress:
[Unity 2020.3.40f1] queued for install.
Progress:
[Unity 2020.3.40f1] queued for install.
Progress:
[Unity 2020.3.40f1] validating installation...
Progress:
[Unity 2020.3.40f1] validating installation...
Progress:
[Unity 2020.3.40f1] installing...
Progress:
[Unity 2020.3.40f1] installed successfully.
All Tasks Completed Successfully.
Progress:
[Unity 2020.3.40f1] installed successfully.
All Tasks Completed Successfully.

Copy link
Contributor

Choose a reason for hiding this comment

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

You guys don't do and validation and testing in editor?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We do. But not on Ubuntu runners.

@github-actions github-actions bot added tests: in-progress This PR's integration tests are in progress. tests: failed This PR's integration tests failed. and removed tests-requested: quick Trigger a quick set of integration tests. tests: failed This PR's integration tests failed. labels Nov 29, 2022
@firebase-workflow-trigger firebase-workflow-trigger bot removed the tests: in-progress This PR's integration tests are in progress. label Nov 30, 2022
@sunmou99 sunmou99 merged commit b3711a6 into main Dec 14, 2022
@sunmou99 sunmou99 deleted the bugfix/unity-linux branch December 14, 2022 23:25
@sunmou99 sunmou99 restored the bugfix/unity-linux branch December 14, 2022 23:25
@github-actions github-actions bot added tests: in-progress This PR's integration tests are in progress. tests: failed This PR's integration tests failed. and removed tests: failed This PR's integration tests failed. labels Dec 15, 2022
@firebase-workflow-trigger firebase-workflow-trigger bot removed the tests: in-progress This PR's integration tests are in progress. label Dec 15, 2022
mrober pushed a commit that referenced this pull request Jan 3, 2023
@firebase firebase locked and limited conversation to collaborators Jan 14, 2023
@a-maurice a-maurice deleted the bugfix/unity-linux branch May 8, 2024 19:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

tests: failed This PR's integration tests failed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants