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

feat(charts/authentik): add blueprints-sidecar to collect from cluster #146

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

Conversation

genofire
Copy link
Contributor

@genofire genofire commented Apr 6, 2023

fix:

Helm values:

serviceAccount:
  create: true
sidecar:
  blueprints:
    enabled: true
    namespace: "ALL" <1>

<1> normally just current namespace

Collects ConfigMaps and Secrets based on label and put them into goauthentik, here an example ConfigMap:

apiVersion: v1
kind: ConfigMap
metadata:
  name: goauthentik-blueprint-sidecar-test
  namespace: "default"
  labels:
    goauthentik_blueprint: "1"
data:
  test.yaml: |-
    version: 1
    metadata:
      name: sidecar-test
    entries: []

@genofire genofire force-pushed the feat/blueprints-sidecar branch 3 times, most recently from ecf0dce to 779cd53 Compare April 6, 2023 00:35
@genofire genofire marked this pull request as ready for review April 6, 2023 00:42
@genofire genofire force-pushed the feat/blueprints-sidecar branch 2 times, most recently from e029b1d to 9a2529a Compare April 6, 2023 07:29
@genofire
Copy link
Contributor Author

@BeryJu please review

@genofire genofire force-pushed the feat/blueprints-sidecar branch 4 times, most recently from ab19593 to e0050a0 Compare April 18, 2023 15:16
@genofire
Copy link
Contributor Author

genofire commented Apr 18, 2023

@BeryJu i do not know where this CI error cames from:

Error: INSTALLATION FAILED: serviceaccounts "authentik-q63g50s5tb" already exists

@genofire genofire force-pushed the feat/blueprints-sidecar branch 4 times, most recently from 122c5c1 to 7be14dd Compare April 18, 2023 17:14
@genofire
Copy link
Contributor Author

solved

@benedikt-bartscher
Copy link

#156

@genofire
Copy link
Contributor Author

thanks @benedikt-bartscher i will add the sidecar to all component

@genofire
Copy link
Contributor Author

done - lets review again

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.

None yet

2 participants