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

manifests: Addition of nvme-cli #564

Merged
merged 1 commit into from
Aug 14, 2020

Conversation

nasirhm
Copy link
Contributor

@nasirhm nasirhm commented Aug 14, 2020

It adds nvme-cli to provide an interface that allows host software utilities to communicate with nvme disks.

The missing nvme binary was brought up over in a tracker issue [1] and also discussed during the 08/12 meeting [2] where there was agreement that we should include this utility in the host.

[1] coreos/fedora-coreos-tracker#601
[2] https://lists.fedoraproject.org/archives/list/coreos@lists.fedoraproject.org/message/5E66QVX7F6HRZXMY4GEWQW6SU3GUOQ3G/

Copy link
Member

@dustymabe dustymabe left a comment

Choose a reason for hiding this comment

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

Nice work @nasirhm - let's squash those two commits into one if possible.

@@ -157,6 +157,8 @@ packages:
- kbd
# Parsing/Interacting with JSON data
- jq
# nvme-cli for udev rules
Copy link
Member

Choose a reason for hiding this comment

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

Since we're adding it independent of the udev rules let's just make the comment more generic. Something like:

Suggested change
# nvme-cli for udev rules
# nvme-cli for managing nvme disks

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure, Thank you for the review @dustymabe .

Copy link
Member

@dustymabe dustymabe left a comment

Choose a reason for hiding this comment

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

LGTM

@dustymabe dustymabe merged commit ca66c68 into coreos:testing-devel Aug 14, 2020
c4rt0 pushed a commit to c4rt0/fedora-coreos-config that referenced this pull request Mar 27, 2023
docs/faq: update multipath instructions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants