Bug 1742764 - nvme-cli-1.10.1 is available
Summary: nvme-cli-1.10.1 is available
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: nvme-cli
Version: rawhide
Hardware: All
OS: Linux
unspecified
high
Target Milestone: ---
Assignee: Andy Lutomirski
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1781928 1794615
TreeView+ depends on / blocked
 
Reported: 2019-08-16 20:17 UTC by Upstream Release Monitoring
Modified: 2020-04-21 17:17 UTC (History)
9 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2020-04-21 17:17:42 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
[patch] Update to 1.10.1 (#1742764) (1.00 KB, patch)
2020-01-08 10:07 UTC, Upstream Release Monitoring
no flags Details | Diff

Description Upstream Release Monitoring 2019-08-16 20:17:59 UTC
Latest upstream release: 1.9
Current version/release in rawhide: 1.8.1-2.fc31
URL: https://github.com/linux-nvme/nvme-cli

Please consult the package updates policy before you issue an update to a stable branch: https://fedoraproject.org/wiki/Updates_Policy


More information about the service that created this bug can be found at: https://fedoraproject.org/wiki/Upstream_release_monitoring


Please keep in mind that with any upstream change, there may also be packaging changes that need to be made. Specifically, please remember that it is your responsibility to review the new version to ensure that the licensing is still correct and that no non-free or legally problematic items have been added upstream.


Based on the information from anitya: https://release-monitoring.org/project/9074/

Comment 1 Upstream Release Monitoring 2019-08-16 20:18:03 UTC
An unexpected error occurred while creating the scratch build and has been automatically reported. Sorry!

Comment 2 David Milburn 2019-09-16 20:50:07 UTC
Hi Andy,

Would you please pull v1.9 into Fedora? Thanks.

Comment 3 David Milburn 2019-10-02 17:21:50 UTC
Hi Andy,

Would you please pull in v1.9? 

It needs to be Fedora before we can use it in RHEL releases.

Thanks for you help,
David

Comment 4 Andy Lutomirski 2019-10-02 17:41:26 UTC
I pushed out a spec update in the master branch and kicked off a scratch build.  One thing I'm a bit uneasy about: the new dracut rule references the new udev rule, but how exactly are the systemd units that are referenced by the udev rule expected to exist in the dracut environment?

Comment 5 David Milburn 2019-10-02 18:02:44 UTC
(In reply to Andy Lutomirski from comment #4)
> I pushed out a spec update in the master branch and kicked off a scratch
> build.  One thing I'm a bit uneasy about: the new dracut rule references the
> new udev rule, but how exactly are the systemd units that are referenced by
> the udev rule expected to exist in the dracut environment?

Hi James, will a change to dracut be needed? Thanks.

Comment 6 Andy Lutomirski 2019-10-02 21:04:32 UTC
I'd also like to confirm that everything still works with /etc/nvme/hostid and /etc/nvme/hostnqn removed.  I *could* autogenerate them on install, but I'd much rather see nvme-cli just work correctly without those files.  See:

https://github.com/linux-nvme/nvme-cli/pull/415

Ideally, the code that reads /etc/nvme/hostid and /etc/nvme/hostnqn would notice that the files don't exist (or perhaps notice that they exist and contain a string like 'auto') and generate the contents on the fly.

Comment 7 Andy Lutomirski 2019-10-02 21:08:07 UTC
Here's what I have right now.  I doubt it works, but it's probably not a regression from 1.8.  I could attempt to patch it up, but I'd rather see an upstream fix that makes everything work without hostid and hostnqn files.  This would be much nicer for Silverblue or whatever it's called these days.

https://src.fedoraproject.org/rpms/nvme-cli/c/b308a8c2587de5bf4eebd960072045fdd012a9cb?branch=master

Comment 8 David Milburn 2019-10-03 15:22:04 UTC
Hi Andy,

(In reply to Andy Lutomirski from comment #6)
> I'd also like to confirm that everything still works with /etc/nvme/hostid
> and /etc/nvme/hostnqn removed.  I *could* autogenerate them on install, but
> I'd much rather see nvme-cli just work correctly without those files.  See:
> 
> https://github.com/linux-nvme/nvme-cli/pull/415
> 
> Ideally, the code that reads /etc/nvme/hostid and /etc/nvme/hostnqn would
> notice that the files don't exist (or perhaps notice that they exist and
> contain a string like 'auto') and generate the contents on the fly.

The hostid and hostnqn are used for NVMe over Fabrics, the original request
to populate /etc/nvme on installation was so they didn't change across reboots.
To auto-generate would require more nvme-cli changes, they would need to be
persistent. Thanks.

Comment 9 Andy Lutomirski 2019-10-03 18:52:00 UTC
(In reply to David Milburn from comment #8)
> The hostid and hostnqn are used for NVMe over Fabrics, the original request
> to populate /etc/nvme on installation was so they didn't change across
> reboots.
> To auto-generate would require more nvme-cli changes, they would need to be
> persistent. Thanks.


Like this?

https://github.com/linux-nvme/nvme-cli/pull/588

Comment 10 David Milburn 2019-10-10 18:25:30 UTC
Hi Andy,

(In reply to Andy Lutomirski from comment #9)
> (In reply to David Milburn from comment #8)
> > The hostid and hostnqn are used for NVMe over Fabrics, the original request
> > to populate /etc/nvme on installation was so they didn't change across
> > reboots.
> > To auto-generate would require more nvme-cli changes, they would need to be
> > persistent. Thanks.
> 
> 
> Like this?
> 
> https://github.com/linux-nvme/nvme-cli/pull/588

Looks like Keith pulled the patch in, I setup a couple of Fedora
systems, pulled the latest from https://github.com/linux-nvme/nvme-cli

On target system

[root@rdma-dev-00 nvme-cli (master)]$ uname -a
Linux rdma-dev-00.lab.bos.redhat.com 5.3.2-300.fc31.x86_64 #1 SMP Tue Oct 1 20:44:46 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
[root@rdma-dev-00 nvme-cli (master)]$ lsblk
NAME                          MAJ:MIN RM   SIZE RO TYPE MOUNTPOINT
sda                             8:0    0 465.8G  0 disk 
├─sda1                          8:1    0     1G  0 part /boot
└─sda2                          8:2    0 464.8G  0 part 
  └─fedora_rdma--dev--00-home 253:2    0 761.5G  0 lvm  /home
nvme0n1                       259:0    0 372.6G  0 disk 
└─nvme0n1p1                   259:1    0 372.6G  0 part 
  ├─fedora_rdma--dev--00-root 253:0    0    70G  0 lvm  /
  ├─fedora_rdma--dev--00-swap 253:1    0   5.9G  0 lvm  [SWAP]
  └─fedora_rdma--dev--00-home 253:2    0 761.5G  0 lvm  /home
[root@rdma-dev-00 nvme-cli (master)]$ ./nvme show-hostnqn
nqn.2014-08.org.nvmexpress:uuid:3b1e25ab846d402d943f29ec6c2c318a

On initiator system
[root@rdma-dev-01 nvme-cli (master)]$ uname -a
Linux rdma-dev-01.lab.bos.redhat.com 5.3.2-300.fc31.x86_64 #1 SMP Tue Oct 1 20:44:46 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
[root@rdma-dev-01 nvme-cli (master)]$ ./nvme discover -t rdma -a 172.31.0.230 -q nqn.2014-08.org.nvmexpress:uuid:3b1e25ab846d402d943f29ec6c2c318a

Discovery Log Number of Records 1, Generation counter 2
=====Discovery Log Entry 0======
trtype:  rdma
adrfam:  ipv4
subtype: nvme subsystem
treq:    not specified, sq flow control disable supported
portid:  1
trsvcid: 4420
subnqn:  testnqn
traddr:  172.31.0.230
rdma_prtype: not specified
rdma_qptype: connected
rdma_cms:    rdma-cm
rdma_pkey: 0x0000

[root@rdma-dev-01 nvme-cli (master)]$ dmesg
[  704.437714] nvme nvme1: new ctrl: NQN "nqn.2014-08.org.nvmexpress.discovery", addr 172.31.0.230:4420
[  704.448925] nvme nvme1: Removing ctrl: NQN "nqn.2014-08.org.nvmexpress.discovery"

On target system:

$ dmesg
160960.190926] nvmet: creating controller 2 for subsystem nqn.2014-08.org.nvmexpress.discovery for NQN nqn.2014-08.org.nvmexpress:uuid:3b1e25ab846d402d943f29ec6c2c318a.

Comment 11 David Milburn 2019-10-14 22:45:54 UTC
Hi Andy,

(In reply to Andy Lutomirski from comment #4)
> I pushed out a spec update in the master branch and kicked off a scratch
> build.

Is your scratch build available for testing? Will you be updating the
Fedora version soon? Thanks.

Comment 12 Andy Lutomirski 2019-10-18 18:22:52 UTC
There are still some issues:

https://github.com/linux-nvme/nvme-cli/pull/594

I'd rather do this right rather than sending some not-entirely-done stuff to Fedora.

Comment 13 David Milburn 2019-11-18 23:09:26 UTC
Hi Andy,

(In reply to Andy Lutomirski from comment #12)
> There are still some issues:
> 
> https://github.com/linux-nvme/nvme-cli/pull/594
> 
> I'd rather do this right rather than sending some not-entirely-done stuff to
> Fedora.

Any progress towards getting v1.9 pulled into Fedora? Thanks.

Comment 14 David Milburn 2019-12-16 21:46:12 UTC
Hello Andy,

Any progress towards getting v1.9 pulled into Fedora? Thanks.

Comment 15 Andy Lutomirski 2019-12-23 08:06:46 UTC
I'm submitting builds of my somewhat nerfed v1.9.

Any chance you could look at the github issue above or my posting to linux-nvme and comment on it?  I'd really like to get this fixed for real upstream and be done with it.

Comment 16 Andy Lutomirski 2019-12-24 02:34:11 UTC
Sigh, temporarily blocked due to a mysterious koji problem.

Comment 17 Upstream Release Monitoring 2020-01-08 10:07:03 UTC
Latest upstream release: 1.10.1
Current version/release in rawhide: 1.9-1.fc32
URL: https://github.com/linux-nvme/nvme-cli

Please consult the package updates policy before you issue an update to a stable branch: https://fedoraproject.org/wiki/Updates_Policy


More information about the service that created this bug can be found at: https://fedoraproject.org/wiki/Upstream_release_monitoring


Please keep in mind that with any upstream change, there may also be packaging changes that need to be made. Specifically, please remember that it is your responsibility to review the new version to ensure that the licensing is still correct and that no non-free or legally problematic items have been added upstream.


Based on the information from anitya: https://release-monitoring.org/project/9074/

Comment 18 Upstream Release Monitoring 2020-01-08 10:07:08 UTC
Created attachment 1650627 [details]
[patch] Update to 1.10.1 (#1742764)

Comment 19 Upstream Release Monitoring 2020-01-08 10:10:08 UTC
the-new-hotness/release-monitoring.org's scratch build of nvme-cli-1.10.1-1.fc29.src.rpm for rawhide failed http://koji.fedoraproject.org/koji/taskinfo?taskID=40274956

Comment 20 David Milburn 2020-01-08 15:42:48 UTC
Hi Andy, can you pull in Keith's latest v1.10.1? This would keep
up-to-date with the latest fixes. Thanks.

Comment 21 Andy Lutomirski 2020-03-20 04:17:10 UTC
I submitted updates.

(The updates aren't linked to the bug because Bodhi appears to be failing to locate the bug.  Maybe it's just me, but the Fedora tooling seems to be getting less and less functional, and I'm getting a bit sick of packaging Fedora given the state of the tooling.  It should not take minutes to find my builds and apparently forever to find bugs when I go to bodhi.fedoraproject.org.)

Comment 22 Andy Lutomirski 2020-04-21 17:17:42 UTC
Closing manually because the magic Fedora infrastructure didn't associate this bug with the actual update.


Note You need to log in before you can comment on or make changes to this bug.