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

chore: update readme #344

Merged
merged 6 commits into from
Oct 26, 2023
Merged

chore: update readme #344

merged 6 commits into from
Oct 26, 2023

Conversation

namkyu1999
Copy link
Member

What this PR does / why we need it:

I updated installation.md for ARM users. Briefly, the official MongoDB Helm chart does not support ARM, so I added commands for users to install Litmus with an unofficial MongoDB image.

Special notes for your reviewer:

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • DCO signed
  • Chart Version bumped
  • Variables are documented in the README.md

Signed-off-by: namkyu1999 <lak9348@konkuk.ac.kr>
Copy link
Member

@Jasstkn Jasstkn left a comment

Choose a reason for hiding this comment

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

Hi @namkyu1999! Thank you for your proposed changes. However, I would like to avoid putting some third-party image as officially recommended by Litmus maintainers.

I think that we can just refer to the original issue: bitnami/charts#7305 and propose an example how to override the image repository and tag via helm values.

Let me know what do you think about it?

@namkyu1999
Copy link
Member Author

namkyu1999 commented Oct 24, 2023

Hi @Jasstkn, Thanks for the comment.
Let me correct one thing I said. In this installation, We continue to utilize the official MongoDB Helm charts created by Bitnami. But they use bitnami/mongodb image by default. So I just override helm values by --set flag like below.

helm install chaos litmuschaos/litmus --namespace=litmus \
--set portal.frontend.service.type=NodePort \
--set mongodb.image.registry=ghcr.io/zcube \
--set mongodb.image.repository=bitnami-compat/mongodb \
--set mongodb.image.tag=6.0.5

Is this still a non-recommended way?

@Jasstkn
Copy link
Member

Jasstkn commented Oct 25, 2023

Hi @Jasstkn, Thanks for the comment. Let me correct one thing I said. In this installation, We continue to utilize the official MongoDB Helm charts created by Bitnami. But they use bitnami/mongodb image by default. So I just override helm values by --set flag like below.

helm install chaos litmuschaos/litmus --namespace=litmus \
--set portal.frontend.service.type=NodePort \
--set mongodb.image.registry=ghcr.io/zcube \
--set mongodb.image.repository=bitnami-compat/mongodb \
--set mongodb.image.tag=6.0.5

Is this still a non-recommended way?

I don't want to include in the README recommendation to use specific third-party image. So basically, leave the section with the ARM use-case, add link to the Bitnami issue and add a command like the following:

helm install chaos litmuschaos/litmus --namespace=litmus \
--set portal.frontend.service.type=NodePort \
--set mongodb.image.registry=<put_registry> \
--set mongodb.image.repository=<put_image_repository> \
--set mongodb.image.tag=<put_image_tag>

Signed-off-by: namkyu1999 <lak9348@konkuk.ac.kr>
@namkyu1999
Copy link
Member Author

namkyu1999 commented Oct 25, 2023

Hi @Jasstkn ,
I updated it like below.

## ARM support
## Litmus helm chart depends on bitnami/mongodb, which uses a mongodb image not supported on ARM.
## To install Litmus on an ARM-based server, use a custom MongoDB ARM image instead of the default.
# helm install chaos litmuschaos/litmus --namespace=litmus \
# --set mongodb.image.registry=<put_registry> \
# --set mongodb.image.repository=<put_image_repository> \
# --set mongodb.image.tag=<put_image_tag>

Can you take a look at this one again, please?

namkyu1999 added 3 commits October 25, 2023 22:24
Signed-off-by: namkyu1999 <lak9348@konkuk.ac.kr>
Signed-off-by: namkyu1999 <lak9348@konkuk.ac.kr>
Signed-off-by: namkyu1999 <lak9348@konkuk.ac.kr>
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Signed-off-by: namkyu1999 <lak9348@konkuk.ac.kr>
@Jasstkn Jasstkn merged commit f675ac4 into litmuschaos:master Oct 26, 2023
4 checks passed
@Jasstkn
Copy link
Member

Jasstkn commented Oct 26, 2023

@namkyu1999 thank you for the contribution! merged 🚀

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

Successfully merging this pull request may close these issues.

5 participants