-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
[dependencies] update prometheus/client_golang v1.16.0 to v1.18.0 #122551
Conversation
/sig instrumentation |
/test pull-kubernetes-unit |
/remove-sig api-machinery |
@alexzielenski: Those labels are not set on the issue: In response to this:
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. |
28bdc63
to
3fb566f
Compare
/priority important-longterm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mechanical changes look good, one question about the new CreatedTimestamp field and whether we need to do anything with it or if just ignoring it is correct
@@ -585,7 +585,7 @@ func TestGetHistogramVecFromGatherer(t *testing.T) { | |||
vec.WithLabelValues("value1-1", "value2-1").Observe(4.5) | |||
metricName := fmt.Sprintf("%s_%s_%s", HistogramOpts.Namespace, HistogramOpts.Subsystem, HistogramOpts.Name) | |||
histogramVec, _ := GetHistogramVecFromGatherer(gather, metricName, tt.lvMap) | |||
if diff := cmp.Diff(tt.wantVec, histogramVec, cmpopts.IgnoreFields(dto.Histogram{}, "state", "sizeCache", "unknownFields"), cmpopts.IgnoreFields(dto.Bucket{}, "state", "sizeCache", "unknownFields")); diff != "" { | |||
if diff := cmp.Diff(tt.wantVec, histogramVec, cmpopts.IgnoreFields(dto.Histogram{}, "state", "sizeCache", "unknownFields", "CreatedTimestamp"), cmpopts.IgnoreFields(dto.Bucket{}, "state", "sizeCache", "unknownFields")); diff != "" { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is the meaning of the new CreatedTimestamp field, and are there any implications for our lazy initialization of metrics? cc @logicalhan
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ty-dc please address this review comment and rebase the PR, thanks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the reminder. this should be a new feature introduced in v1.17.0.
I should not ignore the creation time, but when doing the comparison the following error will appear
--- FAIL: TestGetHistogramVecFromGatherer/filter_with_one_label (0.00s)
metrics_test.go:595: Got unexpected HistogramVec (-want +got):
testutil.HistogramVec{
&{
Histogram: &io_prometheus_client.Histogram{
... // 3 ignored and 2 identical fields
SampleSum: &1.5,
Bucket: {&{CumulativeCount: &0, UpperBound: &0.5}, &{CumulativeCount: &1, UpperBound: &2}, &{CumulativeCount: &1, UpperBound: &5}},
CreatedTimestamp: ×tamppb.Timestamp{
... // 3 ignored fields
Seconds: 1706235962,
- Nanos: 318936000,
+ Nanos: 319122000,
},
Schema: nil,
ZeroThreshold: nil,
... // 8 identical fields
},
},
I tried ignoring "CreatedTimestamp.Nanos", Is it okay to do this? Can you give me some advice? thanks.
3fb566f
to
9eb7459
Compare
9eb7459
to
fefee46
Compare
/test pull-kubernetes-integration |
fefee46
to
622c72e
Compare
0c08de0
to
12cbaaa
Compare
Signed-off-by: tao.yang <tao.yang@daocloud.io>
12cbaaa
to
7cecc49
Compare
Out of date, a new version has been released. https://github.com/prometheus/client_golang/releases/tag/v1.19.0 |
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
This PR updates prometheus/client_golang v1.16.0 to v1.18.0
Changes in v1.17.0
Changes in v1.18.0
Which issue(s) this PR fixes:
Fixes # None
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: