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

github_team_membership team_id not being imported #324

Closed
Jonnymcc opened this issue Jan 16, 2020 · 3 comments
Closed

github_team_membership team_id not being imported #324

Jonnymcc opened this issue Jan 16, 2020 · 3 comments

Comments

@Jonnymcc
Copy link

Terraform Version

Terraform v0.11.14

  • provider.github v2.2.1

Affected Resource(s)

  • github_team_membership

Terraform Configuration Files

resource "github_team" "Managers" {
  name        = "Managers"
  privacy     = "closed"
}

resource "github_team_membership" "Managers-Jonnymcc" {
  username = "Jonnymcc"
  team_id  = "${github_team.Managers.id}"
  role     = "member"
}

Debug Output

github_team_membership.Merge_Managers-Jonnymcc: Destroying... (ID: xxxxx:Jonnymcc)

Error: Error applying plan:

1 error occurred:
	* github_team_membership.Merge_Managers-Jonnymcc (destroy): 1 error occurred:
	* github_team_membership.Merge_Managers-Jonnymcc: Unexpected ID format (""), expected numerical ID. strconv.ParseInt: parsing "": invalid syntax

Expected Behavior

When running terraform show, the github_team_membership resource should have a team_id attribute value.

Actual Behavior

The team_id = "".

Steps to Reproduce

Please list the steps required to reproduce the issue, for example:

  1. terraform import github_team_membership.Managers-Jonnymcc xxxxx:Jonnymcc
  2. terraform apply -target github_team_membership.Managers-Jonnymcc

References

Maybe this bug fix is related...
resource/github_team_membership: Validation of team_id (#253)

@samrees
Copy link
Contributor

samrees commented Jan 16, 2020

@brandoncamenisch
Copy link

brandoncamenisch commented Jan 16, 2020

Same issue for me as well 😢 just popped up this morning.

Terraform v0.11.13

  • provider.github v2.2.1

@radeksimko
Copy link
Contributor

Thank you @samrees for noticing, you are right.
I'm going to close this as a duplicate of #323

I just merged a PR which fixes the bug and we will release it soon.

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

No branches or pull requests

5 participants