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

Add name_prefix to aws_iam_policy #10178

Merged
merged 1 commit into from
Nov 17, 2016
Merged

Add name_prefix to aws_iam_policy #10178

merged 1 commit into from
Nov 17, 2016

Conversation

2rs2ts
Copy link
Contributor

@2rs2ts 2rs2ts commented Nov 16, 2016

This is intended to behave the same way as name_prefix does in aws_iam_role.

Fixes #10176

Feedback on the interface change would be highly appreciated. I was able to get the acceptance tests to run successfully thanks to some help from a peer, so I feel pretty confident about this.

@2rs2ts 2rs2ts force-pushed the master branch 2 times, most recently from 45b34ff to c1f5b62 Compare November 16, 2016 23:58
This is intended to behave the same way as name_prefix does in
aws_iam_role.

Fixes hashicorp#10176
@2rs2ts 2rs2ts changed the title [WIP] Add name_prefix to aws_iam_policy Add name_prefix to aws_iam_policy Nov 17, 2016
@stack72
Copy link
Contributor

stack72 commented Nov 17, 2016

Hi @2rs2ts

This LGTM! Thanks for the work here:

% make testacc TEST=./builtin/providers/aws TESTARGS='-run=TestAWSPolicy_namePrefix'
==> Checking that code complies with gofmt requirements...
go generate $(go list ./... | grep -v /terraform/vendor/)
2016/11/17 14:00:36 Generated command/internal_plugin_list.go
TF_ACC=1 go test ./builtin/providers/aws -v -run=TestAWSPolicy_namePrefix -timeout 120m
=== RUN   TestAWSPolicy_namePrefix
--- PASS: TestAWSPolicy_namePrefix (15.32s)
PASS
ok      github.com/hashicorp/terraform/builtin/providers/aws    15.341s

@stack72 stack72 merged commit 3822c69 into hashicorp:master Nov 17, 2016
jrnt30 pushed a commit to jrnt30/terraform that referenced this pull request Nov 17, 2016
This is intended to behave the same way as name_prefix does in
aws_iam_role.

Fixes hashicorp#10176
gusmat pushed a commit to gusmat/terraform that referenced this pull request Dec 6, 2016
This is intended to behave the same way as name_prefix does in
aws_iam_role.

Fixes hashicorp#10176
fatmcgav pushed a commit to fatmcgav/terraform that referenced this pull request Feb 27, 2017
This is intended to behave the same way as name_prefix does in
aws_iam_role.

Fixes hashicorp#10176
@ghost
Copy link

ghost commented Apr 20, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Apr 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Please implement name_prefix for aws_iam_policy
2 participants