Skip to content

Specify the APIVersion of the autoscaler target.#59

Merged
josephburnett merged 1 commit into
knative:masterfrom
josephburnett:master
Feb 2, 2018
Merged

Specify the APIVersion of the autoscaler target.#59
josephburnett merged 1 commit into
knative:masterfrom
josephburnett:master

Conversation

@josephburnett
Copy link
Copy Markdown
Contributor

@josephburnett josephburnett commented Feb 2, 2018

Autoscaling was broken because the horizontal pod autoscaler was unable to find the deployment. Message was "the HPA controller was unable to get the target's current scale: no matches for /, Kind=Deployment" and it showed / 80%.

Now the autoscale is able to find the deployment and shows the actual CPU usage.

Autoscaling was broken because the horizontal pod autoscaler was unable to find the deployment.
Copy link
Copy Markdown
Contributor

@grantr grantr left a comment

Choose a reason for hiding this comment

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

Thanks for expanding the explanation!

@josephburnett josephburnett merged commit 389378f into knative:master Feb 2, 2018
dprotaso added a commit to dprotaso/serving that referenced this pull request Sep 12, 2018
Includes:
8fc80de Few changes to the configmap package (knative#59)
knative-prow-robot pushed a commit that referenced this pull request Sep 12, 2018
* Bump knative/pkg to 8fc80de

Includes:
8fc80de Few changes to the configmap package (#59)

* Use updated configmap.Watcher New* method names
tcnghia pushed a commit to tcnghia/serving that referenced this pull request Jun 8, 2020
Produced via:
  `./hack/update-deps.sh --upgrade && ./hack/update-codegen.sh`
/assign shashwathi tanzeeb
/cc shashwathi tanzeeb
nak3 referenced this pull request in nak3/serving Dec 6, 2022
#59)

* Add gRPC test work against OpenShift Routes

* Fix typo

Co-authored-by: Kenjiro Nakayama <nakayamakenjiro@gmail.com>
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.

3 participants