ciao-cli: identity: return wrapped errors
ciao-cli: identity: return wrapped errors

The ciao-cli's identity code that handles tokens is never returning an
error.  These leads to strange runtime behaviors in the face of a bad
identity service:  A token related command would fail, but the function
would return a nil error.  The caller would then carry on with
uninitialized "" string variables, which of course led to more failures.
This obfuscates the root issue.

There is an errorf (in some cases of questionable quality) present into
the code adjacent to the nil return.  For simplicity, I've just moved
these existing messages into the errors.Wrap() in addition to the err
itself.  At that point we get the benefit of annotated errors in the
call stack and a non-nil error return so that callers don't carry on
thinking there was no error.

Fixes: #1068

Signed-off-by: Tim Pepper <8dbd764830e941d092a1cb7fa16423169acc0b04@linux.intel.com>

tpepper/ciao,ciao-project/ciao,obedmr/ciao,kaccardi/ciao,tpepper/ciao,markdryan/ciao,mrkz/ciao,matthewrsj/ciao,01org/ciao,rbradford/ciao,matthewrsj/ciao,erick0z/ciao,obedmr/ciao,01org/ciao,erick0z/ciao,mcastelino/ciao,matthewrsj/ciao,mcastelino/ciao,mcastelino/ciao,obedmr/ciao,mrkz/ciao,rbradford/ciao,markdryan/ciao,tpepper/ciao,kaccardi/ciao,erick0z/ciao,mrkz/ciao,ciao-project/ciao