-
Notifications
You must be signed in to change notification settings - Fork 462
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
[gardenlet] Switch Bastion
controller to controller-runtime
#6839
Conversation
Skipping CI for Draft Pull Request. |
/test all |
/assign |
b9295cc
to
b057764
Compare
/test all |
Bastion
controller to controller-runtime
Bastion
controller to controller-runtime
Now, that we have the controller-runtime manager with both caches for garden and seed cluster, we could easily set up watches for the This was also one motivation behind doing this refactoring in the first place. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally, please also follow
- [gardenlet] Switch
BackupBucket
controller tocontroller-runtime
#6837 (comment) - [gardenlet] Switch
BackupBucket
controller tocontroller-runtime
#6837 (comment) - [gardenlet] Switch
BackupBucket
controller tocontroller-runtime
#6837 (comment)
Also, similar to #6837 (comment), let's first address the other comments before doing this refactoring.
0c8a08e
to
8c601ee
Compare
/test all |
8c601ee
to
a42eefe
Compare
@acumino WDYT about adding this to the |
Remove the wait loop for extensions Bastion and instead watch the extensions Bastion in the seed namespace
This function will get project name from shoot technical id
fb14b0a
to
f449262
Compare
f449262
to
3f029c5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/approve
Thanks @ary1992 and @shafeeqes
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: rfranzke The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/milestone v1.60 |
/lgtm |
How to categorize this PR?
/area dev-productivity scalability
/kind enhancement
What this PR does / why we need it:
Refactor the
Bastion
controller to controller-runtime.Which issue(s) this PR fixes:
Part of #4251
Special notes for your reviewer:
Generally, we want to follow this cookbook while refactoring existing controllers:
Add documentation
Add integration test based on envtest (if not already present)
Switch controller to controller-runtime
Release note: