Pagerduty support

[chris-bridgett-nandos]

We've added support for PagerDuty to suit our needs (and have more event sources to come if there is interest). We've done our best to follow the same style as the other parsers and integrated with the setup script.

[google-cla]

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

For more information, open the CLA check for this pull request.

[dinagraves]

This is awesome!

[chris-bridgett-nandos]

Just bumped the parser flask version, I didn't realise the other parsers had moved on. Should fix the test failure. 🤞

@dinagraves would there be interest in similar PR's for more data sources? We also have ArgoCD and Azure Devops parsers in our fork at the moment and a modified setup script to allow multi-choice when selecting which parsers to enable. We'd be happy to submit them as separate PR's.

[dinagraves]

I think we want to do this part in Terraform - @davidstanke

[chris-bridgett-nandos]

Here's one way we could bring secrets management under Terraform (targeting this PagerDuty support PR). 🙂

[dinagraves]

Looks pretty good! Once you get that merged in, we'll do a final review and merge this one in!

[dinagraves]

Just bumped the parser flask version, I didn't realise the other parsers had moved on. Should fix the test failure. 🤞

@dinagraves would there be interest in similar PR's for more data sources? We also have ArgoCD and Azure Devops parsers in our fork at the moment and a modified setup script to allow multi-choice when selecting which parsers to enable. We'd be happy to submit them as separate PR's.

Yes! Extremely interested in the other parsers! Please submit PR's!

[chris-bridgett-nandos]

Yes! Extremely interested in the other parsers! Please submit PR's!

Great. 😄 Incoming! ⏳

[davidstanke]

Hi @chris-bridgett-nandos ! We've updated the test configuration (and a few other things). Can you merge main of this repo into your fork? I think that will re-trigger the tests, and then we can merge this.

[chris-bridgett-nandos]

Hi @davidstanke. Done... I think it's configured to trigger correctly, but is prevented from running still since I'm a first-time contributor.

I now see:

Triggered via pull request 1 minute ago

(So merging the upstream main did trigger), but:

First-time contributors need a maintainer to approve running workflows.

[chris-bridgett-nandos]

Hi @davidstanke, mind approving the test-run again? The failing test should now be fixed.

[dinagraves]

Merged! Thank you!

[colinfrei]

Hi @chris-bridgett-nandos
You wrote that you have an Azure Devops parser running as well - any chance of a PR for that, or a repo I can take a look at?

Thanks!