[PATCH nvme-cli v2 2/2] nvme.spec/debian: Auto generate host nqn as part of install

[J Freyensee]

On Thu, 2016-11-17 at 09:51 -0800, J Freyensee wrote:
> On Thu, 2016-11-17 at 05:36 -0800, Christoph Hellwig wrote:
> > 
> > On Thu, Nov 17, 2016 at 02:54:48PM +0200, Sagi Grimberg wrote:
> > > 
> > > 
> > > The installation will generate a hostnqn and store it in
> > > /etc/nvme/hostnqn file (in case it doesn't exist).
> > > This file will be removed upon uninstallation (purge on for
> > > debian).
> > > 
> > > Signed-off-by: Sagi Grimberg <sagi at grimberg.me>
> > > ---
> > >  debian/postinst | 20 ++++++++++++++++++++
> > >  debian/postrm   | 10 ++++++++++
> > >  nvme.spec.in    | 17 +++++++++++++++++
> > >  3 files changed, 47 insertions(+)
> > >  create mode 100644 debian/postinst
> > >  create mode 100644 debian/postrm
> > > 
> > > diff --git a/debian/postinst b/debian/postinst
> > > new file mode 100644
> > > index 000000000000..b258cf569c3e
> > > --- /dev/null
> > > +++ b/debian/postinst
> > > @@ -0,0 +1,20 @@
> > > +#!/bin/sh
> > > +
> > > +set -e
> > > +
> > > +case "$1" in
> > > +    configure|install)
> > > +	if [ ! -f /etc/nvme/hostnqn ]; then
> > > +		install -D /dev/null /etc/nvme/hostnqn
> > > +		echo $(nvme gen-hostnqn) > /etc/nvme/hostnqn
> > 
> > The $() syntax is a bash-ish.  To work with a traditional bourne
> > shell
> > it needs to be `nvme gen-hostnqn`
> > 
> > > 
> > > 
> > > +		if [ ! -n "$(ls -A /etc/nvme)" ]; then
> > 
> > Same here.
> > 
> > > 
> > > 
> > > +			rm -rf /etc/nvme
> > > +		fi
> > 
> > But maybe you should just do a
> > 
> > 		rmdir /etc/nvme >/dev/null
> 


Nevermind, I'm dumb, Nack the comment, I didn't read that right from
the comments and code separated...more coffee :-/



> Please don't do a remove of the directory, just remove the files
> created from this patch (/etc/nvme/hostnqn).  As I mentioned earlier,
> someone for example could have independently created a discovery.conf
> file in /etc/nvme and might have not wanted that file removed when
> this
> is uninstalled.  If the directory was created from this patch then
> either leave the directory /etc/nvme behind or test to see if
> /etc/nvme
> is empty after removing /etc/nvme/hostnqn before attempting to remove
> the directory /etc/nvme.
> 
> 
> > 
> > 
> > instead?
> 
> _______________________________________________
> Linux-nvme mailing list
> Linux-nvme at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-nvme