Comments (6)
@moshloop thanks for reporting this issue! The webhook configuration deployed by the chart has no functionality at first and will be filled by kiosk during runtime (the caCert will be inserted etc.). Scope of the webhook is limited to the kiosk group config.kiosk.sh and only namespaces that are owned by an account (for correct account quota functionality). I don't think we can narrow down the scope any further for the webhook. Could you elaborate a little bit more about why this should be changed?
from kiosk.
So there are 2 parts to this issue:
-
It should be static and defined up-front rather than dynamically at runtime, this provides 2 benefits:
a) Kiosk will not require permissions to create or modify webhooks
b) The scope of the webhook and any subsequent change is easily reviewable. -
The scope of the webhook itself should be limited, e.g CREATE might be needed on all resources, but Update is only needed for resources that specific limits/requests - This saves unnecessary webhook calls
from kiosk.
@moshloop thanks for clarifying, regarding your points:
-
the main issue is the caCert that needs to be inserted dynamically for the webhook unless you use another solution like cert-manager. So someone has to modify the webhook configuration after its creation and personally I prefer giving the rights to update the webhook to kiosk than to require an additional cluster component. Regarding b), we could think about moving the actual configuration from the code to the yaml, if people want to change the validatingwebhookconfiguration. Currently we assumed that it is better to ensure the configuration is how we need it for kiosk than making it more flexible, but I'm open on rethinking this
-
thats true, we probably can get rid of some UPDATE validations. Would you be willing to create a pull request for this issue?
from kiosk.
We had some issue in production ... the /quota endpoint failed for some reasons and blocked all the applicative deployments.
The failure policy is set to fail and it cause a lot of trouble. We do not want to use account quota and we want to remove this path in the ValidatingWebhookConfiguration. But we canโt because itโs dynamic.
And if a node reboot for patching (we are using AKS) and that some applicative pod start before the kiosk endpoint, the deployment stay in a bad state ...
We have a lot of issue too with call to the webhook failed because of inconsistent rootCA which lead to ssl errors and failed call (and failed deployment). We prefer to use our own certificate and rootCA like all of ours custom admission controllers.
Can you add an option to use our own ValidatingWebhookConfiguration and rootCA/Certificate ?
from kiosk.
@deuch thanks for the information! I'll add an environment variable to disable the dynamic provisioning of the webhook (it is called UPDATE_WEBHOOK), then you are free to deploy your own ValidatingWebhookConfiguration. I'll also do this for the ApiService that currently follows the same pattern. Regarding the RootCA/Certificate, this is already possible, you just have to mount the files /tmp/k8s-webhook-server/serving-certs/tls.crt, /tmp/k8s-webhook-server/serving-certs/tls.key and /tmp/k8s-webhook-server/serving-certs/ca.crt with your custom certs in the kiosk pod
from kiosk.
@deuch this can be now configured with kiosk v0.1.21
from kiosk.
Related Issues (20)
- Is redundant code?
- Account: Allow to apply custom label/annotation on namespace HOT 2
- kubens support HOT 5
- Access problem need help HOT 2
- Kiosk server port is not configurable HOT 1
- Delete Account doesn't delete the underline spaces HOT 2
- unable to retrieve the complete list of server APIs: tenancy.kiosk.sh/v1alpha1 HOT 2
- Kiosk restarted several times HOT 3
- Spaces can not properly be created by ArgoCD, resources show as OutOfSync HOT 6
- Hey is it possible/ how to use kiosk with saml2-based ADFS ? HOT 1
- Account spaceTemplate is not reconciled HOT 3
- Unsupport kubernetes server-side apply HOT 1
- Changing label in namespace doesn't update rolebinding
- Question: How to setup in existing cluster - Namespace migration HOT 3
- accountquotas resources cannot be edited after upgrade k8s to 1.22 HOT 2
- Unable to create template instance with parameters HOT 1
- Can map exists namespace in k8s cluster to new space? HOT 1
- can kiosk deploy's Replicas more than one? HOT 4
- Is Kiosk still maintained HOT 2
- Kiosk: Documentation for predefined parameters HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
๐ Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. ๐๐๐
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google โค๏ธ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from kiosk.