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

generate minimal config during installation #619

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

Conversation

wasaga
Copy link
Collaborator

@wasaga wasaga commented May 1, 2023

Summary

Since v0.22.0 IdP setup is not required for startup, and may be omitted to use hosted authenticate service.

That means the one-time job that generates initial bootstrap secrets may also generate a functional global config CRD in form:

apiVersion: ingress.pomerium.io/v1
kind: Pomerium
metadata:
  name: global
spec:
  secrets: pomerium/bootstrap

TODO: needs a bit of testing as we still may need to wait until CRD is created and recognized by the cluster though, as this job runs concurrently with CRD installation.

Related issues

Checklist

  • reference any related issues
  • updated docs
  • updated unit tests
  • updated UPGRADING.md
  • add appropriate tag (improvement / bug / etc)
  • ready for review

@wasaga wasaga requested a review from a team as a code owner May 1, 2023 21:27
@wasaga wasaga requested review from calebdoxsey and removed request for a team May 1, 2023 21:27
@wasaga wasaga added the enhancement New feature or request label May 1, 2023
@wasaga wasaga marked this pull request as draft May 1, 2023 21:32
@calebdoxsey calebdoxsey removed their request for review September 7, 2023 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant