-
Notifications
You must be signed in to change notification settings - Fork 6.4k
feat: manage clusters via proxy #20374
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
feat: manage clusters via proxy #20374
Conversation
Signed-off-by: pashakostohrys <pavel@codefresh.io>
❌ Preview Environment deleted from BunnyshellAvailable commands (reply to this comment):
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #20374 +/- ##
=========================================
Coverage ? 55.98%
=========================================
Files ? 322
Lines ? 44772
Branches ? 0
=========================================
Hits ? 25067
Misses ? 17105
Partials ? 2600 ☔ View full report in Codecov by Sentry. |
Signed-off-by: pashakostohrys <pavel@codefresh.io>
Signed-off-by: pashakostohrys <pavel@codefresh.io>
Signed-off-by: pashakostohrys <pavel@codefresh.io>
Signed-off-by: pashakostohrys <pavel@codefresh.io>
nice work 👍 |
Signed-off-by: pashakostohrys <pavel@codefresh.io>
Signed-off-by: pashakostohrys <pavel@codefresh.io>
Signed-off-by: pashakostohrys <pavel@codefresh.io>
Signed-off-by: pashakostohrys <pavel@codefresh.io>
Thanks @reggie-k for the in-depth review! |
* feat: proxy support Signed-off-by: pashakostohrys <pavel@codefresh.io> * feat: proxy support Signed-off-by: pashakostohrys <pavel@codefresh.io> * feat: proxy support Signed-off-by: pashakostohrys <pavel@codefresh.io> * fix linter Signed-off-by: pashakostohrys <pavel@codefresh.io> * small improvements Signed-off-by: pashakostohrys <pavel@codefresh.io> * add cluster test Signed-off-by: pashakostohrys <pavel@codefresh.io> * fix linter Signed-off-by: pashakostohrys <pavel@codefresh.io> * change error message Signed-off-by: pashakostohrys <pavel@codefresh.io> * override always will change proxy url Signed-off-by: pashakostohrys <pavel@codefresh.io> --------- Signed-off-by: pashakostohrys <pavel@codefresh.io> Signed-off-by: austin5219 <3936059+austin5219@users.noreply.github.com>
on what version this feature is available? I can see that it's already on master branch |
@dodistyo , it will be part of 2.14 |
currently, the feature should be available in the image tag Update: |
Yeah @dodistyo i it should work under latest tag. I will take a look why it may happens |
* feat: proxy support Signed-off-by: pashakostohrys <pavel@codefresh.io> * feat: proxy support Signed-off-by: pashakostohrys <pavel@codefresh.io> * feat: proxy support Signed-off-by: pashakostohrys <pavel@codefresh.io> * fix linter Signed-off-by: pashakostohrys <pavel@codefresh.io> * small improvements Signed-off-by: pashakostohrys <pavel@codefresh.io> * add cluster test Signed-off-by: pashakostohrys <pavel@codefresh.io> * fix linter Signed-off-by: pashakostohrys <pavel@codefresh.io> * change error message Signed-off-by: pashakostohrys <pavel@codefresh.io> * override always will change proxy url Signed-off-by: pashakostohrys <pavel@codefresh.io> --------- Signed-off-by: pashakostohrys <pavel@codefresh.io> Signed-off-by: Adrian Aneci <aneci@adobe.com>
I'm on version v2.14.5+f463a94 and this doesn't seem to work here:
I also confirmed that access using the given proxyUrl is actually working ... |
Have you tried to debug it by exec to the argocd server pod? @guhilling |
@dodistyo Sorry for the late reply. Was quite busy last week. I just checked but have no clue how the argocd-server would actually access the "other" cluster? Any proposal? I made sure that the proxy server is actually reachable from the namespace/cluster ... |
@guhilling Try to exec pod argocd-application-controller sts make sure the generated kubeconfig has the correct format. if the format is incorrect, adjust accordingly |
@dodistyo the generated kubeconfig is actually missing the certificate data But I also get a "timeout" error when looking at the state of the cluster (trying to access the remote cluster version). |
how did you generate the kubeconfig? usually the certificate is already there when we generate the kubeconfig. also, what kind of kubernetes cluster do you run? |
I added the certificate and the kubeconfig generated by "argocd admin cluster kubeconfig" looks good now (contains inline certificate and correct proxy-url). |
I forgot that i was running custom argocd image, i might added the kubectl when building the image. now since you're successfully generate the kubeconfig, you might try to deploy debug pod with kubectl installed. and try to connect to the cluster with the generated kubeconfig. |
Made sure it works using:
|
I'm experiencing an issue when trying to add multiple GKE clusters to ArgoCD. Currently, ArgoCD uses the Kubernetes API URL as the unique identifier for clusters. However, in our GKE setup, all clusters use the same API IP address format (https://x.x.x.x) even though they're in separate VPCs. This creates a conflict in ArgoCD that prevents us from adding multiple clusters, as they all appear to have the same identifier. It would be extremely useful if ArgoCD could support alternative cluster identification mechanisms or allow custom identifiers to be assigned, rather than relying solely on the API server URL as the unique identifier. |
May fix
#8314
#6043
Forked from: #9496
Huge thanks and credit to initial contributor @ls0f