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

Still no support for PagerDuty Common Event Format (PD-CEF) #170

Open
nekodojo opened this issue Feb 7, 2024 · 1 comment
Open

Still no support for PagerDuty Common Event Format (PD-CEF) #170

nekodojo opened this issue Feb 7, 2024 · 1 comment

Comments

@nekodojo
Copy link

nekodojo commented Feb 7, 2024

This was previously reported as #127

There is currently no way to pass several fields present in the PagerDuty Common Event Format (PD-CEF), most notably Severity, Component, Group, and Class. These fields are potentially very useful in certain deployment strategies, so it would be ideal to see them supported.

I'm resubmitting because we still need this to be solved. We have tried the suggestion of switching to go-pdagent but it has other bugs and we gave up after many days of effort trying to make it work for our situation.

See my comment for more context related to PagerDuty support of the integration (which extends beyond the scope of this bug)

@nekodojo
Copy link
Author

nekodojo commented Feb 7, 2024

We have tried the suggestion of switching to go-pdagent but it has other bugs and we gave up after many days of effort trying to make it work for our situation.

PagerDuty team needs to pick one of these agents to support. Both projects have open bugs for multiple years now. In the official PagerDuty documentation, pdagent (python) is the official recommended agent and go-pdagent is still "beta" and not showing in PD docs as a recommended/supported option. It is OK to support both, and it's OK to support one if that's the official recommended one. it's not OK to stop supporting both.

From #127

After yet another log delay here, we have added this support into our new agent: PagerDuty/go-pdagent.
@aaron-goulet-bfg @apollo13 any chance you all could have a look at this and see if it meets your needs?

Copying my comments from #127 after it was closed

Since this got closed 3 years ago, should we assume that "go-pdagent" is the better agent and this older pdagent will stop getting updates?

I can see on Pagerduty Developer site that pdagent is still listed as the "official" agent, but a footnote says "go-pdagent *This project originated inside of PagerDuty with the eventual goal of replacing the Python based pdagent."

So what are the next steps by PD dev team to promote go-pdagent and retire Python based pdagent? Is this still the plan?

User @NargiT wrote:

go-pdagent is still in beta and does not support all event api v2 fields (PagerDuty/go-pdagent#34)

Good point... I'm hesitant to switch over to go-pdagent since it looks like it has open issues for 2+ years that aren't getting worked on. At least it supports some PD-CEF fields but I guess not all of them?

Anyway, my concern is that PD team is closing bugs against the older pdagent with comments pointing to the new one, but the new one is also not getting any love.

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

No branches or pull requests

1 participant