------- Comment From [email protected] 2016-01-28 10:58 EDT-------
(In reply to comment #5)
> So I uploaded lsvpd to the xenial archive with this change; but then after
> testing, I see that it merely trades one set of errors for another.
>
> - iprconfig is not shipped as /usr/sbin/iprconfig in Ubuntu, but as
> /sbin/iprconfig; so adding the dependency does not remove the error.

Hmmm. This is one of the challenge we constantly face with distros. I
should have verified this :-(

Can you tell me the criteria to place commands to /sbin and /usr/sbin ?
Because I see Ubuntu placed few commands under /sbin/ and few under
/usr/sbin

ubuntu@lep8d:$ which vpdupdate
/usr/sbin/vpdupdate
ubuntu@lep8d:/home/hegdevasant$ which iprconfig
/sbin/iprconfig

Also can you please fix this in Ubuntu tree -OR- do you want us to fix
it in upstream before you fix?

> - adjusting paths so that vpdupdate can find iprconfig results in a new
> stream of other error messages when invoking vpdupdate:
> iprconfig[91936]: Could not open /dev/sg28. No such file or directory
> iprconfig[91936]: Could not open /dev/sg28. No such file or directory
> iprconfig[91936]: Could not open /dev/sg28. No such file or directory
> iprconfig[91936]: Could not open /dev/sg28. No such file or directory
> iprconfig[91936]: Could not open /dev/sg28. No such file or directory
> iprconfig[91936]: Could not open /dev/sg25. No such file or directory
> iprconfig[91936]: Could not open /dev/sg25. No such file or directory
> iprconfig[91936]: Could not open /dev/sg25. No such file or directory
> iprconfig[91936]: Could not open /dev/sg25. No such file or directory
> iprconfig[91936]: Could not open /dev/sg25. No such file or directory
> iprconfig[91936]: Could not open /dev/sg25. No such file or directory
> iprconfig[91936]: Could not open /dev/sg30. No such file or directory
> iprconfig[91936]: Could not open /dev/sg30. No such file or directory
> iprconfig[91936]: Could not open /dev/sg30. No such file or directory
> iprconfig[91936]: Could not open /dev/sg30. No such file or directory
> iprconfig[91936]: Could not open /dev/sg30. No such file or directory
> iprconfig[91936]: Could not open /dev/sg30. No such file or directory
>
> (etc)
>
> - after running vpdupdate with fixed paths, the output of 'lsvpd' is almost
> precisely identical from before.  Only two lines of diference in the output,
> and it appears that the new output contains less information rather than
> more?

Below output is expected.. Basically we use sg utility to get VPD
information for disks/enclosures. Then we run iprconfig to fill missing
information (if any) and update location code information.

>
> @@ -422,7 +422,7 @@
> *XB 3
> *XT 0
> *XL 0
> -*YL U78CB.001.WZS008E-P1-C14-B3-T0-L0
> +*YL U78CB.001.WZS008E-P1-C14
> *FC
> *DS -SCSI Enclosure Services Device
> *AX sg27
> @@ -970,7 +970,7 @@
> *XB 3
> *XT 0
> *XL 0
> -*YL U78CB.001.WZS008E-P1-C15-B3-T0-L0
> +*YL U78CB.001.WZS008E-P1-C15
> *FC
> *DS -SCSI Disk Drive
> *AX sg39
>
> Perhaps because this particular system doesn't support ipr scsi?  Or
> perhaps, it's because if iprutils aren't installed, one can assume that ipr
> scsi isn't configured and therefore iprconfig is not going to provide any
> additional information...
>
> So I'm not sure that the dependency on iprutils is actually required.  But I
> am reasonably certain that vpdupdate should be fixed to suppress error
> output from iprconfig instead of displaying it.

Agreed. I believe most of the errors are fixed with latest libvpd
(v2.2.5) .. But I suspect we still have some errors like below .. which
needs to be fixed.

Missing device: sg11
Missing device: sg10
Missing device: sg14
Missing device: sg12
Missing device: sg13

>
> Please consider whether making this change to lsvpd upstream is a better fix.

Agreed. Thanks for the suggestion.

-Vasant

------- Comment From [email protected] 2016-01-28 10:59 EDT-------
(In reply to comment #6)
> This bug was fixed in the package lsvpd - 1.7.6-0ubuntu2
>
> ---------------
> lsvpd (1.7.6-0ubuntu2) xenial; urgency=medium
>
> * Add dependency on iprutils, since iprconfig is needed.  LP: #1537116.
>

Thanks!

Did you fix iprconfig path as well?

-Vasant

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1537116

Title:
  lsvpd: Missing package dependency

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/lsvpd/+bug/1537116/+subscriptions

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to