Project Team not updating

[barroca]

Hello, when I run a plan to change the team for a project, i see the changes like this:

sentry_project.serviceA team: "teamA" => "teamB"

But when applying they don't affect Sentry (I cannot see the change on their side).

I have a guess that we are missing the team information on the code here:

terraform-provider-sentry/sentry/resource_sentry_project.go

Line 142 in b9d9128

func resourceSentryProjectUpdate(d *schema.ResourceData, meta interface{}) error {

[brandonweng-db]

Also seeing the same thing for our use case.

Would this just be adding this in resourceSentryProjectUpdate?

	platform := d.Get("team").(string)
	if platform != "" {
		params.Team = team
	}

[taiidani]

Running into the same problem on my side. The apply will no-op and the drift will persist into the next plan.

Slight iteration on @brandonweng-db

team := d.Get("team").(string)
if team != "" {
    params.Team = team
}

[bilalahmad99]

facing same issue

[semangard]

same issue

[jianyuan]

It seems that the team name has been deprecated. Sentry just needs a unique slug, which is reflected in the web UI. I will look into this.

[everops-billm]

I'm having this issue too. It would also be helpful to be able to add additional teams to a project using the project provider, like you can from the web UI, or conversely add projects to the teams resources, also like you can do in the Web UI.

[jebeaudet]

Any update on this? Thanks

[mogagnon]

The problem comes from the way the API handles the addition of a team. Updating a team with the same call the provider does is deprecated.

The slug of new team for the project. Note, will be deprecated soon when multiple teams can have access to a project.

As far as I understand, the new way of managing a team for a project is with those two calls.

POST /projects/org_slug/project_slug/teams/teams_slug
DELETE /projects/org_slug/project_slug/teams/teams_slug

I have a fix on the way but as @everops-billm mentioned it would be nice to support multiple teams

[mogagnon]

Follows up for those that are interested. There are two pr to fix this bug, one is in sentry-go and the other one which is still a draft is here.

[jianyuan]

Thanks for your draft PR, @mogagnon! I've adapted your changes to #172.