Skip to content
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

[actions] use pagerduty CLI to create alert #1218

Merged
merged 9 commits into from
Jul 16, 2024

Conversation

charliecruzan-stripe
Copy link
Contributor

@charliecruzan-stripe charliecruzan-stripe commented Jul 15, 2024

Reviewers

r? @
cc @stripe/developer-products

Summary

The github action isn't maintained and doesn't have a lot of features, let's just use Pager Duty CLI instead. This makes it so that the title of the pagerduty alert isn't a very long and confusing string plus url (plus my name, which im not a super big fan of 😄 ). Hopefully this will make this page a lot clearer to the runner

@charliecruzan-stripe charliecruzan-stripe requested a review from a team as a code owner July 15, 2024 22:34
Copy link
Contributor

@fhats-stripe fhats-stripe left a comment

Choose a reason for hiding this comment

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

👍

.github/workflows/install-test.yml Show resolved Hide resolved
@charliecruzan-stripe charliecruzan-stripe merged commit b241c27 into master Jul 16, 2024
13 checks passed
@charliecruzan-stripe charliecruzan-stripe deleted the charliecruzan-installtestpagerduty branch July 16, 2024 19:04
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