newrelic-quickstarts icon indicating copy to clipboard operation
newrelic-quickstarts copied to clipboard

Published 20 hours ago • verified publisher icon newrelic

Prometheus Remote Write Integration

Open jpvajda opened this issue 2 years ago • 8 comments

This quickstart is lacking in dashboards and alerts and doesn't have an install plan. Due to the interest in it from our customers we should add a dashboard an alert to make this quickstart higher quality.

Justification

We have an upcoming event (Kubecon) where this will be showcased

Acceptance Criteria

  • [ ] add at least one new dashboard
  • [ ] add at least one alert
  • [ ] use this install plan for prometheus

jpvajda avatar Mar 10 '22 23:03 jpvajda

For the 3rd AC, I am confirming our approach with a few I/O teams, so that is something that is currently a work in progress how we'll solve that.

jpvajda avatar Mar 11 '22 16:03 jpvajda

I have added a dashboard for getting users started with monitoring their own Prometheus instance. The install plan I don't think adds that for users at the moment since the link takes users directly to configuring their remote write integration.

tclayton-newr avatar Mar 17 '22 16:03 tclayton-newr

@tclayton-newr yes, that's a limitation we have due to how we are trying to leverage the existing AMD workflows, let me ask the DevEx team if there is anyway to get the dashboards to install for the user using this flow.

jpvajda avatar Mar 17 '22 17:03 jpvajda

@tclayton-newr so this flow should allow the user to install the dashboards and alerts, the workflow follows our ruby quickstart install flow, which you can see by attempting to install Ruby on a new relic account, essentially a user is taken back to the install plan flow to finish the final step, after they finish the setup in the AMD nerdlet.

I'd suggest finishing one of these prometheus quickstarts and merging it, then we can test the flow to ensure the install plan is doing what we expect.

User starts here: Screen Shot 2022-03-17 at 11 21 41 AM

on click begin installation they go here Screen Shot 2022-03-17 at 11 20 25 AM

User finishes here: Screen Shot 2022-03-17 at 11 21 41 AM

jpvajda avatar Mar 17 '22 17:03 jpvajda

That's awesome! I'll have a look at running that from the new onboarding experience.

tclayton-newr avatar Mar 25 '22 14:03 tclayton-newr

@tclayton-newr yeah, it be really great to test this flow after we merge in this PR, sometimes the intended flow, isn't what we expected, so testing it out would be a fantastic idea.

jpvajda avatar Mar 25 '22 15:03 jpvajda

Old issues will be closed after 105 days of inactivity. This issue has been quiet for 90 days and is being marked as stale. Reply here to keep this issue open.

github-actions[bot] avatar Jun 24 '22 02:06 github-actions[bot]

I tested this some time ago and forgot to report back here 😛 The installation flow is there via the quickstart but it still leaves some things to be desired. Since the changes would have to be made internal to NR, I think it would be okay to close this ticket out.

To summarize, you currently have to close the integration install pane via the ❌ in the top right, which is not intuitive for new users. image

tclayton-newr avatar Jun 24 '22 14:06 tclayton-newr

Old issues will be closed after 105 days of inactivity. This issue has been quiet for 90 days and is being marked as stale. Reply here to keep this issue open.

github-actions[bot] avatar Dec 05 '22 02:12 github-actions[bot]

This issue is being closed due to inactivity. Is this a mistake? Please re-open this issue or create a new one.

github-actions[bot] avatar Mar 21 '23 01:03 github-actions[bot]