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

providers/heroku: use a helper for common app post-create tasks #13990

Merged
merged 1 commit into from
Apr 26, 2017

Conversation

danp
Copy link
Contributor

@danp danp commented Apr 26, 2017

Buildpacks were being configured post-create for non-org apps but not
for org apps. Consolidate post-create tasks into a helper used by both
paths.

Buildpacks were being configured post-create for non-org apps but not
for org apps. Consolidate post-create tasks into a helper used by both
paths.
Copy link
Contributor

@catsby catsby left a comment

Choose a reason for hiding this comment

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

LGTM!

[github.com/hashicorp/terraform][pr-13990](4)$ make testacc TEST=./builtin/providers/heroku TESTARGS="-run=TestAccHeroku"
==> Checking that code complies with gofmt requirements...
go generate $(go list ./... | grep -v /terraform/vendor/)
2017/04/26 17:00:33 Generated command/internal_plugin_list.go
TF_ACC=1 go test ./builtin/providers/heroku -v -run=TestAccHeroku -timeout 120m
=== RUN   TestAccHerokuAddon_Basic
--- PASS: TestAccHerokuAddon_Basic (4.31s)
=== RUN   TestAccHerokuAddon_noPlan
--- PASS: TestAccHerokuAddon_noPlan (7.39s)
=== RUN   TestAccHerokuApp_Basic
--- PASS: TestAccHerokuApp_Basic (2.78s)
=== RUN   TestAccHerokuApp_NameChange
--- PASS: TestAccHerokuApp_NameChange (5.26s)
=== RUN   TestAccHerokuApp_NukeVars
--- PASS: TestAccHerokuApp_NukeVars (6.16s)
=== RUN   TestAccHerokuApp_Buildpacks
--- PASS: TestAccHerokuApp_Buildpacks (7.14s)
=== RUN   TestAccHerokuApp_ExternallySetBuildpacks
--- PASS: TestAccHerokuApp_ExternallySetBuildpacks (4.47s)
=== RUN   TestAccHerokuApp_Organization
--- PASS: TestAccHerokuApp_Organization (3.50s)
=== RUN   TestAccHerokuApp_Space
--- SKIP: TestAccHerokuApp_Space (0.00s)
        resource_heroku_app_test.go:220: HEROKU_SPACE is not set; skipping test.
=== RUN   TestAccHerokuCert_Basic
--- PASS: TestAccHerokuCert_Basic (18.61s)
=== RUN   TestAccHerokuDomain_Basic
--- PASS: TestAccHerokuDomain_Basic (3.37s)
=== RUN   TestAccHerokuDrain_Basic
--- PASS: TestAccHerokuDrain_Basic (3.46s)
=== RUN   TestAccHerokuSpace_Basic
--- FAIL: TestAccHerokuSpace_Basic (0.17s)
        testing.go:280: Step 0 error: Error applying: 1 error(s) occurred:

                * heroku_space.foobar: 1 error(s) occurred:

                * heroku_space.foobar: Post https://api.heroku.com/spaces: Space management is not enabled for this team.
FAIL
exit status 1
FAIL    github.com/hashicorp/terraform/builtin/providers/heroku 66.649s

I don't think the last failure is related 😄

@catsby catsby merged commit cf6e86a into hashicorp:master Apr 26, 2017
@catsby
Copy link
Contributor

catsby commented Apr 26, 2017

Also, "hi Dan 👋"

@danp danp deleted the heroku-buildpacks-create branch May 31, 2017 18:22
vanstee pushed a commit to vanstee/terraform that referenced this pull request Sep 28, 2017
…icorp#13990)

Buildpacks were being configured post-create for non-org apps but not
for org apps. Consolidate post-create tasks into a helper used by both
paths.
@ghost
Copy link

ghost commented Apr 11, 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 11, 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.

2 participants