-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
test framework: Enable metrics collection #3262
test framework: Enable metrics collection #3262
Conversation
46879ed
to
4d810b4
Compare
c7517d5
to
1475c22
Compare
Signed-off-by: Naadir Jeewa <[email protected]>
1475c22
to
f429fb5
Compare
/lgtm |
/milestone v0.3.7 |
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.
I'm not fully sure the metrics-erro.txt file is meaningful, but it won't hurt
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: fabriziopandini The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What this PR does / why we need it:
As part of investigating kubernetes-sigs/cluster-api-provider-aws#1764, found it useful to collect metrics during an e2e test. PR'ing the bits that I've used.
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #