-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Add configuration flag for enable_binary_authorization
#141
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
Comments
+1 |
Fixed by #160 |
We only added this feature for private clusters. Public clusters love binary auth too. See #186 |
This needs to be enabled on the |
it was already out in as non beta, so if we do that it will be a breaking change. So let me know if you want me to put it in as a breaking change. |
No, it's only on the private cluster module (which uses the beta provider). We can add it to the public beta module as well, but we don't need to add it to the main module. |
I misunderstood. Terraform provider. Now the question is do we put this feature only into our beta modules, and remove it from the private template |
Hello is this possible, I search and found that there is an option to use UBUNTU but have not seen a way to use CentOS
Remo
|
@itlinux I think you might be looking in the wrong place. This issue is about adding a |
https://www.terraform.io/docs/providers/google/r/container_cluster.html#enable_binary_authorization
Suggest plumbing the flag for it with the default as false. It allows for enabling the BinAuthZ Admission controller for being able to set a whitelist policy for approved container registry paths and also enforce image signing if desired. Note that can safely be set to be true if desired as the GCP project's default BinAuthZ is allow all/permissive.
The text was updated successfully, but these errors were encountered: