Skip to content

Allow deleting alert_grouping_parameters with a time_window equal to 86400 #1035

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

Merged
merged 1 commit into from
Jul 1, 2025

Conversation

cjgajard
Copy link
Member

@cjgajard cjgajard commented Jun 27, 2025

+go test ./pagerduty -run TestAccPagerDutyService_ -v
=== RUN   TestAccPagerDutyService_AlertGrouping
--- PASS: TestAccPagerDutyService_AlertGrouping (0.00s)
=== RUN   TestAccPagerDutyService_AlertGroupingContentBased
--- PASS: TestAccPagerDutyService_AlertGroupingContentBased (71.03s)
=== RUN   TestAccPagerDutyService_AlertContentGroupingIntelligentTimeWindow
--- PASS: TestAccPagerDutyService_AlertContentGroupingIntelligentTimeWindow (21.03s)
=== RUN   TestAccPagerDutyService_AlertGroupingParametersAddConfigField
--- PASS: TestAccPagerDutyService_AlertGroupingParametersAddConfigField (21.28s)
=== RUN   TestAccPagerDutyService_AutoPauseNotificationsParameters
--- PASS: TestAccPagerDutyService_AutoPauseNotificationsParameters (26.96s)
=== RUN   TestAccPagerDutyService_import
--- PASS: TestAccPagerDutyService_import (10.07s)
=== RUN   TestAccPagerDutyService_Basic
--- PASS: TestAccPagerDutyService_Basic (24.49s)
=== RUN   TestAccPagerDutyService_BasicWithIncidentUrgencyRules
--- PASS: TestAccPagerDutyService_BasicWithIncidentUrgencyRules (30.79s)
=== RUN   TestAccPagerDutyService_FromBasicToCustomIncidentUrgencyRules
--- PASS: TestAccPagerDutyService_FromBasicToCustomIncidentUrgencyRules (21.06s)
=== RUN   TestAccPagerDutyService_Delete24HAlertGrouping
--- PASS: TestAccPagerDutyService_Delete24HAlertGrouping (36.86s)
PASS

@cjgajard cjgajard force-pushed the cjavier/allow-delete-24h-content-based branch from 34e3302 to eff6489 Compare July 1, 2025 16:48
@cjgajard cjgajard merged commit a7172b9 into master Jul 1, 2025
10 checks passed
@cjgajard cjgajard deleted the cjavier/allow-delete-24h-content-based branch July 1, 2025 16:54
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