Skip to content

Upgrade controller-runtime to v0.16.3#235

Merged
MarcosDY merged 2 commits intospiffe:mainfrom
MarcosDY:upgrade-controller-runtime
Dec 20, 2023
Merged

Upgrade controller-runtime to v0.16.3#235
MarcosDY merged 2 commits intospiffe:mainfrom
MarcosDY:upgrade-controller-runtime

Conversation

@MarcosDY
Copy link
Copy Markdown
Collaborator

No description provided.

@MarcosDY MarcosDY requested a review from azdagron as a code owner October 20, 2023 21:42
if len(o.Cache.Namespaces) == 0 && configSpec.CacheNamespace != "" {
o.Cache.Namespaces = []string{configSpec.CacheNamespace}
if len(o.Cache.DefaultNamespaces) == 0 && configSpec.CacheNamespace != "" {
o.Cache.DefaultNamespaces[configSpec.CacheNamespace] = cache.Config{}
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

controller runtime now supports to have a map of allowed namespaces,
to support that feature we must add new configurations with the more granular configurations,
may we add an issue to address that?

@Anhui-tqhuang
Copy link
Copy Markdown

@MarcosDY @azdagron hey when could this PR gets merged ...

Marcos Yacob added 2 commits December 20, 2023 11:36
Signed-off-by: Marcos Yacob <marcos.yacob@hpe.com>
Signed-off-by: Marcos Yacob <marcos.yacob@hpe.com>
@MarcosDY MarcosDY force-pushed the upgrade-controller-runtime branch from feb9f0f to 8213bf9 Compare December 20, 2023 14:36
@MarcosDY MarcosDY merged commit a655a08 into spiffe:main Dec 20, 2023
@MarcosDY MarcosDY deleted the upgrade-controller-runtime branch December 20, 2023 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants