Skip to content

fix(logs_alert): add Computed to filter_type and incident_settings fields#6716

Open
lorepanichi wants to merge 1 commit intoIBM-Cloud:masterfrom
lorepanichi:fix/logs-alert-computed-defaults
Open

fix(logs_alert): add Computed to filter_type and incident_settings fields#6716
lorepanichi wants to merge 1 commit intoIBM-Cloud:masterfrom
lorepanichi:fix/logs-alert-computed-defaults

Conversation

@lorepanichi
Copy link
Contributor

Community Note

  • Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for pull request followers and do not help prioritize the request

Output from acceptance testing:

$ make testacc TEST=./ibm/service/logs TESTARGS='-run=TestAccIbmLogsAlert'

Test failures are pre-existing and unrelated to this change:

summary: 'CreateAlertWithContext failed: ''notification_groups'' field cannot be set for ''new_value'' alerts'

The schema change (adding Computed: true to filter_type and incident_settings fields)
was validated manually via OpenTofu plan/apply — perpetual diff no longer occurs.

Summary

Add Computed: true to filter_type and incident_settings block (and some nested fields) in ibm_logs_alert.

These fields are populated by the API with defaults when not set by the user. Without Computed, Terraform treats API-returned values as drift and plans to remove them on every subsequent apply.

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