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

docker: restore the support for fixed UID/GID and standalone container #13877

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

shr-project
Copy link
Contributor

@AndrewXWei AndrewXWei requested a review from changsh726 July 2, 2021 03:06
@AndrewXWei AndrewXWei added the Infra System/Infrastructure related label Jul 2, 2021
@shr-project
Copy link
Contributor Author

shr-project commented Sep 7, 2021

ea64c50 (#14010) broke it even more, because now bootstrap.sh fails with:

[ OK ] Launched module dreamview.
Failed to start Dreamview. Please check /apollo/data/log or /apollo/data/core for more information

because after this commit the dreamview binary is not included in the standalone container anymore:

[apollo@in-runtime-docker:/apollo]$ /apollo/bazel-bin/modules/dreamview/dreamview
bash: /apollo/bazel-bin/modules/dreamview/dreamview: No such file or directory

@shr-project shr-project marked this pull request as ready for review November 11, 2021 10:23
ApolloAuto@ea64c50

This PR broke standalone docker image even more, because now bootstrap.sh fails with:

[ OK ] Launched module dreamview.
Failed to start Dreamview. Please check /apollo/data/log or /apollo/data/core for more information

because after this commit the dreamview binary is not included in the standalone container anymore:

[apollo@in-runtime-docker:/apollo]$ /apollo/bazel-bin/modules/dreamview/dreamview
bash: /apollo/bazel-bin/modules/dreamview/dreamview: No such file or directory
@ApolloAuto ApolloAuto locked and limited conversation to collaborators Nov 22, 2021
@ApolloAuto ApolloAuto unlocked this conversation Nov 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Infra System/Infrastructure related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants