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

Spec for workload cluster OIDC auth as the default auth method #1229

Closed
Tracked by #2473
gusevda opened this issue Jul 11, 2022 · 2 comments
Closed
Tracked by #2473

Spec for workload cluster OIDC auth as the default auth method #1229

gusevda opened this issue Jul 11, 2022 · 2 comments
Assignees
Labels
team/bigmac Team BigMac

Comments

@gusevda
Copy link

gusevda commented Jul 11, 2022

There are different aspects we need to touch to achieve workload cluster auth

Scope

We start with CAPA first, then look at other providers. The idea is that updating default apps will enable this.

Default Apps

  • nginx ingress controller
  • Dex
  • athena
  • cert-manager

We can add these to the default apps with sane defaults but we need to be mindful of existing apps installed
We could add a value to disable dex/oidc but we shouldn't really do it. Rather let customers migrate oidc setup first before updating to the new default apps version.
What about private clusters?
What about custom certs?

Dex operator

We need to solve the race condition issue with secret vs configmap connector configuration

Auth

We need to ensure that a clusterrolebinding is present for (giantswarm) admin group

Cluster App

We also have to think about the api server flags that need to be added, and restarting control plane nodes
We also have some customers that use oidc but do not use dex and we need to migrate them and make sure not to override existing settings

Login

We want to have a seamless login experience using kgs/opsctl

@gusevda gusevda added the needs/refinement Needs refinement in order to be actionable label Jul 11, 2022
@anvddriesch
Copy link

Update:
Dex-operator manages configuration for WCs now, so let's flesh this one out.

@gawertm gawertm removed the needs/refinement Needs refinement in order to be actionable label May 17, 2023
@architectbot architectbot added the team/bigmac Team BigMac label May 31, 2023
@anvddriesch
Copy link

The spec points we came up with during refinement are all reflected as issues in the main epic here #2473
I will close this for now.

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

No branches or pull requests

4 participants