-
Notifications
You must be signed in to change notification settings - Fork 51
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
Add ToMatcher to TargetHandler #178
Conversation
Codecov Report
@@ Coverage Diff @@
## master #178 +/- ##
==========================================
+ Coverage 44.92% 44.94% +0.02%
==========================================
Files 59 59
Lines 2776 2779 +3
==========================================
+ Hits 1247 1249 +2
- Misses 1286 1287 +1
Partials 243 243
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Failure from Gatekeeper Test is expected as this changes the TargetHandler interface. |
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.
LGTM with 1 nit
The idea here is that we can cache the golang match criteria for each Constraint, rather than parsing it out each time. Also fix bug in SemanticEqual where one Constraint has Object defined and the other does not. Signed-off-by: Will Beason <willbeason@google.com>
Signed-off-by: Will Beason <willbeason@google.com>
Signed-off-by: Will Beason <willbeason@google.com>
9c2d097
to
b3bed95
Compare
The idea here is that we can cache the golang match criteria for each
Constraint, rather than parsing it out each time. This PR does not actually implement the behavior; it just defines the interface so that we can start using it in Gatekeeper.
Also fix bug in SemanticEqual where one Constraint has Object defined
and the other does not.
Signed-off-by: Will Beason willbeason@google.com