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

Erroneous certificate validation error #6

Open
cnweaver opened this issue Feb 6, 2024 · 1 comment
Open

Erroneous certificate validation error #6

cnweaver opened this issue Feb 6, 2024 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@cnweaver
Copy link
Contributor

cnweaver commented Feb 6, 2024

Terraform is producing errors about certificate validation records which look like:

│ Error: 1 error occurred:
│ 	* missing login.scimma.org DNS validation record: _33408912c503b310e6cf65fded7bff5b.login.scimma.org
│   with module.keycloak.aws_acm_certificate_validation.validation,
│   on .terraform/modules/keycloak/main.tf line 133, in resource "aws_acm_certificate_validation" "validation":
│  133: resource "aws_acm_certificate_validation" "validation" {

even when the record in question exists (and in fact the certificate has already been issued):

$ dig _33408912c503b310e6cf65fded7bff5b.login.scimma.org
_33408912c503b310e6cf65fded7bff5b.login.scimma.org. 3292 IN CNAME _6c67972ded5b4fe51e07595bfdcc29df.gfjzbhpkrq.acm-validations.aws.

This noise makes it hard to use terraform and we should find a way to clean it up.

@cnweaver cnweaver added the bug Something isn't working label Feb 6, 2024
@cmccully cmccully assigned cmccully and rdt12 and unassigned cmccully Feb 15, 2024
@cnweaver
Copy link
Contributor Author

cnweaver commented Mar 1, 2024

This problem has a couple of parts: The first is that the cert_validation aws_route53_record is being created for each domain name to be validated but using the same zone_id for all, which will not work when the records need to belong to different zones. This problem should be fixed in this code. Doing so is necessary, but not sufficient to solve the overall problem, however, because in the case where this is coming up, the second zone needed is scimma.org, which is not managed through Route53/terraform.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants