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
Optimize utilization card by splitting items to smaller components ea… #3790
Optimize utilization card by splitting items to smaller components ea… #3790
Conversation
6bb98a6
to
a25914c
Compare
a25914c
to
b9eb356
Compare
/assign @mareklibra |
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.
Just minor comments, otherwise lgtm
humanizeValue={(v) => ({ string: `${v}`, value: v as number, unit: '' })} | ||
query={queries[HostQuery.NUMBER_OF_PODS].utilization} | ||
utilizationQuery={queries[HostQuery.NUMBER_OF_PODS].utilization} | ||
humanizeValue={React.useCallback( |
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.
Can we move useCallback()
up and just pass value here?
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.
fixed
namespace={projectName} | ||
position={PopoverPosition.top} | ||
<PrometheusUtilizationItem | ||
title="Network Transfer in/out" |
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.
IN/OUT
instead?
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.
we should have just a Network transfer
here, so I changed it accordingly.
…ch fetching its own data
b9eb356
to
09f1c35
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mareklibra, rawagner 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 |
/retest |
/retest Please review the full test history for this PR and help us cut down flakes. |
…ch fetching its own data