Addressing pagerduty_slack_connection unable to set "No Priority" vs "Any Priority" for priorities configuration #519
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Addressing #456
After validating the behaviour of PagerDuty's API in regards of configuring the priorities for Slack Connection resource and comparing it with the current capabilities of our TF Provider, I could make sure that there ware no way to configure a
No Priority
priorities setting sticking to the interface of the PagerDuty's API. So it was necessary to add a more explicit way to achieve this configuration. As a result We end up making a little upgrade to the priorities attribute interface:priorities
- (Optional) Allows you to filter events by priority. Needs to be an array of PagerDuty priority IDs. Available through pagerduty_priority data source.[]
) in the configuration for a Slack Connection, its default behaviour is to setpriorities
toNo Priority
value.["*"]
its corresponding value forpriorities
in Slack Connection's configuration will beAny Priority
.Test results: