Skip to content
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

Add SR-IOV gather script #297

Merged
merged 1 commit into from
May 5, 2022

Conversation

liornoy
Copy link
Contributor

@liornoy liornoy commented Apr 17, 2022

This PR adds SR-IOV gather script.
Gathering the following:

  • openshift-sriov-network-operator namespace.
    • sriovnetworknodepolicies.
    • sriovnetworknodestates.
    • sriovnetworkpoolconfigs.
    • sriovnetworks.
    • sriovoperatorconfigs.
    • sriovibnetworks.
  • The following logs from the sriov-config-daemon:
    • /tmp/sno-initial-node-state.json.
    • /proc/cmdline.
    • dmseg.
    • ip link.
    • var/multus.log (if exists).
    • netns and output of ip a fron every ns.
    • NIC firmware and driver from ethtool.

Copy link
Contributor

@RickJWagner RickJWagner left a comment

Choose a reason for hiding this comment

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

Looks good. Approved.

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 18, 2022
@zshi-redhat
Copy link

/cc @bn222

@openshift-ci openshift-ci bot requested a review from bn222 April 19, 2022 08:29
this script is gathering the following:
* openshift-sriov-network-operator namespace.
* the following resources:
- sriovnetworknodepolicies.
- sriovnetworknodestates.
- sriovnetworkpoolconfigs.
- sriovnetworks.
- sriovoperatorconfigs.
- sriovibnetworks.
* the following logs from the sriov-config-daemon:
- /tmp/sno-initial-node-state.json.
- /proc/cmdline.
- dmseg.
- ip link.
- var/multus.log (if exists).
- netns and output of `ip a` fron every ns.
- NIC firmware and driver from ethtool.
@bn222
Copy link

bn222 commented May 5, 2022

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label May 5, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented May 5, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bn222, liornoy, RickJWagner

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci
Copy link
Contributor

openshift-ci bot commented May 5, 2022

@liornoy: 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.

@openshift-merge-robot openshift-merge-robot merged commit 5339ac7 into openshift:master May 5, 2022
zeeke added a commit to zeeke/sriov-network-operator-1 that referenced this pull request May 31, 2022
`must-gather` logic has been moved to
openshift/must-gather repository with
openshift/must-gather#297

Signed-off-by: Andrea Panattoni <[email protected]>
zeeke added a commit to zeeke/sriov-network-operator that referenced this pull request Jun 20, 2022
`must-gather` logic has been moved to
openshift/must-gather repository with
openshift/must-gather#297

Signed-off-by: Andrea Panattoni <[email protected]>
novacain1 pushed a commit to novacain1/sriov-network-operator that referenced this pull request Jul 5, 2022
`must-gather` logic has been moved to
openshift/must-gather repository with
openshift/must-gather#297

Signed-off-by: Andrea Panattoni <[email protected]>
@liornoy liornoy deleted the add-sriov-gather branch August 24, 2022 11:56
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. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants