-
Notifications
You must be signed in to change notification settings - Fork 65
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
[MAIN-2976] new relic support #266
base: master
Are you sure you want to change the base?
Conversation
Avi-Robusta
commented
Feb 4, 2025
•
edited
Loading
edited
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some small comment about docs & the newrelic URL.
|
||
For more details on New Relic's API and authentication methods, refer to [New Relic API documentation](https://docs.newrelic.com/docs/apis). | ||
|
||
</details> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be moved to a page on robusta docs where we gathered all the toolset documentation: https://docs.robusta.dev/master/configuration/holmesgpt/builtin_toolsets.html
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Example PR: robusta-dev/robusta#1727
""" | ||
} | ||
|
||
url = "https://api.newrelic.com/graphql" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be overridable for people having on premise newrelic?
super().__init__( | ||
name="newrelic", | ||
description="Toolset for interacting with New Relic to fetch logs and traces", | ||
docs_url="https://docs.newrelic.com/docs/apis/nerdgraph-api/", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Similar comment to the above regarding docs, URL should be similar to https://docs.robusta.dev/master/configuration/holmesgpt/toolsets/newrelic.html