-
Notifications
You must be signed in to change notification settings - Fork 784
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
Feature - Add checks for k8s version when Kyverno starts #831
Conversation
@shravanshetty1 Can you also help to review this PR :) |
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.
All existing middlewares and future middlewares should be a part of the middleware package.
Also existing logic regarding webhookTimeout needs to be removed - and needs to be tested with the older version of kubernetes as stated in issue #771
Also mutate and validate is not allowed before kubernetes 1.14 - hence we should refactor the code to handle this and any future cases where based on version we dont want to support some features. |
…actore kubernetes version check
* 'fix-771' of github.com:evalsocket/kyverno: Removed log
Problem
Prior to Kubernetes 1.14, there is no option to tune
timeout
for webhookconfigurations. We need to add strict checks on the supported version of clusters:generate
feature. Mutate / Validate policy should be ignored in these versions of clusters.Solution
We are adding additional check for Kubernetes version and added timeout middleware
Other changes