-
Notifications
You must be signed in to change notification settings - Fork 56
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
chore: bump to GK 3.13 #1019
chore: bump to GK 3.13 #1019
Conversation
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #1019 +/- ##
=======================================
Coverage 58.34% 58.34%
=======================================
Files 93 93
Lines 5541 5541
=======================================
Hits 3233 3233
Misses 1992 1992
Partials 316 316 ☔ View full report in Codecov by Sentry. |
Tests failing likely due to new GK ED response cache which has a ttl of 3 min. Working with GK team to see how we can turn off or reduce ttl via helm chart |
TODO items to discuss:
|
As discussed in community meeting on 8/30/23:
|
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.
Looks good to me, left a minor comment
@@ -28,7 +28,7 @@ SLEEP_TIME=1 | |||
assert_success | |||
run bash -c "kubectl logs -l app.kubernetes.io/name=ratify -c ratify --tail=-1 -n gatekeeper-system | grep 'cache hit for subject descriptor'" | |||
assert_success | |||
|
|||
sleep 3 |
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.
suggestion to add a comment here for 3min gatekeeper cache TTL
Should we return an "error" instead of a verifier report then? |
Description
What this PR does / why we need it:
Bumps testing and scripts to use GK 3.13 as default.
Removes
enableExternalData
flag from GK installation where GK version is specified to be >= 3.11.0Which issue(s) this PR fixes (optional, using
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when the PR gets merged):Fixes #995
Type of change
Please delete options that are not relevant.
main
branch)How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration
Checklist:
Post Merge Requirements
Helm Chart Change