Skip to content

Remove custom-metric APIServer code.#8318

Merged
knative-prow-robot merged 5 commits intoknative:masterfrom
markusthoemmes:remove-custom-metrics
Jun 22, 2020
Merged

Remove custom-metric APIServer code.#8318
knative-prow-robot merged 5 commits intoknative:masterfrom
markusthoemmes:remove-custom-metrics

Conversation

@markusthoemmes
Copy link
Contributor

Fixes #8164

Proposed Changes

In accordance to our release principles, a deprecation notice is not necessary here since this is an alpha feature. It's been verified with every Knative vendor that this feature is not shipped in any productized version of Knative, neither has there been any proof that any customer, be it OSS or product, is actively using this.

/hold

Holding until having proper signoff by the Autoscaling WG and maybe also ToC.

@knative-prow-robot knative-prow-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jun 15, 2020
@googlebot googlebot added the cla: yes Indicates the PR's author has signed the CLA. label Jun 15, 2020
@knative-prow-robot knative-prow-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Jun 15, 2020
@knative-prow-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: markusthoemmes

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

@knative-prow-robot knative-prow-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 15, 2020
@knative-prow-robot knative-prow-robot added area/API API objects and controllers area/autoscale labels Jun 15, 2020
Copy link
Contributor

@knative-prow-robot knative-prow-robot left a comment

Choose a reason for hiding this comment

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

@markusthoemmes: 0 warnings.

Details

In response to this:

Fixes #8164

Proposed Changes

In accordance to our release principles, a deprecation notice is not necessary here since this is an alpha feature. It's been verified with every Knative vendor that this feature is not shipped in any productized version of Knative, neither has there been any proof that any customer, be it OSS or product, is actively using this.

/hold

Holding until having proper signoff by the Autoscaling WG and maybe also ToC.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@knative-prow-robot knative-prow-robot added the area/test-and-release It flags unit/e2e/conformance/perf test issues for product features label Jun 15, 2020
@markusthoemmes markusthoemmes force-pushed the remove-custom-metrics branch from 334e287 to 8fa3d3a Compare June 22, 2020 05:44
@markusthoemmes
Copy link
Contributor Author

/unhold

Time's up. We haven't gotten any indication of usage.

@knative-prow-robot knative-prow-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jun 22, 2020
@knative-metrics-robot
Copy link

The following is the coverage report on the affected files.
Say /test pull-knative-serving-go-coverage to re-run this coverage report

File Old Coverage New Coverage Delta
pkg/reconciler/autoscaling/hpa/hpa.go 97.0% 96.7% -0.3

Copy link
Contributor

@julz julz left a comment

Choose a reason for hiding this comment

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

/lgtm

\o/

@knative-prow-robot knative-prow-robot added the lgtm Indicates that a PR is ready to be merged. label Jun 22, 2020
@knative-prow-robot knative-prow-robot merged commit ac2f58a into knative:master Jun 22, 2020
@yanweiguo
Copy link
Contributor

\o/

@julz julz mentioned this pull request Jul 23, 2020
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. area/API API objects and controllers area/autoscale area/test-and-release It flags unit/e2e/conformance/perf test issues for product features cla: yes Indicates the PR's author has signed the CLA. lgtm Indicates that a PR is ready to be merged. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

State of our HPA/custom-scaler support

6 participants