Skip to content

Added autoscaling docs and scale-to-zero details#1363

Merged
knative-prow-robot merged 4 commits into
knative:masterfrom
abrennan89:1332
May 31, 2019
Merged

Added autoscaling docs and scale-to-zero details#1363
knative-prow-robot merged 4 commits into
knative:masterfrom
abrennan89:1332

Conversation

@abrennan89
Copy link
Copy Markdown
Contributor

Fixes #1362

Proposed Changes

  • added autoscaling.md
  • updated links

@googlebot googlebot added the cla: yes Indicates the PR's author has signed the CLA. label May 22, 2019
@knative-prow-robot knative-prow-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label May 22, 2019
@abrennan89 abrennan89 changed the title 1332 Added autoscaling docs adn scale-to-zero details May 22, 2019
@abrennan89 abrennan89 changed the title Added autoscaling docs adn scale-to-zero details Added autoscaling docs and scale-to-zero details May 22, 2019
@abrennan89
Copy link
Copy Markdown
Contributor Author

/assign @samodell

Copy link
Copy Markdown
Contributor

@samodell samodell left a comment

Choose a reason for hiding this comment

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

This is looking great! Just a few small suggestions.

Comment thread docs/serving/README.md Outdated
The `revision.serving.knative.dev` resource is a point-in-time snapshot of the
code and configuration for each modification made to the workload. Revisions
are immutable objects and can be retained for as long as useful.
Knative Serving Revisions can be automatically scaled up and down according to incoming traffic. See [Autoscaling](./autoscaling.md) for more information.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

.md formatting nit -- this should be at the same level of indentation as the list item above, if you wanted it to be part of that. And if not, create a new line between the list item and this sentence.

Comment thread docs/serving/autoscaling.md Outdated
@@ -0,0 +1,114 @@
<!-- Add metadata as included for other files -->
# Autoscaling
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
# Autoscaling
---
title: "Configuring the Autoscaler"
weight: 10
type: "docs"
---

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'd update the title of the page to reflect whatever title you settle on. We try to shoot for action verbs in the page titles, which is why I added "Configuring," but if there's another word you'd rather use, go for it!

Comment thread docs/README.md Outdated
- [Orchestrating source-to-URL workflows on Kubernetes](./serving/samples/source-to-url-go/)
- [Routing and managing traffic with blue/green deployment](./serving/samples/blue-green-deployment.md)
- [Scaling automatically and sizing workloads based on demand](./serving/samples/autoscale-go/)
- [Scaling automatically and sizing workloads based on demand](./serving/autoscaling.md)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If you update the name of the file below based on my suggestions, make sure to also update it here.

Comment thread docs/serving/autoscaling.md Outdated

If there is no `/target` annotation, the autoscaler is configured as if `/target` == `containerConcurrency`.

## Configuring CPU based autoscaling
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
## Configuring CPU based autoscaling
## Configuring CPU-based autoscaling

Comment thread docs/serving/autoscaling.md Outdated
## Additional resources

- [Go autoscaling sample](https://knative.dev/docs/serving/samples/autoscale-go/index.html)
- Knative v0.3 Autoscaling  - A Love Story [blog post](https://medium.com/knative/knative-v0-3-autoscaling-a-love-story-d6954279a67a)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Knative v0.3 Autoscaling  - A Love Story [blog post](https://medium.com/knative/knative-v0-3-autoscaling-a-love-story-d6954279a67a)
- [Knative v0.3 Autoscaling  - A Love Story blog post](https://medium.com/knative/knative-v0-3-autoscaling-a-love-story-d6954279a67a)

Comment thread docs/serving/autoscaling.md Outdated
<!-- Add metadata as included for other files -->
# Autoscaling

Since Knative v0.2, per revision autoscalers have been replaced by a single shared autoscaler. This is, by default, the Knative Pod Autoscaler (KPA), which provides fast, request based autoscaling capabilities out of the box.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
Since Knative v0.2, per revision autoscalers have been replaced by a single shared autoscaler. This is, by default, the Knative Pod Autoscaler (KPA), which provides fast, request based autoscaling capabilities out of the box.
Since Knative v0.2, per revision autoscalers have been replaced by a single shared autoscaler. This is, by default, the Knative Pod Autoscaler (KPA), which provides fast, request-based autoscaling capabilities out of the box.

@abrennan89
Copy link
Copy Markdown
Contributor Author

Thanks @samodell :) the changes should be ready now

@abrennan89
Copy link
Copy Markdown
Contributor Author

/assign abrennan89

@samodell
Copy link
Copy Markdown
Contributor

/lgtm
/approve

Thanks for adding this, Ashleigh!

@knative-prow-robot knative-prow-robot added the lgtm Indicates that a PR is ready to be merged. label May 31, 2019
@knative-prow-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: abrennan89, samodell

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Indicates the PR's author has signed the CLA. lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create documentation for autoscaling

5 participants