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.

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

[Question] Inconsistency regarding the count of Env Injector installed in the cluster #98

Open
lshi18 opened this issue Mar 23, 2023 · 1 comment

Comments

@lshi18
Copy link

lshi18 commented Mar 23, 2023

I'd like to ask a question concerning the inconsistency regarding the count of Env Injector installed in the cluster. Why is it that in the Requiement, it says

Note: You should at most have only one Env Injector installment in your cluster, because multiple instances of the Env Injector mutating webhook might compete and fail when a new pod (that needs environment injection) is created.

but at the same time, the default value for the env_injector.replicaCount in the chart is set to 2?

Using the default value of 2 seems to have led us into this issue.

Thanks

@lshi18 lshi18 changed the title Question about the count of Env Injector installed in the cluster [Question] Inconsistency regarding the count of Env Injector installed in the cluster Mar 23, 2023
@tspearconquest
Copy link
Contributor

The replicas fall behind the same Service resource because replicas are part of a Deployment. The documentation is advising not to have multiple Deployments of AKV2K8S env-injector (one might do this to have a separation of privileges by deploying different env-injectors to different namespaces, much the way Gitlab advises to deploy their Gitlab Agent, however this setup is not one that can be supported because the cluster can have at most one AKV2K8S MutatingWebhookConfiguration resource.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants