Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix for NotIn operator in matchCapabilityChanges selector #78

Merged
merged 2 commits into from
May 25, 2022
Merged

Fix for NotIn operator in matchCapabilityChanges selector #78

merged 2 commits into from
May 25, 2022

Conversation

tpapagian
Copy link
Member

Add a test for NotIn operator in matchCapabilityChanges selector and fix an issue with that.

- matchCapabilityChanges: 
  - type: Effective
     operator: NotIn
     values:
     - "CAP_SYS_ADMIN"

This will match if there is a change in any capabilities other than CAP_SYS_ADMIN.

Signed-off-by: Anastasios Papagiannis tasos.papagiannnis@gmail.com

@tpapagian
Add a test for matchCapabilityChanges selector
092c130 
... with operator == NotIn

Signed-off-by: Anastasios Papagiannis <tasos.papagiannnis@gmail.com>
@tpapagian
Fix NotIn operator in matchCapabilityChanges selector
912d12c 
Signed-off-by: Anastasios Papagiannis <tasos.papagiannnis@gmail.com>
@tpapagian tpapagian requested a review from a team as a code owner May 25, 2022 11:43
@jrfastab jrfastab merged commit b296e0d into main May 25, 2022
@jrfastab jrfastab deleted the pr/apapag/match_capchanges_fix branch May 25, 2022 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jrfastab jrfastab jrfastab approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tpapagian @jrfastab