-
Notifications
You must be signed in to change notification settings - Fork 1.4k
improve flow for K8s SJM configuration #22020
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
Conversation
|
Hi @keegoid-nr 👋 Thanks for your pull request! Your PR is in a queue, and a writer will take a look soon. We generally publish small edits within one business day, and larger edits within three days. Please ensure the propsed changes look good by building it first in your local environment. Refer to this contribution guide to get the site up and running in your local. If you really require a preview url, reach out to one of the writers and they will generate one for you. |
|
Netlify build fork |
|
✅ Your PR has been mirrored to our repository as PR #22026. |
...content/docs/synthetics/synthetic-monitoring/private-locations/job-manager-configuration.mdx
Outdated
Show resolved
Hide resolved
...content/docs/synthetics/synthetic-monitoring/private-locations/job-manager-configuration.mdx
Outdated
Show resolved
Hide resolved
...content/docs/synthetics/synthetic-monitoring/private-locations/job-manager-configuration.mdx
Outdated
Show resolved
Hide resolved
...content/docs/synthetics/synthetic-monitoring/private-locations/job-manager-configuration.mdx
Outdated
Show resolved
Hide resolved
...content/docs/synthetics/synthetic-monitoring/private-locations/job-manager-configuration.mdx
Outdated
Show resolved
Hide resolved
...content/docs/synthetics/synthetic-monitoring/private-locations/job-manager-configuration.mdx
Outdated
Show resolved
Hide resolved
...content/docs/synthetics/synthetic-monitoring/private-locations/job-manager-configuration.mdx
Outdated
Show resolved
Hide resolved
...content/docs/synthetics/synthetic-monitoring/private-locations/job-manager-configuration.mdx
Outdated
Show resolved
Hide resolved
...content/docs/synthetics/synthetic-monitoring/private-locations/job-manager-configuration.mdx
Outdated
Show resolved
Hide resolved
...content/docs/synthetics/synthetic-monitoring/private-locations/job-manager-configuration.mdx
Outdated
Show resolved
Hide resolved
...content/docs/synthetics/synthetic-monitoring/private-locations/job-manager-configuration.mdx
Outdated
Show resolved
Hide resolved
...content/docs/synthetics/synthetic-monitoring/private-locations/job-manager-configuration.mdx
Outdated
Show resolved
Hide resolved
d7a6e66 to
6abdf07
Compare
|
Netlify build fork |
|
✅ Your updates have been mirrored to our repository in PR #22026. |
PallaviWrite
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
- Expanded the section on Kubernetes and OpenShift job manager configuration, detailing how to size deployments, calculate parallelism and completions, and provided examples for scaling out with multiple SJM instances.
- fix formatting
- Update job-manager-configuration.mdx
fix: formatting and content
- remove duplicate content
- stick with one unit type (minutes) for $D_{avg,m}$
- reorganize headings by VM and K8s
- add content around fixed-capacity values and how that improves
maintainability by scaling Helm releases instead
- fix mdx errors
- use \cdot for multiplication
- fixes and improvements
|
@keegoid-nr, as informed in the Slack channel, this PR is ready to merge. |
|
netlify build fork |
|
✅ Your updates have been mirrored to our repository in PR #22026. |
adutta-newrelic
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This PR improves the clarity and steps, and adds details about how to scale out using fixed-capacity units for parallelism and completions per Helm deployment.