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

Create dependency between cluster issuers and cert-manager #13

Merged
merged 1 commit into from
Feb 10, 2024

Conversation

connec
Copy link
Member

@connec connec commented Feb 10, 2024

  • 522317c fix: create dependency between cluster issuers and cert-manager

    The cluster issuer resources depend on a customer resource definition
    supplied by the cert-manager chart, but they did not previously depend
    on that chart being created first.

    Unfortunately, that alone doesn't allow deployment to happen since the
    kubernetes_manifest resource type attempts to validate the resource
    type at plan time, when the CRD is not yet installed. This could only
    be fixed by moving the deployment of resources into a completely
    independent deployment (even a submodule doesn't help).

    The kubectl provider also provides a "manifest" resource type, which
    does not very the resource types at plan time. Using this instead allows
    planning to complete.

The cluster issuer resources depend on a customer resource definition
supplied by the cert-manager chart, but they did not previously depend
on that chart being created first.

Unfortunately, that alone doesn't allow deployment to happen since the
`kubernetes_manifest` resource type attempts to validate the resource
type at plan time, when the CRD is not yet installed. This could only
be fixed by moving the deployment of resources into a completely
independent deployment (even a submodule doesn't help).

The `kubectl` provider also provides a "manifest" resource type, which
does not very the resource types at plan time. Using this instead allows
planning to complete.
@ekadas ekadas merged commit 56bcbcc into main Feb 10, 2024
3 checks passed
@ekadas ekadas deleted the fix-issuers branch February 10, 2024 19:03
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

Successfully merging this pull request may close these issues.

2 participants