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

[tests](docker)Add the Docker-compose test case yaml file for Apache Doris #2228

Merged
merged 3 commits into from
Feb 10, 2023

Conversation

FreeOnePlus
Copy link
Member

@FreeOnePlus FreeOnePlus commented Feb 3, 2023


I recently supported and optimized Apache Doris's Docker image orchestration work. We had an exchange last year. You hope that we can provide an easy-to-run docker-compose script for connectivity testing. Sorry for the long delay, I have submitted it PR to Master, if you have any questions, you can continue to communicate via email, my email is [email protected]. Good luck.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

  • Run mvn clean install apache-rat:check to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
  • If you have a group of commits related to the same change, please squash your commits into one and force push your branch using git rebase -i.
  • Mention the appropriate issue in your description (for example: addresses #123), if applicable.

To make clear that you license your contribution under the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

@FreeOnePlus
Copy link
Member Author

It should be noted that in the ENV parameter in the yaml script, where the local_host parameter is mentioned, it needs to be manually modified to the intranet IP of the current node, or the IP can be specified by building a subnet bridge.

@hansva
Copy link
Contributor

hansva commented Feb 3, 2023

thank you so much @FreeOnePlus !

@FreeOnePlus
Copy link
Member Author

Can this PR be integrated?

@hansva hansva merged commit 532950d into apache:master Feb 10, 2023
@hansva
Copy link
Contributor

hansva commented Feb 10, 2023

Thanks for the PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants