nvmetcli: port decoration coloring only if its enabled

Only color the port if its enabled (has subsystems or
referrals symlinks).

Also, don't display none trsvcid (like in the fc case).

Signed-off-by: Sagi Grimberg <sagi@grimberg.me>
Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmetcli: decorate namespace, subsystem and port UI

Display some info on the on the UI node.

Signed-off-by: Sagi Grimberg <sagi@grimberg.me>
Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmetcli: expose ls to dump UI configuration

for running: nvmetcli ls

Signed-off-by: Sagi Grimberg <sagi@grimberg.me>
Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmetcli: remove status method for the UIPortNode

nvmetcli defines a status method for the UIPortNode class but there is
no corresponding 'enable' file in configfs so typing in status will
always result in nvmetcli printing disbaled if one types 'status' in
the port node:

/> cd ports/1/
/ports/1> status
Status for /ports/1: disabled
/ports/1>

There are two possible ways for fixing this issue, provide a configfs
file to enable/disable the port node in the kernel or delete the
method in nvmetcli. Having a configfs file in the kernel is pointless,
as ports get enabled when symlinked to a namespace so zap the pointless status
method in user-space.

Signed-off-by: Johannes Thumshirn <jthumshirn@suse.de>
Reported-by: Hannes Reinecke <hare@suse.de>
Signed-off-by: Christoph Hellwig <hch@lst.de>

bump version to v0.5

add a make clean pass to bump-ver.sh

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmetcli: allow setting of the subsystem version

Signed-off-by: Johannes Thumshirn <jthumshirn@suse.de>
Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmetcli: allow setting of serial number number

Allow a user to set a serial number of a controller in configfs in order to
have persistent serials that survive a reboot of the target.

Signed-off-by: Johannes Thumshirn <jthumshirn@suse.de>
Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmetcli: add support for NS UUIDs

Add support for setting the Namespace Universally Unique Identifier in
a NVMe over Fabrics namespace.

Signed-off-by: Johannes Thumshirn <jthumshirn@suse.de>
Signed-off-by: Christoph Hellwig <hch@lst.de>

bump version to v0.4

Add fc transport support to nvmetcli

Add fc.json file for example FC configuration.
Small text update for FC in examples/documentation

Signed-off-by: James Smart <james.smart@broadcom.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmetcli: create json file's dir first if it doesn't exist

The saveconfig operation will be failed as the /etc/nvmet dir
doesn't exist by default, so create it first before save operation.

$ ./nvmetcli
/> saveconfig
[Errno 2] No such file or directory: '/etc/nvmet/config.json.temp'

Signed-off-by: Yi Zhang <yizhan@redhat.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmetcli: Install nvmetcli in /usr/sbin with setup.py

According to the README, one can install nvmetcli through setup.py.
When one tries this though, the nvmetcli isn't configured in system's
$PATH.

Adding nvmetcli to scripts argument in setup call does half of the job.
Then, in order to install it in the correct location in a maintanable
way it's introduced setup.cfg file which contains 'install_scripts'
parameter for 'install' command set to /usr/sbin.

Nothing changes on the .deb package creation, the .rpm package process
in the other hand was made simpler by lines that are now done by
setup.py + setup.cfg settings.

Signed-off-by: Mauro S. M. Rodrigues <maurosr@linux.vnet.ibm.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmetcli: fix Referral operation in shell

It's not possible to create or list a Referral under Port entity,
because Referral class is not imported and thus it fails:

/ports/1/referrals> ls

returning "'module' object has no attribute 'Referral'".

This is caused by changes introduced in:
a21ebd54f0 ("nvmetcli: Fix nvmet import").

This patch also fixes the attribute type helper expected for Referral's
portid attribute. It was 'int', causing the system to show error
messages like:

"'UIReferralNode' object has no attribute 'ui_type_int'".
The correct type is 'number'.

Reviewed-by: Guilherme G. Piccoli<gpiccoli@linux.vnet.ibm.com>
Reviewed-by: Sagi Grimberg <sagi@grimberg.me>
Signed-off-by: Mauro S. M. Rodrigues <maurosr@linux.vnet.ibm.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmetcli: fix Port and Host operations in shell

Operations against Port and Host entities are broken. When performing
operations like the follow:

/ports> create 1
/hosts> create 1

nvmetcli fails, saying the object doesn't have the attribute we're
trying to access, for instance:
'module' object has no attribute 'Port'

This patch fixes this issue by importing the classes Port and Host in
nvmet/__init__.py which was forgotten during the follow change:

a21ebd54f0 ("nvmetcli: Fix nvmet import").

Tested-by: Guilherme G. Piccoli <gpiccoli@linux.vnet.ibm.com>
Tested-by: Sagi Grimberg <sagi@grimberg.m>
Signed-off-by: Mauro S. M. Rodrigues <maurosr@linux.vnet.ibm.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>

bump version to v0.3

add a bump-ver.sh script to automate cutting releases

Signed-off-by: Christoph Hellwig <hch@lst.de>

update .gitignore for the auto-generated manpage formats

Signed-off-by: Christoph Hellwig <hch@lst.de>

add example configuration files

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmetcli: update README

Most usage information has moved to the manpage.  Also add a blurb
that bugs reports should go to the linux-nvme list.

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmetcli: Adding manpage/html generation

Signed-off-by: Jay Freyensee <james_p_freyensee@linux.intel.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmetcli: add python-six to rpm package building

From experience, if python-six isn't installed on a Fedora
system, hard-to-decipher errors can occur trying to use
nvmetcli.

Signed-off-by: Jay Freyensee <james.p.freyensee@intel.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmetcli: Fix nvmet import

We should just need to import nvmet, which will run __init__.py and import
just the public things from nvmet/nvme.py.

Signed-off-by: Andy Grover <agrover@redhat.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>

Use relative import in nvmet/__init__.py

This is supported by Python 2.7 and later.

Signed-off-by: Andy Grover <agrover@redhat.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmetcli: README description of discovery

Some blurbs on discovery and the connection
between using nvmetcli to set up an NVMe target and
what gets seen by an NVMe Host when accessing
the discovery controller.

Signed-off-by: Jay Freyensee <james_p_freyensee@linux.intel.com>

nvmetcli: More refined cleanup of README

Adds a few more tidbits to the README file, minus
discovery/referral info (still WIP).

Signed-off-by: Jay Freyensee <james_p_freyensee@linux.intel.com>
Reviewed-by: Sagi Grimberg <sagi@grimberg.me>
Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmetcli: Updated README to current functionality

Also added a referrals/discovery placeholder TBD section.

Signed-off-by: Jay Freyensee <james_p_freyensee@linux.intel.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>

add RPM packaging

Build and install tested on Fedora 23.

Signed-off-by: Christoph Hellwig <hch@lst.de>

add a systemd unit file

This restores the configuration when started and clears it when shut down.

Signed-off-by: Christoph Hellwig <hch@lst.de>

add Debian packaging

Signed-off-by: Christoph Hellwig <hch@lst.de>

move the default config into a /etc/nvmet/

no need to clutter /etc directly.

Signed-off-by: Christoph Hellwig <hch@lst.de>

rename README.md to README

Seems like our code examples aren't valid markdown..

Reported-by: Steve Wise <swise@chelsio.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>

update README and examples

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmetcli: add descriptions for all configfs attributes

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmet: remove broken automatic portid assignment

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmet,nvmetcli: support referrals

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmet: small fixes and cleanups

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmet,nvmetcli: support per-port subsystem enablement

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmet,nvmetcli: add support for host NQN based access control

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmetcli: fix a couple see also references

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvme: a couple typo fixes

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmet: fix attribute listing

We only want to split at the first underscore, not at any.

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmet, nvmetcli: add support for NVMe ports

Signed-off-by: Christoph Hellwig <hch@lst.de>

tests: check for invalid subsystem names

Signed-off-by: Christoph Hellwig <hch@lst.de>

tests: check that the nguid is properly saved and restored

Signed-off-by: Christoph Hellwig <hch@lst.de>

update default config file

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmet: improve enable semantics

Allow for a None state of the _enable variable, and use that fact to
automatically set up the enabled state in the primary Node class.

Signed-off-by: Christoph Hellwig <hch@lst.de>

pep8

Signed-off-by: Christoph Hellwig <hch@lst.de>

tests: initial version of the testsuite

This requires nose2 and can be run using make test

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmet: reject Namespace and Subsystems lookups without identifier

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmet: fix invalid access to self.subsystems in the Namespace constructor

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmet, nvmetcli: implement enable/disable semantics

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmetcli: improve error reporting

Report a nice error message in red instead of a python backtrace.

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmetcli: add description strings for the Namespace class

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmetcli: introduce UINode class

This allows calling saveconfig from each context, and factors some common
group handling code.

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmet: make attr_groups public for nvmetcli

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmetcli: remove the unused Controllers and Controller classes

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmetcli: remove unused DEFAULT_SAVE_FILE defintion

Signed-off-by: Christoph Hellwig <hch@lst.de>

Ensure internal buffers are flushed json after dumping json to disk

A power-outage, unexpected reboot, etc can lead to zero byte file
after a saveconfig.

Based on a rtslib-fb patch from Jon Magrini <jmagrini@redhat.com>.

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmet: fix up various docstrings

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmetcli: fix up the default config file to agree with the documentation

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmet: 0 is not a valid namespace ID

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmet: load the nvmet module if not already loaded

Signed-off-by: Christoph Hellwig <hch@lst.de>

nvmetcli: print a useful warning when not called as root user

Signed-off-by: Christoph Hellwig <hch@lst.de>

initial public release