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

Issue#437 - Fix team_id update issue #438

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

Conversation

SharmaRakeshKumar
Copy link

Issue#437 - Fix team_id update issue

Issue raised - #437
Support ticket raised - https://support.atlassian.com/requests/PCS-181010/

Problem statement:
When we try to update "Owner team" i.e; team_id via opsgenie_service resource then team_id (Owner team) is not updated. Service name, description, tags are updated correctly during update, but team_id fails to get updated.
Same happens when we try to update the team manually. i.e when we remove a team manually from a service via UI and immediately add another service, then the service doesn't gets updated. To update the new "Owner team" manually we need to wait for some time after removing the previous Owner and then attach the new Owner.

Quick fix:
Adding delete & creation on service in the update, to delete & create the required service back, as team_id update is not possible as explained in the problem statement.

opsgenie#437

To achieve the team_id update, applying a quick solution - Deleting & creating service again, as there is no issue with service creation or deletion. Hence to apply a quick fix to the problem, we can delete and create the service again.
@tim-blacklane
Copy link

I’d suggest to add

ForceNew: true,

between line 32 and line 33 https://github.com/opsgenie/terraform-provider-opsgenie/blob/master/opsgenie/resource_opsgenie_service.go#L32-L33
This will already achieve what you are trying here while also just update the remaining attributes. Also it might help get this change approved and merged.

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.

2 participants