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.

Sign up for GitHub

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

Jump to bottom

feat: Fully support Events API v2 in PagerDuty step #697

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

feat: Fully support Events API v2 in PagerDuty step #697

wants to merge 11 commits into from

Conversation

kiyanmair
Copy link

@kiyanmair kiyanmair commented Jul 14, 2024 •
edited
Loading

What

  • Implement full support for PagerDuty's Events API v2.
  • Enable triggering incidents using the Events API.
  • Add a new event value to the PAGERDUTY_ALERT_TYPE enum.
  • Add a new PAGERDUTY_SEVERITY argument.
  • Update the step specification.
  • Update the step documentation.
  • Bump the step's minor version.
  • Maintain backwards compatibility.
  • Resolve PagerDuty step does not fully support Events v2 API #696.

Why

  • PagerDuty's Events API was only supported for creating change events.
  • Triggering incidents was only possible with PagerDuty's REST API.
  • More details in the linked issue.

Notes

  • I have not tested this PR.
  • I'm a new contributor to this repo and might have missed something.
  • This PR only supports the trigger event action, not acknowledge or resolve, because they add complexity to the step and I'm not sure they're useful in the Codefresh context.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

PagerDuty step does not fully support Events v2 API
1 participant
@kiyanmair