-
Notifications
You must be signed in to change notification settings - Fork 379
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
[#2379] fix(hive-ci-image): Let user hive be able to access MySQL #2523
Conversation
@danhuawang Could you please help to test these changes in your environment, thanks! |
It's ok in my test environment. |
@mchades please help to continue this PR. |
The version of |
The modified Dockfile also passed tests in my environment. Afer @charliecheng630 address the comments, I will push the new image |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent contribution, looking forward to more in the future!
What changes were proposed in this pull request?
Let user hive be able to access MySQL in gravitino-ci-hive container.
Why are the changes needed?
Fix: #2379
Does this PR introduce any user-facing change?
No
How was this patch tested?
./build-docker.sh --platform linux/arm64 --type hive --image datastrato/gravitino-ci-hive --tag 0.1.9