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

Update fleet autoscaling limited signification(#2828) #3448

Merged
merged 4 commits into from
Oct 25, 2023

Conversation

atgane
Copy link
Contributor

@atgane atgane commented Oct 21, 2023

What type of PR is this?:

/kind documentation

What this PR does / Why we need it:

image image
  • It's actually ScalingUpOrDownLimited in applyBufferPolicy(). It works like above image:

  • ✅ So I want to change grafana metric title ScalingLimited to Scaling Down or Up Limited and enhance code readability

Which issue(s) this PR fixes:

Special notes for your reviewer: N/A

@google-oss-prow google-oss-prow bot requested a review from EricFortin October 21, 2023 06:51
@google-cla
Copy link

google-cla bot commented Oct 21, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@google-oss-prow google-oss-prow bot requested a review from markmandel October 21, 2023 06:51
@github-actions github-actions bot added the kind/documentation Documentation for Agones label Oct 21, 2023
@atgane atgane force-pushed the docs/fleetAutoScaler branch from a9d3eb9 to bbb0d4e Compare October 23, 2023 12:22
@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: aa917877-851e-431c-bb1c-0141f9e7639b

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website (the last 30 builds are retained):

To install this version:

  • git fetch https://github.com/googleforgames/agones.git pull/3448/head:pr_3448 && git checkout pr_3448
  • helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.36.0-dev-bbb0d4e-amd64

@@ -1084,7 +1084,7 @@ data:
}
],
"thresholds": "",
"title": "Scaling Limited",
"title": "Scaling Down or Up Limited",
Copy link
Collaborator

Choose a reason for hiding this comment

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

nitpick but scale out/in sounds more suitable here.

@google-oss-prow
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: atgane, gongmax
Once this PR has been reviewed and has the lgtm label, please assign alekser for approval. For more information see the Kubernetes Code Review Process.

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

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

@google-oss-prow google-oss-prow bot removed the lgtm label Oct 24, 2023
@google-oss-prow
Copy link

New changes are detected. LGTM label has been removed.

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 269596b9-d025-4151-a1da-7347bcaae6de

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website (the last 30 builds are retained):

To install this version:

  • git fetch https://github.com/googleforgames/agones.git pull/3448/head:pr_3448 && git checkout pr_3448
  • helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.36.0-dev-41921b3-amd64


if replicas < b.MinReplicas {
replicas = b.MinReplicas
limited = true
scalingDownLimited = true
Copy link
Collaborator

@gongmax gongmax Oct 24, 2023

Choose a reason for hiding this comment

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

Sorry for the back and forth but better to change this and line 233, 236 too for consistency

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: b19463c9-6c5f-4a0d-bb04-f0e851d4bf2f

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: ab4f6983-3d91-4363-a4fa-dfd7187d3152

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 2ddcc15e-06d6-4957-a847-9c0ce7dbe931

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website (the last 30 builds are retained):

To install this version:

  • git fetch https://github.com/googleforgames/agones.git pull/3448/head:pr_3448 && git checkout pr_3448
  • helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.36.0-dev-81ba000-amd64

@gongmax gongmax merged commit ff5b522 into googleforgames:main Oct 25, 2023
2 checks passed
@atgane atgane deleted the docs/fleetAutoScaler branch October 25, 2023 23:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/documentation Documentation for Agones size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FleetAutoScaler scalingLimited is reporting the incorrect state
3 participants