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

PagerDuty dynamic/additional fields #92388

Open
arisonl opened this issue Feb 23, 2021 · 9 comments
Open

PagerDuty dynamic/additional fields #92388

arisonl opened this issue Feb 23, 2021 · 9 comments
Labels
Feature:Actions/ConnectorTypes Issues related to specific Connector Types on the Actions Framework Feature:Actions Meta Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)

Comments

@arisonl
Copy link
Contributor

arisonl commented Feb 23, 2021

Currently the PagerDuty action fields are hardcoded. In addition, there are are requests for support of additional fields,
e.g. 1) custom_details, images, links and 2) urgency.

cc @YulNaumenko

@arisonl arisonl added the Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) label Feb 23, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-alerting-services (Team:Alerting Services)

@mikecote
Copy link
Contributor

@arisonl is this a duplicate of #63933?

@arisonl
Copy link
Contributor Author

arisonl commented Feb 23, 2021

@mikecote It contains it as it is broader. Also it is about making the action fields dynamic.

@mikecote
Copy link
Contributor

@arisonl gotcha, thanks! Should we close the other one in favour of this one?

@pmuellr
Copy link
Member

pmuellr commented Feb 23, 2021

making the action fields dynamic

Not sure what this means. The ability to add new action parameters to a specific invocation of a connector? Is this like the ServiceNow mapping fields?

@mikecote mikecote added the Meta label Feb 23, 2021
@pmuellr
Copy link
Member

pmuellr commented Feb 24, 2021

I was going to try to reform this as a meta issue, but I still have some questions that we should settle on before re-orging all the PagerDuty issues:

  • re: urgency - that's not a field in the PD events API: https://developer.pagerduty.com/docs/events-api-v2/trigger-events/ - so there's nowhere for us to put a value like that

  • for images, I'll end up referring to Add image/chart support to actions #49908 as a blocker, since I think we'll want to somehow create images from Kibana views, and will need some fundamental architecture to do this. For additional complication, it doesn't appear PagerDuty has "image upload" support, for us to upload an image, like Slack and GitHub do. That means we would need to arrange to either upload the images somewhere else, or host them in Kibana, but then how do we provide RBAC, or any auth, on those? I'm hesitant to even create an issue here because there's so little known about how we would deal with this

  • for links, I think UI wise this is going to mean we'll allow the user to create a "list" of links, where each link has link text and a URL. So kinda like webhook headers. Is that acceptable?

  • for custom details, this could also be similar to webhook headers - key/value pairs. Or were we thinking this should be more aligned with the way we do custom fields in ServiceNow, etc?

@pmuellr
Copy link
Member

pmuellr commented Feb 24, 2021

Here's a list of (at least some of the) relevant issues, once we re-org this:

@pmuellr pmuellr self-assigned this Feb 24, 2021
@pmuellr
Copy link
Member

pmuellr commented Apr 12, 2021

@arisonl just noticing I'm still assigned to this issue, so wanted to see what we want to do about it. Not sure if there's anything actionable, if we need some more work on it somehow? It's also not being tracked in our project ...

@runtman
Copy link

runtman commented May 23, 2021

I was going to try to reform this as a meta issue, but I still have some questions that we should settle on before re-orging all the PagerDuty issues:

You are absolutely right but severity is still something. Can this be customised in favour of urgency?

payload.severityRequired | String | The perceived severity of the status the event is describing with respect to the affected system.

From memory watcher creates everything as critical.

@gmmorris gmmorris added Feature:Actions Feature:Actions/ConnectorTypes Issues related to specific Connector Types on the Actions Framework Meta and removed Meta labels Jul 2, 2021
@kobelb kobelb added the needs-team Issues missing a team label label Jan 31, 2022
@botelastic botelastic bot removed the needs-team Issues missing a team label label Jan 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Actions/ConnectorTypes Issues related to specific Connector Types on the Actions Framework Feature:Actions Meta Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)
Projects
None yet
Development

No branches or pull requests

7 participants