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

feat: add sentry_organization_repository_azuredevops resource #293

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

dan-hammond
Copy link

Hi,

I've added a sentry_organization_repository_azuredevops resource, as I'm using this provider at work and need the ability to link our created projects to our Azure DevOps repositories. I would be very grateful if you could merge and release a new version with this addition.

I based the code on the existing sentry_organization_repository_github and have tested it locally against our environment, which seems to work.

The only issue I have spotted is that the resource seems to struggle with being destroyed for me, as it errors with the following:

Error: DELETE https://sentry.io/api/0/organizations/{org_name}/repos/{repo_internal_id}/: 403 You do not have permission to perform this action.

The API token I was testing with was granted full access, so I'm not sure why it doesn't have permission. If I delete the repo manually via the Sentry dashboard I can see that the API call is made with the exact URL that Terraform tries to use, so the provider code seems to be correct. I don't know whether you are able to advise on what that could be. Even if we're unable to resolve that issue, I don't think it's necessarily a blocker for getting this merged.

Thanks

@dan-hammond
Copy link
Author

@jianyuan sorry to @ you, but I've just started working with Terraform again and rediscovered that I'm currently missing this functionality. Do you mind having a look at this when you find a moment? Thanks

# Conflicts:
#	sentry/provider.go
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

Successfully merging this pull request may close these issues.

1 participant