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

Cert Manager Currently Using deprecated option #420

Open
ansasi opened this issue Oct 8, 2024 · 1 comment · May be fixed by #435
Open

Cert Manager Currently Using deprecated option #420

ansasi opened this issue Oct 8, 2024 · 1 comment · May be fixed by #435
Labels
enhancement New feature or request

Comments

@ansasi
Copy link

ansasi commented Oct 8, 2024

Cert_Manager is currently using installCRDs by default, which is a deprecated option of the helm chart of cert_manager.
This should be updated to use crds.enabled = true + crds.keep = true.

These are the lines of coded that should be reviewed:

set = concat([
{
name = "installCRDs"
value = true
},
{
name = "serviceAccount.name"
value = local.cert_manager_service_account
}
],
try(var.cert_manager.set, [])
)
set_sensitive = try(var.cert_manager.set_sensitive, [])

@askulkarni2 askulkarni2 added the enhancement New feature or request label Oct 8, 2024
@ansasi
Copy link
Author

ansasi commented Nov 6, 2024

Hello,

I attach the official documentation on the Helm Chart of cert-manager.

As showed installCRDs is deprecated and not used anymore. In exchange now crds.enabled and crds.keep are set to true.

image

Have a nice day.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
2 participants