Skip to content

Prepare for v1.10.0-alpha.2 release; bump dependencies#801

Merged
SgtCoDFish merged 2 commits intomasterfrom
v1.10.0-alpha.2
May 6, 2026
Merged

Prepare for v1.10.0-alpha.2 release; bump dependencies#801
SgtCoDFish merged 2 commits intomasterfrom
v1.10.0-alpha.2

Conversation

@FelixPhipps
Copy link
Copy Markdown
Contributor

Adds support for the per-gatherer excludeAnnotationKeysRegex and excludeLabelKeysRegex config fields, which previously had no effect. Resources with annotation or label keys matching the configured patterns are now excluded entirely from upload, rather than just having those keys stripped from their metadata.

felix.phipps added 2 commits May 5, 2026 16:59
Signed-off-by: felix.phipps <felix.phipps@cyberark.com>
Signed-off-by: felix.phipps <felix.phipps@cyberark.com>
Copy link
Copy Markdown
Contributor

@SgtCoDFish SgtCoDFish left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

Thank you!

Comment thread go.mod
Comment on lines -22 to -26
k8s.io/api v0.34.3
k8s.io/apimachinery v0.34.3
k8s.io/client-go v0.34.3
k8s.io/component-base v0.34.3
sigs.k8s.io/controller-runtime v0.22.4
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

note: I'm not fully confident that bumping these k8s deps will be totally safe given we'll want to also support older versions of k8s. We'll have to be careful to test this after we merge this PR, and check that the agent still works with the oldest k8s version we can find.

(I can't give a specific concern, but I think it's a focus point!)

Comment on lines +308 to +309
// Note: We used to test with smaller key sizes but the library started to reject such small keys.
// Therefore, we have removed the test for the time being.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nb: I suggested replacing this test with a comment. No point in testing that we reject small keys if the library does it for us.

@SgtCoDFish SgtCoDFish merged commit 3383004 into master May 6, 2026
5 checks passed
@SgtCoDFish SgtCoDFish deleted the v1.10.0-alpha.2 branch May 6, 2026 10:56
@SgtCoDFish SgtCoDFish changed the title v1.10.0-alpha.2 release Prepare for v1.10.0-alpha.2 release; bump dependencies May 6, 2026
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.

2 participants