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

New Relic Synthetics quota #8956

Merged
merged 6 commits into from
May 8, 2024
Merged

New Relic Synthetics quota #8956

merged 6 commits into from
May 8, 2024

Conversation

tyherbert
Copy link
Contributor

We are adding a quota for New Relic to 10K per workspace, updating docs to reflect this change.

Closes #

Summary

Doc Page Title - <Enter a one sentence summation of the pertinent changes (including not-yet-completed work) provided by this PR.>

Effect

The following changes are already committed:

Remaining Work and Prerequisites

The following changes still need to be completed:

  • List any outstanding work here

Dependencies and Timing

  • Other prerequisites that must be completed before merging this PR

Release:

  • When ready
  • After date: $DATE

Post Launch

Do not remove - To be completed by the docs team upon merge:

  • Redirect /old-path/ => /new-path/ (if applicable)
  • Include/exclude pages ^ respectively within docs search service provider (if applicable)

We are adding a quota for New Relic to 10K per workspace, updating docs to reflect this change.
@tyherbert tyherbert requested a review from a team as a code owner April 23, 2024 22:37
Copy link

👋 @tyherbert
Thanks for opening your first pull request! @pantheon-systems/docs-admins is excited to review this!

If you like this project, please ⭐star⭐ our repo.

Copy link

⚡ Deployed with Pantheon Decoupled

This build was successfully deployed with Pantheon. You can track the build logs here.

👀 Preview: https://pr-8956-documentation.appa.pantheon.site
🛠️ Manage in Pantheon: https://dashboard.pantheon.io/site/2b30153f-e8b1-4427-b076-6109e704ba5d/overview

I am updating the docs to call out the 10K synthetic check limit per Pantheon workspace.
@rachelwhitton rachelwhitton mentioned this pull request Apr 24, 2024
6 tasks
Copy link

⚡ Deployed with Pantheon Decoupled

This build was successfully deployed with Pantheon. You can track the build logs here.

👀 Preview: https://pr-8956-documentation.appa.pantheon.site
🛠️ Manage in Pantheon: https://dashboard.pantheon.io/site/2b30153f-e8b1-4427-b076-6109e704ba5d/overview

Copy link
Member

@rachelwhitton rachelwhitton left a comment

Choose a reason for hiding this comment

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

@tyherbert thanks for opening this PR! This change should have a release note as well, can you add that to this PR?

Copy link

⚡ Deployed with Pantheon Decoupled

This build was successfully deployed with Pantheon. You can track the build logs here.

👀 Preview: https://pr-8956-documentation.appa.pantheon.site
🛠️ Manage in Pantheon: https://dashboard.pantheon.io/site/2b30153f-e8b1-4427-b076-6109e704ba5d/overview

@rachelwhitton rachelwhitton changed the title Update 02-activate-new-relic.md New Relic Synthetics quota Apr 24, 2024
Copy link

⚡ Deployed with Pantheon Decoupled

This build was successfully deployed with Pantheon. You can track the build logs here.

👀 Preview: https://pr-8956-documentation.appa.pantheon.site
🛠️ Manage in Pantheon: https://dashboard.pantheon.io/site/2b30153f-e8b1-4427-b076-6109e704ba5d/overview


<Alert title="Note" type="info">

When a Pantheon workspace has more than 10K Synthetics checks are running monthly Pantheon will adjust the frequency or number of locations a Synthetics check originates from the reduce the number of checks. This is required to allow Synthetic to continue to operate correctly under the overall New Relic quota for Pantheon.
Copy link
Contributor

Choose a reason for hiding this comment

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

@tyherbert will our customers know that/when this reduction happens?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, we plan to notify the customer via email if a change has been made to their Synthetics monitors.

Copy link
Contributor

Choose a reason for hiding this comment

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

So is the flow something like

  • Pantheon staff somehow sees that a workspace has exceed 10K
  • Pantheon staff manually adjust settings in New Relic
  • Pantheon staff manually emails customer?

Who from the workspace will get that email?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Correct. This is something that Manoj and I will be working on. We are following the customer communication flow (https://docs.google.com/document/d/1Myagjy-2JhfnG7AqLs6clQ2YV0jxRvz_6nxQK898RQg/edit#heading=h.2licx9cbwmja). Currently we send an email to the site owner and cc the admins (this is a process that Shaq and Manoj put together).

@tyherbert
Copy link
Contributor Author

@rachelwhitton Here is the PR for the release notes: #8980

Co-authored-by: tyherbert <tyherbert@gmail.com>
Copy link

⚡ Deployed with Pantheon Decoupled

This build was successfully deployed with Pantheon. You can track the build logs here.

👀 Preview: https://pr-8956-documentation.appa.pantheon.site
🛠️ Manage in Pantheon: https://dashboard.pantheon.io/site/2b30153f-e8b1-4427-b076-6109e704ba5d/overview

@rachelwhitton rachelwhitton merged commit 8145f52 into main May 8, 2024
5 of 8 checks passed
Copy link

⚡ Deployed with Pantheon Decoupled

This build was successfully deployed with Pantheon. You can track the build logs here.

👀 Preview: https://pr-8956-documentation.appa.pantheon.site
🛠️ Manage in Pantheon: https://dashboard.pantheon.io/site/2b30153f-e8b1-4427-b076-6109e704ba5d/overview

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

Successfully merging this pull request may close these issues.

3 participants