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

Small changes to doc #3233

Merged
merged 1 commit into from Jul 22, 2021
Merged

Small changes to doc #3233

merged 1 commit into from Jul 22, 2021

Conversation

HenryTech
Copy link
Contributor

Process Running (PR) alert conditions are being migrated to use NRQL (Willamette pipeline). This move will mean that they can no longer tell if a process failed to start when a host is spun up. I have tried to remove all references to that sort of behavior, since PR conditions will change to being more "Process NO LONGER running."

I feel like these changes will not need to be applied until we move the PR condition types over to Willamette (happening soon, but has not happened yet). On the other hand, these don't feel like breaking changes to the document, so could be published immediately without any impact (I think).

Please do send me a DM in Slack (@shenry) if you have any questions about these changes.

Give us some context

  • What problems does this PR solve?
  • Add any context that will help us review your changes such as testing notes,
    links to related docs, screenshots, etc.
  • If your issue relates to an existing GitHub issue, please link to it.

Are you making a change to site code?

If you're changing site code (rather than the content of a doc), please follow
conventional commit standards
in your commit messages and pull request title.

Process Running (PR) alert conditions are being migrated to use NRQL (Willamette pipeline). This move will mean that they can no longer tell if a process failed to start when a host is spun up. I have tried to remove all references to that sort of behavior, since PR conditions will change to being more "Process NO LONGER running."

I feel like these changes will not need to be applied until we move the PR condition types over to Willamette (happening soon, but has not happened yet). On the other hand, these don't feel like breaking changes to the document, so could be published immediately without any impact (I think).

Please do send me a DM in Slack (@shenry) if you have any questions about these changes.
@github-actions github-actions bot added this to Needs Triage in Docs PRs and Issues Jul 22, 2021
@mmfred mmfred added content requests related to docs site content from_internal Identifies issues/PRs from Relics (except writers) pg_AI labels Jul 22, 2021
@mmfred mmfred moved this from Needs Triage to Hero: To do in Docs PRs and Issues Jul 22, 2021
@mmfred mmfred merged commit 538555e into newrelic:develop Jul 22, 2021
@mmfred
Copy link
Contributor

mmfred commented Jul 22, 2021

Thanks for the fixes @HenryTech!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content requests related to docs site content from_internal Identifies issues/PRs from Relics (except writers)
Projects
Docs PRs and Issues
Scheduled work
Development

Successfully merging this pull request may close these issues.

None yet

2 participants