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] every action execution showing evidence of refresh: wait_for #99101

Closed
pmuellr opened this issue May 3, 2021 · 3 comments
Closed
Assignees
Labels
Feature:Actions Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)

Comments

@pmuellr
Copy link
Member

pmuellr commented May 3, 2021

During some analysis of alert and action duration timings, it appears actions are paying a cost of refresh:wait_for latency. We should investigate to see if we can change this.

For alerting, I noticed we turned it false, here:

try {
await partiallyUpdateAlert(client, alertId, attributes, {
ignore404: true,
namespace,
refresh: false,
});
} catch (err) {
this.logger.error(
`error updating alert execution status for ${this.alertType.id}:${alertId} ${err.message}`
);
}

And I'm thinking this is the primary way alerts are updated now (partially), when we re-org'd that.

So we can probably do the same for actions, but of course will need to review the interactions in the action SO's to make sure this is ok.

@pmuellr pmuellr added Feature:Actions Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) labels May 3, 2021
@elasticmachine
Copy link
Contributor

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

@YulNaumenko YulNaumenko self-assigned this May 18, 2021
@YulNaumenko
Copy link
Contributor

Related PR #99919

@YulNaumenko
Copy link
Contributor

Closing in favor of the existing issue, which has more context and currently is in the To-Do of Alerting project board

@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 Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)
Projects
None yet
Development

No branches or pull requests

4 participants