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.

Sign up for GitHub

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

Jump to bottom

Add bootstrap manager for certificates #329

Merged
merged 7 commits into from
Nov 3, 2021
Merged

Add bootstrap manager for certificates #329

merged 7 commits into from
Nov 3, 2021

Conversation

chrismuellner
Copy link
Collaborator

  • Add 2nd manager to create and add certificates to CRD/webhook configurations on initial run
  • Start remaining controllers after successful bootstrapper run
  • Refactor subcommands in main.go to use switch

@chrismuellner chrismuellner marked this pull request as draft November 2, 2021 15:24
@chrismuellner chrismuellner marked this pull request as ready for review November 2, 2021 16:15
@chrismuellner chrismuellner changed the title Add bootstrap manager Add bootstrap manager for certificates Nov 2, 2021
@chrismuellner chrismuellner mentioned this pull request Nov 2, 2021
Closed
@chrismuellner chrismuellner merged commit e15602c into master Nov 3, 2021
@chrismuellner chrismuellner deleted the bugfix/certificates-missing-bootstrapmgr branch November 3, 2021 09:09
chrismuellner added a commit that referenced this pull request Nov 4, 2021
@chrismuellner
Add bootstrap manager for certificates (#329)
48c23a1 
Co-authored-by: Gabriel Krenn <gabriel.krenn@gmail.com>
(cherry picked from commit e15602c)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@0sewa0 0sewa0 0sewa0 approved these changes

@gkrenn gkrenn Awaiting requested review from gkrenn

@baichinger baichinger Awaiting requested review from baichinger

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@chrismuellner @0sewa0 @gkrenn