Skip to content

Conversation

AE-RobertMcDonnell
Copy link
Contributor

@AE-RobertMcDonnell AE-RobertMcDonnell commented Aug 8, 2025

… OpsGenieNotifier.buildUrl. This identifierType is required to successfully close an open alert when using an alias as the identifier.

closes #4551

… OpsGenieNotifier.buildUrl. This identifierType is required to successfully close an open alert when using an alias as the identifier.
@AE-RobertMcDonnell
Copy link
Contributor Author

Here is a link to the issue where I reported this bug: #4551

I accidentally closed the ticket when leaving a comment. Hopefully someone with permission can reopen it for me.

@hzpz
Copy link
Contributor

hzpz commented Aug 10, 2025

Not sure, why this pull request is stuck. Will try to close and then reopen it.

… OpsGenieNotifier.buildUrl. This identifierType is required to successfully close an open alert when using an alias as the identifier.
@SteKoe SteKoe force-pushed the bugfix/add-identifiertype-to-opsgenienotifier-buildurl branch from 3d2af45 to c750248 Compare August 11, 2025 07:39
@hzpz hzpz enabled auto-merge (squash) August 11, 2025 07:42
@hzpz
Copy link
Contributor

hzpz commented Aug 11, 2025

@AE-RobertMcDonnell Thank you for your contribution! There are test failures, could you please have a look?

…Notifier.buildUrl() method, this time with updates to the unit test.
auto-merge was automatically disabled August 11, 2025 16:21

Head branch was pushed to by a user without write access

…er-buildurl' into bugfix/add-identifiertype-to-opsgenienotifier-buildurl
@AE-RobertMcDonnell
Copy link
Contributor Author

I pushed commits that update the unit test. I see that Yash's push was rejected. The updated branch should pass tests now.

AE-RobertMcDonnell and others added 3 commits August 11, 2025 10:46
…er-buildurl' into bugfix/add-identifiertype-to-opsgenienotifier-buildurl
…genienotifier-buildurl' into bugfix/add-identifiertype-to-opsgenienotifier-buildurl
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.37%. Comparing base (4f9d0ae) to head (45e7c4c).
⚠️ Report is 2459 commits behind head on master.

Additional details and impacted files
@@ Coverage Diff @@ ## master #4557 +/- ## ============================================ - Coverage 83.64% 82.37% -1.28%  + Complexity 1247 1220 -27  ============================================ Files 156 164 +8 Lines 3644 4113 +469 Branches 258 257 -1 ============================================ + Hits 3048 3388 +340  - Misses 464 603 +139  + Partials 132 122 -10 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
@SteKoe SteKoe changed the title Added identifierType=alias to the OpsGenie close URL fix(#4551): adds missing identifierType=alias to the OpsGenie close URL Aug 15, 2025
@SteKoe SteKoe merged commit bc95ed5 into codecentric:master Aug 15, 2025
1 check passed
@SteKoe SteKoe added this to the 3.5.2 milestone Sep 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants