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
Build gather-sysinfo #714
Build gather-sysinfo #714
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: MarSik 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 |
c0f8b1c
to
1903288
Compare
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.
importing gather-sysinfo
into the tree may and should be the chance for a good and long-needed cleanup, but I guess we don't have time/capacity for that, do we?
1903288
to
ad7619a
Compare
thanks for the updates! next time I recommend considering a verbatim import + fixing in the subsequent commits in order to preserve a good audit trail. In some cases this is unpractical but in this case it could have been. |
ad7619a
to
af73d89
Compare
8037f9d
to
e53bbff
Compare
e53bbff
to
93e37b6
Compare
/retest-required |
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.
The changes overall seem fine, but it is not clear to me how the collected information gets into must-gather. The commit message only states:
The gather-sysinfo binary is used by must-gather to collect
structured hardware information data about an OCP node.
Can you expand on that a bit how must-gather gets that information? Perhaps even in the commit message? Saw some nits so I pointed them out.
Edit: Oh, I missed openshift/must-gather#345 ... So my question is answered then.
93e37b6
to
17c0cc9
Compare
The gather-sysinfo binary is used by must-gather to collect structured hardware information data about an OCP node. This data is then also used by the Performance Profile Creator also present in this repository. A pciutils package is also added to the final NTO image, because it contains the lspci tool the must-gather script requires.
17c0cc9
to
dbd4d2f
Compare
@MarSik: all tests passed! Full PR test history. Your PR dashboard. 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. I understand the commands that are listed here. |
Thank you for the changes and the PR, looking forward to future customer's must-gathers. |
The gather-sysinfo binary is used by must-gather to collect
structured hardware information data about an OCP node.
This data is then also used by the Performance Profile Creator
also present in this repository.
Must-gather also needs the lspci binary from the pciutils package.