Skip to content

Add alert source APIs#543

Merged
rocktavious merged 2 commits into
mainfrom
kr/alert-source-apis
May 15, 2025
Merged

Add alert source APIs#543
rocktavious merged 2 commits into
mainfrom
kr/alert-source-apis

Conversation

@rocktavious
Copy link
Copy Markdown
Contributor

Resolves #

Problem

We have new APIs for working with alert sources more generically but not way to do this via opslevel-go

Solution

Codegen the structs and add the necessary client functions for this

Checklist

  • I have run this code, and it appears to resolve the stated issue.
  • This PR does not reduce total test coverage
  • This PR has no user interface changes or has already received approval from product management to change the interface.
  • Does this change require a Terraform schema change?
    • If so what is the ticket or PR #
  • Make a changie entry that explains the customer facing outcome of this change

@rocktavious rocktavious requested a review from p25marti May 14, 2025 19:44
@rocktavious rocktavious self-assigned this May 14, 2025
Base automatically changed from kr/update-defaults to main May 15, 2025 13:31
Copy link
Copy Markdown
Contributor

@p25marti p25marti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@rocktavious rocktavious merged commit 11406ad into main May 15, 2025
2 checks passed
@rocktavious rocktavious deleted the kr/alert-source-apis branch May 15, 2025 15:18
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