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] IRSA docs added #4682

Merged
merged 8 commits into from
Jan 13, 2021

Conversation

AntonAleksandrov13
Copy link
Contributor

@AntonAleksandrov13 AntonAleksandrov13 commented Dec 10, 2020

Description of the change
Docs update for users who are deploying to AWS EKS and use IRSA
Benefits
This information is only available here
Possible drawbacks

Applicable issues

  • eliminates confusion for AWS EKS deployments with IRSA

Additional information

Checklist

  • Chart version bumped in Chart.yaml according to semver.
  • Variables are documented in the README.md
  • Title of the PR starts with chart name (e.g. [bitnami/chart])
  • If the chart contains a values-production.yaml apart from values.yaml, ensure that you implement the changes in both files

⚠️ Keep in mind that if you want to make changes to the kubeapps chart, please implement them in the kubeapps repository. This is only a synchronized mirror.

Copy link
Contributor

@dani8art dani8art left a comment

Choose a reason for hiding this comment

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

Hi @AntonAleksandrov13 thank you for the PR, please, take a look at my comments.

bitnami/external-dns/README.md Show resolved Hide resolved
bitnami/external-dns/README.md Outdated Show resolved Hide resolved
bitnami/external-dns/README.md Outdated Show resolved Hide resolved
@stale
Copy link

stale bot commented Jan 10, 2021

This Pull Request has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thank you for your contribution.

@stale stale bot added the stale 15 days without activity label Jan 10, 2021
@dani8art
Copy link
Contributor

Hi @AntonAleksandrov13 did you have a chance to work on this?

@stale stale bot removed the stale 15 days without activity label Jan 11, 2021
@AntonAleksandrov13
Copy link
Contributor Author

sorry, totally forgot about it. I will do that

@AntonAleksandrov13
Copy link
Contributor Author

Please check the PR

Copy link
Contributor

@dani8art dani8art left a comment

Choose a reason for hiding this comment

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

LGTM!

@dani8art dani8art merged commit 28e6e6f into bitnami:master Jan 13, 2021
Cellebyte pushed a commit to Cellebyte/charts that referenced this pull request Feb 12, 2021
* IRSA docs added

* Version bump

* Added example arguments

* Updated version

* Removed new line

* Added new line in yaml file + removed line in docs

Co-authored-by: Anton Aleksandrov <[email protected]>
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