danielfoehrKn / kubeswitch

The kubectx for operators.
https://danielfoehrkn.medium.com/the-case-of-kubeswitch-aff4b6a04ae7
Apache License 2.0
832 stars 80 forks source link

garden integration broken #83

Closed MichaelSp closed 6 months ago

MichaelSp commented 1 year ago

I've configured gardener with the following options:

# ~/.garden/gardenctl-v2.yaml
gardens:
  - identity: "landscape-canary"
    kubeconfig: "~/.garden/landscape-canary.yaml"
  - identity: "landscape-live"
    kubeconfig: "~/.garden/landscape-live.yaml"
# ~/.kube/switch-config.yaml
kind: SwitchConfig
version: v1alpha1
showPreview: false
refreshIndexAfter: 4h
kubeconfigStores:
- kind: gardener
  paths:
  - "garden"
  config:
    gardenerAPIKubeconfigPath: "~/.garden/landscape-canary.yaml"
    landscapeName: "landscape-canary"
- kind: gardener
  refreshIndexAfter: 6h
  config:
    gardenerAPIKubeconfigPath: "~/.garden/landscape-live.yaml"
    landscapeName: "landscape-live"

The WARN message:

Why does kubeswitch want to lookup the cluster-identity? That's already in the config.

danielfoehrKn commented 1 year ago

Hi, yes, currently it always tries to get the landscape-identity from the config map, no matter if an alternative landscape name is given. That should be fixed. Thanks for reporting.

danielfoehrKn commented 6 months ago

Hi, as I have left the Gardener team at SAP, I don't have access to a Gardener installation any more. This means that I wont be able to provide a fix that works with enough assurance here. I know that many Gardener users use gardenctl which I encourage everyone to also try (though I still preferred using kubeswitch over gardenctl during my time at SAP due to having one place for all of my kubeconfigs). AFAIK the Gardener integration in kubeswitch still works at this moment in time but probably won't be developed further (at least not by me) - so I'll close this issue for now. If people report that the integration is fundamentally broken for newer Gardener versions, I'll deprecate Gardener support.