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

feat(apdex-optimizer): Added a quickstart for apdex-optimizer #2113

Merged
merged 7 commits into from
Nov 2, 2023

Conversation

DarrenDoyle
Copy link
Contributor

Summary

A concise description of the changes being introduced. Please review the pre-merge checklist section to validate this pull request is ready for review and merge. If it is not ready, please mark the pull request as a draft.

The owners of this repo are not experts in the subject matter of the quickstarts. We review for the quickstart to be functional and for security risks. If you are seeking feedback on the content of the quickstart, please seek out a subject matter expert. If you are not an internal NR contributor, we can do our best to connect you with a content reviewer.

Pre merge checklist

  • [Yes ] Did you check you NRQL syntax? - Does it work?
  • [Yes ] Did you include a Data source and Documentation reference?
  • [Yes ] Are all documentation links publicly accessible?
  • [ Yes] Did you check your descriptive content for voice and tone?
  • [Yes ] Did you check your descriptive content for spelling and grammar errors?
  • [Yes ] Did you review your content with a subject matter expert? (e.g. a Browser agent quickstart is reviewed with a member of the Browser Agent team)

Dashboards

  • [ Yes] Does the PR contain a screenshot for each of your dashboards?
  • [Yes ] Do your screenshots show data?
  • [ Yes] Has the sanitization script been run on each dashboard?

@DarrenDoyle DarrenDoyle requested a review from seemantk as a code owner October 13, 2023 10:08
@CLAassistant
Copy link

CLAassistant commented Oct 13, 2023

CLA assistant check
All committers have signed the CLA.

@github-actions
Copy link

Thank you for your contribution, our team will be reviewing this shortly, please be available for any follow up questions or code review feedback!

Copy link
Contributor

@aswanson-nr aswanson-nr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @DarrenDoyle, this is looking good! You just need to add in a data source to make this quickstart "installable" on user's accounts. Here are the docs on data sources

@aswanson-nr
Copy link
Contributor

Also, @DarrenDoyle could you sign the CLA as well?

@DarrenDoyle
Copy link
Contributor Author

Thanks @aswanson-nr how do I sign the CLA?

@aswanson-nr
Copy link
Contributor

Thanks @aswanson-nr how do I sign the CLA?

If you head over to the link in this it will take you through the process! Thank you

@DarrenDoyle
Copy link
Contributor Author

Thanks @aswanson-nr how do I sign the CLA?

If you head over to the link in this it will take you through the process! Thank you

Done Thanks

Comment on lines 11 to 41
## 1. Apdex Score Overview:
- Timeseries chart displaying Apdex scores for each APM application.
- Highlights performance trends over time.

## 2. Apdex Breakdown:
- Detailed breakdown table showcasing Apdex scores for different aspects of your application.
- Identifies areas of improvement and potential bottlenecks.

## 3. Transaction-level Insights:
- In-depth Apdex information on individual transactions.
- Pinpoints specific transaction performance for focused optimization.

## 4. Threshold Recommendations:
- Application Apdex T Value Recommendation table guides optimal threshold settings.
- Helps fine-tune Apdex parameters for enhanced performance monitoring.

## 5. Transaction Apdex Values:
- Transaction Apdex Values table provides granular insights into transaction-level performance.
- Enables precise adjustments for optimal user satisfaction.

### Actionable Takeaways:
- Identify and address performance trends through Apdex score patterns.
- Optimize specific transactions based on detailed Apdex insights.
- Fine-tune Apdex thresholds for applications to improve overall monitoring effectiveness.

### Benefits:
- Enhanced visibility into application performance.
- Proactive identification of potential issues.
- Improved user satisfaction through targeted optimizations.

This comprehensive Apdex Optimizer dashboard empowers you with actionable data to make informed decisions, ensuring your applications perform at their best.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we need these lines to be indented passed the description key. At the moment we would just get the description to be # summary of apdex optimizer dashboard.

If we indent these lines by 1, it should fix it!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @DarrenDoyle thanks for making that change. Unfortunately, you increased the indent by 2 rather than by 1. I think this might cause the text to render incorrectly in the product. Can you please update this?

Thanks.

fixed indent
@zstix zstix requested a review from rahulbasu-nr as a code owner November 2, 2023 16:50
@zstix zstix merged commit 1a0cb55 into newrelic:main Nov 2, 2023
2 checks passed
@DarrenDoyle
Copy link
Contributor Author

Hi @aswanson-nr Do you know when this dashboard will be available?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

5 participants