[PATCH configure] Added options for verbose/debug logging in cleanup

[Martyn Taylor]

From: Martyn Taylor

---
 bin/aeolus-cleanup |   27 ++++++++++++++++++++++++++-
 1 files changed, 26 insertions(+), 1 deletions(-)

diff --git a/bin/aeolus-cleanup b/bin/aeolus-cleanup
index a2ff45f..552b9ed 100644
--- a/bin/aeolus-cleanup
+++ b/bin/aeolus-cleanup
@@ -1,4 +1,28 @@
 #!/bin/sh
+usage()
+{
+cat << EOF
+
+USAGE:
+aeolus-cleanup [-d|--debug] [-h|--help] [-v|--verbose]
+
+OPTIONS:
+   -h | --help       Show this message.
+   -d | --debug      Debug logging mode.
+   -v | --verbose    Verbose logging mode.
+EOF
+}
+
+eval set -- `getopt -o hdv --long help,debug,verbose -- "$@"`
+while true ; do
+        case "$1" in
+                -h|--help) usage ; exit 1 ;  shift ;;
+                -d|--debug) LOGLEVEL="--debug" ; shift ;;
+                -v|--verbose) LOGLEVEL="--verbose" ; shift  ;;
+                --) shift ; break ;;
+                *)  usage ; exit 1 ;;
+        esac
+done

export FACTER_AEOLUS_ENABLE_HTTPS=true
 export FACTER_AEOLUS_ENABLE_SECURITY=false
@@ -6,5 +30,6 @@ puppet /usr/share/aeolus-configure/modules/aeolus/manifests/defaults.pp \
        --modulepath=/usr/share/aeolus-configure/modules/ \
        --external_nodes '/usr/sbin/aeolus-node cleanup' --node_terminus exec \
        --logdest=/var/log/aeolus-configure/aeolus-configure.log \
-       --logdest=console
+       --logdest=console \
+       $LOGLEVEL

[Francesco Vollero]

Nack, not seems to work properly, fix it please :)

Cheers, 
Francesco

On Thu, Jun 02, 2011 at 02:18:05PM +0100, [e-mail] wrote:
> From: Martyn Taylor
>
> bin/aeolus-cleanup |   27 ++++++++++++++++++++++++++-
>  1 files changed, 26 insertions(+), 1 deletions(-)
>
> diff --git a/bin/aeolus-cleanup b/bin/aeolus-cleanup
> index a2ff45f..552b9ed 100644
> --- a/bin/aeolus-cleanup
> +++ b/bin/aeolus-cleanup
> @@ -1,4 +1,28 @@
>  #!/bin/sh
> +usage()
> +{
> +cat << EOF
>
> +USAGE:
> +aeolus-cleanup [-d|--debug] [-h|--help] [-v|--verbose]
>
> +OPTIONS:
> -h | --help       Show this message.
> -d | --debug      Debug logging mode.
> -v | --verbose    Verbose logging mode.
>
> +EOF
> +}
>
> +eval set -- `getopt -o hdv --long help,debug,verbose -- "$@"`
> +while true ; do
>    case "$1" in
>            -h|--help) usage ; exit 1 ;  shift ;;
>            -d|--debug) LOGLEVEL="--debug" ; shift ;;
>            -v|--verbose) LOGLEVEL="--verbose" ; shift  ;;
>            --) shift ; break ;;
>            *)  usage ; exit 1 ;;
>    esac
>
> +done
>
> export FACTER_AEOLUS_ENABLE_HTTPS=true
>  export FACTER_AEOLUS_ENABLE_SECURITY=false
> @@ -6,5 +30,6 @@ puppet /usr/share/aeolus-configure/modules/aeolus/manifests/defaults.pp \
>         --modulepath=/usr/share/aeolus-configure/modules/ \
>         --external_nodes '/usr/sbin/aeolus-node cleanup' --node_terminus exec \
>         --logdest=/var/log/aeolus-configure/aeolus-configure.log \
>   --logdest=console
>   --logdest=console \
>   $LOGLEVEL
>
> -- 
> 1.7.4
>
> aeolus-devel mailing list
> [e-mail]
> https://fedorahosted.org/mailman/listinfo/aeolus-devel

[Justin Clift]

On 02/06/2011, at 11:18 PM, [e-mail] wrote:
<snip>
> bin/aeolus-cleanup |   27 ++++++++++++++++++++++++++-
> 1 files changed, 26 insertions(+), 1 deletions(-)
>
> diff --git a/bin/aeolus-cleanup b/bin/aeolus-cleanup
> index a2ff45f..552b9ed 100644
> --- a/bin/aeolus-cleanup
> +++ b/bin/aeolus-cleanup

<snip>
>   --logdest=/var/log/aeolus-configure/aeolus-configure.log \

That should be aeolus-cleanup.log shouldn't it?

Regards and best wishes,

Justin Clift

--
Aeolus Community Manager
http://www.aeolusproject.org

[Martyn Taylor]

Thanks Justin.

Yes it should, although, that change was not part of this patchset.  So I will send a separate fix for this bug.

Cheers

Martyn

----- Original Message -----
From: "Justin Clift" 
To: [e-mail]
Cc: [e-mail]
Sent: Friday, 3 June, 2011 3:22:08 AM
Subject: Re: [PATCH configure] Added options for verbose/debug logging in	cleanup

On 02/06/2011, at 11:18 PM, [e-mail] wrote:
<snip>
> bin/aeolus-cleanup |   27 ++++++++++++++++++++++++++-
> 1 files changed, 26 insertions(+), 1 deletions(-)
>
> diff --git a/bin/aeolus-cleanup b/bin/aeolus-cleanup
> index a2ff45f..552b9ed 100644
> --- a/bin/aeolus-cleanup
> +++ b/bin/aeolus-cleanup

<snip>
>   --logdest=/var/log/aeolus-configure/aeolus-configure.log \

That should be aeolus-cleanup.log shouldn't it?

Regards and best wishes,

Justin Clift

--
Aeolus Community Manager
http://www.aeolusproject.org

_______________________________________________
aeolus-devel mailing list
[e-mail]
https://fedorahosted.org/mailman/listinfo/aeolus-devel