feat: add use_fullname input to control IAM role name generation#45
use_fullname input to control IAM role name generation#45Conversation
|
Important Title is necessary and should not be empty.Kindly provide a meaningful title for this Pull Request. |
Co-authored-by: nitrocode <7775707+nitrocode@users.noreply.github.com>
use_fullname input to control IAM role name generation
|
/terratest |
|
There are no real tests for this component. So we set terratest statuses to successful execution without running any tests |
|
Thanks @Copilot for creating this pull request! A maintainer will review your changes shortly. Please don't be discouraged if it takes a while. While you wait, make sure to review our contributor guidelines. Tip Need help or want to ask for a PR review to be expedited?Join us on Slack in the |
|
Heads up! This pull request looks stale. It will be closed soon, if there are no new commits. ⏳ |
|
Thanks @Copilot for creating this pull request! A maintainer will review your changes shortly. Please don't be discouraged if it takes a while. While you wait, make sure to review our contributor guidelines. Tip Need help or want to ask for a PR review to be expedited?Join us on Slack in the |
|
Heads up! This pull request looks stale. It will be closed soon, if there are no new commits. ⏳ |
|
Thanks @Copilot for creating this pull request! A maintainer will review your changes shortly. Please don't be discouraged if it takes a while. While you wait, make sure to review our contributor guidelines. Tip Need help or want to ask for a PR review to be expedited?Join us on Slack in the |
|
Thanks @Copilot for creating this pull request! A maintainer will review your changes shortly. Please don't be discouraged if it takes a while. While you wait, make sure to review our contributor guidelines. Tip Need help or want to ask for a PR review to be expedited?Join us on Slack in the |
|
Heads up! This pull request looks stale. It will be closed soon, if there are no new commits. ⏳ |
|
Thanks @Copilot for creating this pull request! A maintainer will review your changes shortly. Please don't be discouraged if it takes a while. While you wait, make sure to review our contributor guidelines. Tip Need help or want to ask for a PR review to be expedited?Join us on Slack in the |
|
Heads up! This pull request looks stale. It will be closed soon, if there are no new commits. ⏳ |
|
Thanks @Copilot for creating this pull request! A maintainer will review your changes shortly. Please don't be discouraged if it takes a while. While you wait, make sure to review our contributor guidelines. Tip Need help or want to ask for a PR review to be expedited?Join us on Slack in the |
|
Heads up! This pull request looks stale. It will be closed soon, if there are no new commits. ⏳ |
|
Thanks @Copilot for creating this pull request! A maintainer will review your changes shortly. Please don't be discouraged if it takes a while. While you wait, make sure to review our contributor guidelines. Tip Need help or want to ask for a PR review to be expedited?Join us on Slack in the |
|
Heads up! This pull request looks stale. It will be closed soon, if there are no new commits. ⏳ |
|
Thanks @Copilot for creating this pull request! A maintainer will review your changes shortly. Please don't be discouraged if it takes a while. While you wait, make sure to review our contributor guidelines. Tip Need help or want to ask for a PR review to be expedited?Join us on Slack in the |
|
Heads up! This pull request looks stale. It will be closed soon, if there are no new commits. ⏳ |
|
Thanks @Copilot for creating this pull request! A maintainer will review your changes shortly. Please don't be discouraged if it takes a while. While you wait, make sure to review our contributor guidelines. Tip Need help or want to ask for a PR review to be expedited?Join us on Slack in the |
|
Heads up! This pull request looks stale. It will be closed soon, if there are no new commits. ⏳ |
|
Thanks @Copilot for creating this pull request! A maintainer will review your changes shortly. Please don't be discouraged if it takes a while. While you wait, make sure to review our contributor guidelines. Tip Need help or want to ask for a PR review to be expedited?Join us on Slack in the |
|
Heads up! This pull request looks stale. It will be closed soon, if there are no new commits. ⏳ |
|
Thanks @Copilot for creating this pull request! A maintainer will review your changes shortly. Please don't be discouraged if it takes a while. While you wait, make sure to review our contributor guidelines. Tip Need help or want to ask for a PR review to be expedited?Join us on Slack in the |
The IAM role name was always derived from the full label context ID (e.g.
acme-core-gbl-auto-gha-iam), with no escape hatch to use a short, explicit name.Changes
src/variables.tf— Addsuse_fullname(bool, defaulttrue) mirroring the same input fromcloudposse/terraform-aws-iam-roleand following the same convention asbucket_nameincloudposse/terraform-aws-s3-bucket. Defaulttruepreserves existing behavior.src/main.tf— Role name now conditionally usesmodule.this.idormodule.this.name:README.yaml— Adds example and rationale for usinguse_fullname: falseto create a role by short name:🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.