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

docs(kuberay-operator): update argo cd guide #2240

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hongbo-miao
Copy link

@hongbo-miao hongbo-miao commented Jul 12, 2024

Why are these changes needed?

New Argo CD today provide ServerSideApply=true option, so no need split to deploy any more. ☺️

This article explains: https://www.arthurkoziel.com/fixing-argocd-crd-too-long-error

This "Server Side Apply" is actually a good practice https://kubernetes.io/blog/2022/10/20/advanced-server-side-apply/ and will be default option for future Kubernetes version.

Related issue number

Checks

  • I've made sure the tests are passing.
  • Testing Strategy
    • Unit tests
    • Manual tests
    • This PR is not tested :(

You can see I successfully deployed:

image

@kevin85421
Copy link
Member

We have already moved most docs to Ray website. Would you mind opening a PR in the Ray repository? GitOps (ArgoCD, Flux) is something I want KubeRay to integrate well with. Are you interested in scheduling a chat so that I can understand more about how KubeRay can improve to better support ArgoCD? If you are interested in that, you can ping me in the Ray Slack workspace. My Slack handle is "Kai-Hsun Chen (ray team)". Thanks!

@kevin85421 kevin85421 self-assigned this Jul 12, 2024
@hongbo-miao
Copy link
Author

Sounds good, I will open a pull there! ☺️
I will ping you after.

@hongbo-miao
Copy link
Author

hongbo-miao commented Jul 13, 2024

Hmm I didn't find any Argo CD related document in the new website. Does it mean have to create a new document?

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.

2 participants