-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Fix 1681 (1) #2039
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
Fix 1681 (1) #2039
Conversation
Signed-off-by: wind57 <[email protected]>
Signed-off-by: wind57 <[email protected]>
Signed-off-by: wind57 <[email protected]>
Signed-off-by: wind57 <[email protected]>
| * @author wind57 | ||
| */ | ||
| public interface SecretsCache { | ||
| public enum ReadType { |
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.
the idea of this PR is here and this is what the OP has described in the issue.
At the moment, we read configmaps and secrets in a batch fashion, meaning all in a namespace. For some users this means reading a lot of sources and its not OK. So we introduce the possibility to read them individually.
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.
its a lot of changes, but they all stem from the above separation. I've written quite a lot of tests to be sure that we are OK with the old and new implementation
Signed-off-by: wind57 <[email protected]>
Signed-off-by: wind57 <[email protected]>
Signed-off-by: wind57 <[email protected]>
|
@ryanjbaxter fix one is ready also: lots of breaking changes were needed to support this. thank you |
| by setting `spring.cloud.kubernetes.config.retry.enabled=false`. | ||
|
|
||
|
|
||
| NOTE: Since version `5.0.0-M3`, we have introduced the possibility to read sources individually. Until now, we would go to the namespace and read all the configmaps / secrets available and then filter out the ones requested. Since `5.0.0-M3` you can specify that you want to read them individually, by setting the property: `spring.cloud.kubernetes.config.read-type=SINGLE`. The previous option to read them all in a namespace is controlled by `spring.cloud.kubernetes.config.read-type=BATCH` and it is the default option. |
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.
I would drop the -M3 here, no one will use a milestone once the GA versions is out
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.
ok, changed
Signed-off-by: wind57 <[email protected]>
No description provided.