Fixed all volume group run

Howdy,

Fixed the handling for "all" volume groups.
Added in handling for UNKNOWN status and exiting with a fair error message in that case.


Later.
I don't see any problems using -a option, can you send me what is the
error?

Maybe my little fixes in last commit will help...



On Tue, Jul 1, 2014 at 8:48 PM, Bill Carlson <notifications@github.com>
wrote:

> In check_vg_size:
>
> > @@ -85,7 +85,7 @@ checkArgValidity () {
> >
> >  #Does volume group actually exist?
> >  volumeGroupExists () {
> > -	local volGroup=$1
> > +	local volGroup=${@}
>
> On 07/01/2014 12:30 PM, veverjak wrote: Great thanks for another commit.
> Would you mind not changing this line? I've intended this fction to check
> only for first argument. If you would input multiple args, you wouldn't get
> positive result for neither. I will gladly merge rest to my repo.
> Howdy, I changed this for correctness, with past setup if it were passed
> multiple volume groups, it would check the first and ignore any additional
> entries passed. This is what allowed the "all" method to seem like it
> worked when in fact the wrong parameters were passed. Example: vgs system
> and backup, function was passed "backup system", it was dropping 'system',
> which is really wrong. Another alternative is to quote the parameter passed
> so that spaces do not make it extra parameters, so pass: "backup system"
> instead of backup system That way $1 is the correct (though something that
> will fail) parameter passed. Later,
> … <#146f3422a4021746_>
> -- Bill Carlson Anything is possible, given Time and Money.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/veverjak/check_vg_size/pull/6/files#r14422312>.
>