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

HDDS-7047. Add Recon to ozone-topology sample compose environment #3641

Merged
merged 3 commits into from Aug 9, 2022

Conversation

dombizita
Copy link
Contributor

What changes were proposed in this pull request?

I added Recon as a service to the compose/ozone-topology environment. I also added the Recon robot tests to the test.sh in the ozone-topology folder.

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-7047

How was this patch tested?

I tested it in the compose/ozone-topology environment via docker and run the test.sh locally.
https://github.com/dombizita/ozone/actions/runs/2769844964

Change-Id: I4d555a1f123175ce7c173355fa128d497182a390
Change-Id: I822cbeab897707c5fc34e74ba0424003f3387b22
Copy link
Contributor

@adoroszlai adoroszlai left a comment

Choose a reason for hiding this comment

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

Thanks @dombizita for the patch, LGTM with minor comment.

env_file:
- docker-config
environment:
OZONE-SITE.XML_ozone.replication: ${OZONE_REPLICATION_FACTOR:-1}
Copy link
Contributor

Choose a reason for hiding this comment

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

OZONE_REPLICATION_FACTOR lets us use scalable environments (ozone, ozone-ha, etc.) with a single datanode by default, and scale to 3 if needed. It is not applicable to the ozone-topology environment. With a fixed set of 6 datanodes it is configured as:

So we can remove this extra config:

Suggested change
OZONE-SITE.XML_ozone.replication: ${OZONE_REPLICATION_FACTOR:-1}

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for explaining this, I updated my patch.

Change-Id: I9645f18f4bb1e832ca02fdbf5fa8b169d963d143
@fapifta fapifta merged commit 1afaa65 into apache:master Aug 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants