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

Serverless app repo #5961

Merged
merged 25 commits into from
Nov 25, 2020
Merged

Serverless app repo #5961

merged 25 commits into from
Nov 25, 2020

Conversation

gdavison
Copy link
Contributor

@gdavison gdavison commented Sep 22, 2018

Adds AWS Serverless Application Repository applications to the Terraform provider.

Changes proposed in this pull request:

  • Adds the resource aws_serverlessrepository_stack
    • Can specify the version
  • Adds the data source aws_serverlessrepository_application
    • I'm not sure this is all that useful

Fixes #3981

Needs:

  • tags
  • documentation
  • import functionality

Output from acceptance testing:

make testacc TEST=./aws TESTARGS='-run=ServerlessRepo'
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./aws -v -parallel 20 -run=ServerlessRepo -timeout 120m
=== RUN   TestAccDataSourceAwsServerlessRepositoryApplication_Basic
--- PASS: TestAccDataSourceAwsServerlessRepositoryApplication_Basic (13.93s)
=== RUN   TestAccDataSourceAwsServerlessRepositoryApplication_Versioned
--- PASS: TestAccDataSourceAwsServerlessRepositoryApplication_Versioned (10.72s)
=== RUN   TestAccAwsServerlessRepositoryStack_basic
--- PASS: TestAccAwsServerlessRepositoryStack_basic (69.76s)
=== RUN   TestAccAwsServerlessRepositoryStack_versioned
--- PASS: TestAccAwsServerlessRepositoryStack_versioned (57.98s)
=== RUN   TestAccAwsServerlessRepositoryStack_tagged
--- PASS: TestAccAwsServerlessRepositoryStack_tagged (60.22s)
=== RUN   TestAccAwsServerlessRepositoryStack_versionUpdate
--- PASS: TestAccAwsServerlessRepositoryStack_versionUpdate (113.40s)
=== RUN   TestAccAwsServerlessRepositoryStack_update
--- PASS: TestAccAwsServerlessRepositoryStack_update (98.15s)
PASS
ok  	github.com/terraform-providers/terraform-provider-aws/aws	424.206s

@ghost ghost added size/XL Managed by automation to categorize the size of a PR. documentation Introduces or discusses updates to documentation. tests PRs: expanded test coverage. Issues: expanded coverage, enhancements to test infrastructure. labels Sep 22, 2018
@bflad bflad added the service/serverlessrepo Issues and PRs that pertain to the serverlessrepo service. label Sep 24, 2018
@bflad
Copy link
Contributor

bflad commented Oct 10, 2018

Hi @gdavison is this still work in progress? Need any help?

@bflad bflad added the waiting-response Maintainers are waiting on response from community or contributor. label Oct 10, 2018
@gdavison
Copy link
Contributor Author

Hi @bflad. I'm still working on it slowly. I haven't had a chance to do much work on it recently.

@ghost ghost added size/XXL Managed by automation to categorize the size of a PR. service/apigateway Issues and PRs that pertain to the apigateway service. service/cloudformation Issues and PRs that pertain to the cloudformation service. service/cloudfront Issues and PRs that pertain to the cloudfront service. service/codedeploy service/ec2 Issues and PRs that pertain to the ec2 service. service/elasticache Issues and PRs that pertain to the elasticache service. service/emr Issues and PRs that pertain to the emr service. service/firehose Issues and PRs that pertain to the firehose service. service/gamelift Issues and PRs that pertain to the gamelift service. service/opsworks Issues and PRs that pertain to the opsworks service. service/rds Issues and PRs that pertain to the rds service. service/redshift Issues and PRs that pertain to the redshift service. service/route53 Issues and PRs that pertain to the route53 service. service/s3 Issues and PRs that pertain to the s3 service. and removed size/XL Managed by automation to categorize the size of a PR. labels Oct 17, 2018
@bflad bflad removed service/apigateway Issues and PRs that pertain to the apigateway service. service/cloudformation Issues and PRs that pertain to the cloudformation service. service/cloudfront Issues and PRs that pertain to the cloudfront service. service/codedeploy service/ec2 Issues and PRs that pertain to the ec2 service. labels Oct 18, 2018
@gdavison gdavison force-pushed the serverless_app_repo branch from df6cfa9 to 753fa55 Compare September 5, 2019 20:26
Graham Davison added 3 commits September 5, 2019 15:35
This confirms that no tags show up in the state when no tags are added to the resource
@ghost ghost added size/XXL Managed by automation to categorize the size of a PR. and removed size/XL Managed by automation to categorize the size of a PR. labels Sep 10, 2019
@gdavison gdavison changed the title [WIP] Serverless app repo Serverless app repo Sep 10, 2019
@gdavison
Copy link
Contributor Author

@bflad I've dropped the "[WIP]" :)

@ghost ghost removed the waiting-response Maintainers are waiting on response from community or contributor. label Sep 10, 2019
@gdavison gdavison changed the title Serverless app repo [WIP] Serverless app repo Sep 12, 2019
@gdavison
Copy link
Contributor Author

I've put the "[WIP]" label back on. It needed import functionality, and that's causing a number of changes under the hood.

@gdavison
Copy link
Contributor Author

I ran an informal survey at #3981 (comment), and it's 7-0 for releasing now and backfilling import later.

So I'm removing the [WIP] again, @bflad

@gdavison gdavison changed the title [WIP] Serverless app repo Serverless app repo Sep 18, 2019
@kitsuneninetails
Copy link

Is this in limbo, or is someone going to push this forward? It's been almost 3 months since the last update...

@dmccaffery
Copy link

... can we move this forward, please :)

@dmccaffery
Copy link

@gdavison ... I'm going to whip out a can of whoop pretty soon 💯

I am super happy to help get this over the finish line with code, beer, wine, hugs, whatever is required.

@weslambert
Copy link

This is something I would like as well! Thanks!

@chenrui333
Copy link

Any update on this?? Love this to be merged soon :)

@gdavison
Copy link
Contributor Author

gdavison commented Jun 5, 2020

Finishing this is needed to complete the acceptance test TestAccAwsSecretsManagerSecretRotation() in #9487

@pacecreative
Copy link

+1 For this. :)

@gdavison gdavison merged commit 18132b1 into hashicorp:master Nov 25, 2020
@ghost
Copy link

ghost commented Dec 26, 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 feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks!

@ghost ghost locked as resolved and limited conversation to collaborators Dec 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Introduces or discusses updates to documentation. provider Pertains to the provider itself, rather than any interaction with AWS. service/serverlessrepo Issues and PRs that pertain to the serverlessrepo service. size/XXL Managed by automation to categorize the size of a PR. tests PRs: expanded test coverage. Issues: expanded coverage, enhancements to test infrastructure.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support for AWS Serverless Application Model and Repository
7 participants