Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Added ADR for the provisioning functionality in KIM #202
Added ADR for the provisioning functionality in KIM #202
Changes from all commits
b1d72f2
d1e330e
32e7b22
6db8a66
e9ac688
8ae664e
1e975a7
3ec8899
a317f5e
68b441c
033249b
754b5d4
d56059a
93ca887
7b68956
adf0029
c47deac
20029d5
e63d364
5125080
01c53d7
eca112a
ff8a2f0
c7468f0
3d91dcb
fa9269b
35f2134
8c65e3a
a3147e6
ce71c36
506e938
a00237d
af9a219
b379f63
a3267a8
5e598bd
c12ce30
2a54485
5a52cb3
52234ed
aba0080
7dc7e90
18b165c
2de7101
ccdd130
06630bb
d8707aa
7b07657
53aa78b
8dbbca7
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Please also add support for these
workers[*].machineControllerManager
fields:https://gardener.cloud/docs/gardener/api-reference/core/#core.gardener.cloud/v1beta1.Worker
SRE sometimes manually tweak these settings, knowing that provisioner doesn't revert this change, however these are important details to control. Especially with KIM, we would like to provide default values for
machineDrainTimeout
andmaxEvictRetries
attributes different than the gardeber defaults.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.
It's not included in the examples, but will be supported out of the box (we use full Gardener type for workers definition).