Skip to content

Add aws_route53_resolver_dnssec_config resource - #1942

Merged
turkenf merged 1 commit into
crossplane-contrib:mainfrom
ey22:add-route53resolver-resource
Jan 20, 2026
Merged

Add aws_route53_resolver_dnssec_config resource#1942
turkenf merged 1 commit into
crossplane-contrib:mainfrom
ey22:add-route53resolver-resource

Conversation

@ey22

@ey22 ey22 commented Jan 4, 2026

Copy link
Copy Markdown
Contributor

Description of your changes

Fixes #1898

This PR adds aws_route53_resolver_dnssec_config resource.

I have:

  • Read and followed Crossplane's contribution process.
  • Run make generate and committed the results (ideally in a separate commit).
  • Not made any manual changes to generated files, and verified this with make check-diff.

How has this code been tested

Manual Testing

I tested this locally using LocalStack configuration. Since this resource wasn't supported in Moto/LocalStack yet, I implemented the mocks locally first (PR) to make it work.

Screenshot 2026-01-03 at 18 13 57

Signed-off-by: Eylül Geyik <eylulgyk@gmail.com>
@jeanduplessis

Copy link
Copy Markdown
Collaborator

/test-examples="examples/route53resolver/cluster/v1beta1/dnssecconfig.yaml"

@ey22

ey22 commented Jan 20, 2026

Copy link
Copy Markdown
Contributor Author

Hi @jeanduplessis, thanks for running the uptest. Since the test was successful, I wanted to reach out to see if we can proceed to the next steps, if it's appropriate. Thanks!

@jeanduplessis jeanduplessis left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @ey22, this LGTM.

@turkenf turkenf left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @ey22, LGTM.

@turkenf
turkenf merged commit e74e922 into crossplane-contrib:main Jan 20, 2026
9 checks passed
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.

Request for route53_resolver_dnssec_config resource

3 participants