On 12/05/2014 08:53 AM, Michal Privoznik wrote:
On 04.12.2014 15:45, John Ferlan wrote:
> Recently while reviewing some documentation for iSCSI authentication
> description I found that virsh pool-{create|define}-as did not provide
> a mechanism to add the <auth> and/or <adapter> options on the command
> line - so I figured adding them would be simple. In the course of doing
> this I also noted a few typos on the formatstorage html page and (to
> say the last) a fairly incomplete or lacking man page description for
> pool-{create|define}-as.
>
>
> John Ferlan (4):
> docs: Fix a couple of typos on the storage pool html
> virsh.pod: Fix the pool-define-as and pool-create-as description
> virsh: Add auth options for pool-{create|define}-as
> virsh: Add adapter options for pool-{create|define}-as
>
> docs/formatstorage.html.in | 4 ++--
> tools/virsh-pool.c | 59 ++++++++++++++++++++++++++++++++++++++++++++--
> tools/virsh.pod | 53 ++++++++++++++++++++++++++++++++++++-----
> 3 files changed, 106 insertions(+), 10 deletions(-)
>
ACK series
Michal
Tks - now pushed (with a fix for noted typo in 1/4 commit message)
John