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

[bitnami/external-dns] Add aws.zoneMatchParent option #25575

Merged

Conversation

clemensg
Copy link
Contributor

@clemensg clemensg commented May 7, 2024

Description of the change

Add support for the aws.zoneMatchParent option (defaulting to false) and add corresponding documentation.

Benefits

Users can set aws.zoneMatchParent to true and set a domain filter for subdomains, e.g. a.b.example.com. This was not possible before with external-dns and Route53.
See kubernetes-sigs/external-dns#4236

Possible drawbacks

None

Applicable issues

None

Additional information

N/A

Checklist

  • Chart version bumped in Chart.yaml according to semver. This is not necessary when the changes only affect README.md files.
  • Variables are documented in the values.yaml and added to the README.md using readme-generator-for-helm
  • Title of the pull request follows this pattern [bitnami/<name_of_the_chart>] Descriptive title
  • All commits signed off and in agreement of Developer Certificate of Origin (DCO)

@github-actions github-actions bot added external-dns triage Triage is needed labels May 7, 2024
@github-actions github-actions bot requested a review from carrodher May 7, 2024 12:22
@carrodher carrodher added verify Execute verification workflow for these changes in-progress labels May 7, 2024
@github-actions github-actions bot removed the triage Triage is needed label May 7, 2024
@github-actions github-actions bot removed the request for review from carrodher May 7, 2024 13:06
@github-actions github-actions bot requested a review from dgomezleon May 7, 2024 13:06
Copy link
Member

@dgomezleon dgomezleon left a comment

Choose a reason for hiding this comment

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

Thanks for contributing. Just a minor change regarding the version.

@@ -28,4 +28,4 @@ maintainers:
name: external-dns
sources:
- https://github.com/bitnami/charts/tree/main/bitnami/external-dns
version: 7.2.1
version: 7.2.2
Copy link
Member

Choose a reason for hiding this comment

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

Please bump the minor version to add new values.

Suggested change
version: 7.2.2
version: 7.3.0

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will do

Signed-off-by: Clemens Gruber <clemensgru@gmail.com>
@clemensg clemensg force-pushed the external_dns_aws_zone_match_parent branch from 9681e79 to 75c211c Compare May 7, 2024 14:25
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
dgomezleon
dgomezleon previously approved these changes May 8, 2024
Copy link
Member

@dgomezleon dgomezleon left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: David Gomez <dgomezleon@vmware.com>
@dgomezleon dgomezleon merged commit e337e96 into bitnami:main May 8, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
external-dns solved verify Execute verification workflow for these changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants