Skip to content

Drop the use of serving specific metric keys declared in knative.dev/pkg#11658

Merged
knative-prow-robot merged 2 commits into
knative:mainfrom
dprotaso:drop-pkg-serving-metrickeys
Jul 12, 2021
Merged

Drop the use of serving specific metric keys declared in knative.dev/pkg#11658
knative-prow-robot merged 2 commits into
knative:mainfrom
dprotaso:drop-pkg-serving-metrickeys

Conversation

@dprotaso
Copy link
Copy Markdown
Member

@dprotaso dprotaso commented Jul 12, 2021

Part of: knative/pkg#608

  • 🧹 moves serving metric constants in knative.dev/pkg to here

@google-cla google-cla Bot added the cla: yes Indicates the PR's author has signed the CLA. label Jul 12, 2021
@knative-prow-robot knative-prow-robot added approved Indicates a PR has been approved by an approver from all required OWNERS files. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. area/API API objects and controllers area/autoscale area/monitoring area/networking labels Jul 12, 2021
@dprotaso dprotaso force-pushed the drop-pkg-serving-metrickeys branch from a1a2e60 to 1b7ea77 Compare July 12, 2021 20:12
@dprotaso dprotaso force-pushed the drop-pkg-serving-metrickeys branch from 1b7ea77 to e702f03 Compare July 12, 2021 20:28
@dprotaso
Copy link
Copy Markdown
Member Author

/assign @skonto @evankanderson @MontyCarter

This will let us drop the serving specific metrickeys in knative.dev/pkg

@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 12, 2021

Codecov Report

Merging #11658 (2975fe9) into main (9c2ec0f) will increase coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main   #11658      +/-   ##
==========================================
+ Coverage   87.86%   87.87%   +0.01%     
==========================================
  Files         190      190              
  Lines        9253     9253              
==========================================
+ Hits         8130     8131       +1     
  Misses        874      874              
+ Partials      249      248       -1     
Impacted Files Coverage Δ
pkg/activator/handler/metrics.go 90.90% <100.00%> (ø)
pkg/metrics/tags.go 89.28% <100.00%> (ø)
pkg/queue/request_metric.go 93.68% <100.00%> (ø)
pkg/reconciler/configuration/configuration.go 86.92% <0.00%> (+0.76%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9c2ec0f...2975fe9. Read the comment docs.

Copy link
Copy Markdown
Member

@evankanderson evankanderson left a comment

Choose a reason for hiding this comment

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

It looks like we're not changing any of the external metrics outputs here (intentionally at least), just moving constants from one place to another.

/lgtm
/approve

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

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dprotaso, evankanderson

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 merged commit be52cef into knative:main Jul 12, 2021
@dprotaso dprotaso deleted the drop-pkg-serving-metrickeys branch July 13, 2021 00:20
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/monitoring area/networking 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.

5 participants