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

fix: Add tags to IRSA created roles #1250

Merged
merged 5 commits into from
Dec 11, 2022
Merged

Conversation

vishalsaugat
Copy link
Contributor

@vishalsaugat vishalsaugat commented Dec 11, 2022

What does this PR do?

Fix issue when tags are not being applied to IRSA roles created for ebs-csi drivers. See issue 1249 for more info

🛑 Please open an issue first to discuss any significant work and flesh out details/direction - we would hate for your time to be wasted.
Consult the CONTRIBUTING guide for submitting pull-requests.

Motivation

More

  • Yes, I have tested the PR using my local account setup (Provide any test evidence report under Additional Notes)
  • Yes, I have added a new example under examples to support my PR
  • Yes, I have created another PR for add-ons under add-ons repo (if applicable)
  • Yes, I have updated the docs for this feature
  • Yes, I ran pre-commit run -a with this PR

Note: Not all the PRs require a new example and/or doc page. In general:

  • Use an existing example when possible to demonstrate a new addons usage
  • A new docs page under docs/add-ons/* is required for new a new addon

For Moderators

  • E2E Test successfully complete before merge?

Additional Notes

vishalsaugat and others added 5 commits November 30, 2022 01:17
tags are not being applied policy created for cert manager irsa even when I pass the parameters. 
See aws-ia#1216 for more info
@vishalsaugat vishalsaugat requested a review from a team as a code owner December 11, 2022 21:41
@vishalsaugat vishalsaugat temporarily deployed to EKS Blueprints Test December 11, 2022 21:41 — with GitHub Actions Inactive
@vishalsaugat vishalsaugat changed the title fix: 1249 Tags not appearing for irsa roles fix: Tags not appearing for irsa roles #1249 Dec 11, 2022
@bryantbiggs bryantbiggs changed the title fix: Tags not appearing for irsa roles #1249 fix: Add tags to IRSA created roles #1249 Dec 11, 2022
@bryantbiggs bryantbiggs changed the title fix: Add tags to IRSA created roles #1249 fix: Add tags to IRSA created roles Dec 11, 2022
@bryantbiggs bryantbiggs merged commit 209be35 into aws-ia:main Dec 11, 2022
allamand pushed a commit to allamand/terraform-aws-eks-blueprints that referenced this pull request Dec 15, 2022
allamand pushed a commit to allamand/terraform-aws-eks-blueprints that referenced this pull request Jan 10, 2023
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.

Tags not appearing for irsa roles
2 participants