-
Notifications
You must be signed in to change notification settings - Fork 94
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
Add to argocd documentation #151
Add to argocd documentation #151
Conversation
Signed-off-by: Omar Farag <ofarag@redhat.com>
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.
This is a great addition to the doc. Just one minor nit because I think it only applies to Linux environments.
@@ -32,6 +32,44 @@ Your IP address is usually next to the last entry of 'inet'. An IP address is al | |||
cluster1 true https://cluster1-control-plane:6443 18s | |||
cluster2 true https://cluster2-control-plane:6443 1s | |||
``` | |||
<details> | |||
<summary>Known issue when running setup script</summary> |
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.
<summary>Known issue when running setup script</summary> | |
<summary>Known issue when running the setup script on a Linux environment</summary> |
Signed-off-by: Omar Farag <ofarag@redhat.com>
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.
/approve
/lgtm
Signed-off-by: Omar Farag <ofarag@redhat.com>
@mikeshng I also added it to https://github.com/open-cluster-management-io/ocm/tree/main/solutions/setup-dev-environment since it has a similar setup script |
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.
/approve
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mikeshng, o-farag The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Signed-off-by: tytao <1522014784@qq.com> Signed-off-by: tytao <1522014784@qq.com>
I had trouble trying to follow this documentation because I couldn't complete the
./setup-ocm.sh
step. It took me a while to figure out the solution, hopefully this can help others who may run into the same issue.