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

[APM] Cannot create transaction duration rule outside of context of APM app #107940

Closed
dgieselaar opened this issue Aug 9, 2021 · 6 comments
Closed
Labels
apm:alerting bug Fixes for quality problems that affect the customer experience Team:APM All issues that need APM UI Team support

Comments

@dgieselaar
Copy link
Member

When the user wants to create a transaction duration rule outside of the context of the APM app, the trigger component never renders, as serviceName is missing. The consequence is that the user cannot create these rules outside of the APM app.

To solve this, we could implement a free-form text field to allow the user to set serviceName themselves. This would also open the possibility of using this for environment and transactionType, and allowing the user to create this rule anywhere in the APM app, rather than just in the context of a specific service.

@dgieselaar dgieselaar added bug Fixes for quality problems that affect the customer experience Team:APM All issues that need APM UI Team support labels Aug 9, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/apm-ui (Team:apm)

@cauemarcondes
Copy link
Contributor

cauemarcondes commented Aug 9, 2021

Might be related to it, currently when I try to create an alert having transaction type: All an error happens fixed in #107942:

Screen Shot 2021-08-09 at 15 41 22

@dgieselaar
Copy link
Member Author

We could make it clear to users that they cannot create alerts from here:

  );
  
  if(!metadata?.serviceName) {
  	return <EuiCallout>Creating alerts from here is not supported. Please go to the [APM app](link))</EuiCallout>
  }

Originally posted by @sqren in #107942 (comment)

@smith
Copy link
Contributor

smith commented Aug 17, 2021

Is this a duplicate of #106786?

@sorenlouv
Copy link
Member

sorenlouv commented Aug 19, 2021

Is this a duplicate of #106786?

Sort of. #106786 refers to editing existing APM alerts, while this issue is about creating new APM alerts from Stack Management. I think the solution is the same (free form text fields), and they could probably be grouped as a single ticket "Ability to add and edit alerts outside of APM UI"

Unless there's any objections I'll close this one.

@sorenlouv
Copy link
Member

Replaced by #106786

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
apm:alerting bug Fixes for quality problems that affect the customer experience Team:APM All issues that need APM UI Team support
Projects
None yet
Development

No branches or pull requests

5 participants