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

[synthetics-job-manager] Update docs in response to GTSE. #1181

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

sidnaiknewrelic
Copy link
Contributor

@sidnaiknewrelic sidnaiknewrelic commented Oct 24, 2023

Is this a new chart

No.

What this PR does / why we need it:

ticket: https://new-relic.atlassian.net/jira/software/c/projects/NR/boards/500?selectedIssue=NR-170132

In response to the above GTSE, I am updating the README here to have the (Not yet configurable) note next to replica count. This is necessary to avoid confusion of a customer trying to change the replica count when we have not added support for that yet.

The note can also be changed to (Not configurable yet) or (Not supported yet). I left this choice up to the reviewer though, if they think that sounds better.

Special notes for your reviewer:

GTSE ticket should have the details for the issue.

Checklist

  • Variables are documented in the README.md
  • Title of the PR starts with chart name (e.g. [mychartname])

ticket: https://new-relic.atlassian.net/jira/software/c/projects/NR/boards/500?selectedIssue=NR-170132

In response to the above GTSE, I am updating the README here to have the (Not yet configurable) note next to replica count. This is necessary to avoid confusion of a customer trying to change the replica count when we have not added support for that yet.
@sidnaiknewrelic sidnaiknewrelic requested a review from a team October 24, 2023 18:00
@sidnaiknewrelic sidnaiknewrelic changed the title fix: Update docs in response to GTSE. fix: [synthetics-job-manager] Update docs in response to GTSE. Oct 24, 2023
@sidnaiknewrelic sidnaiknewrelic changed the title fix: [synthetics-job-manager] Update docs in response to GTSE. [synthetics-job-manager] Update docs in response to GTSE. Oct 24, 2023
@@ -53,7 +53,7 @@ This chart will deploy the New Relic Synthetics Containerized Private Job Manage

| Parameter | Description | Default |
|---------------------------------------------|------------------------------------------------------------------------------------------------------|------------------------------------|
| `ping-runtime.replicaCount` | Number of ping-runtime replicas to maintain | `1` |
| `ping-runtime.replicaCount` | *(Not yet configurable)* Number of ping-runtime replicas to maintain | `1` |
Copy link
Contributor

Choose a reason for hiding this comment

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

Fix table spacing 👀

kondracek-nr
kondracek-nr previously approved these changes Oct 24, 2023
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.

2 participants