Re: [Cluster-devel] [cluster.git/STABLE32][PATCH] xml: ccs_update_schema: be verbose about extraction fail

2014-04-29 Thread Fabio M. Di Nitto
ACK

Fabio

On 4/29/2014 11:30 PM, Jan Pokorný wrote:
> Previously, the distillation of resource-agents' metadata could fail
> from unexpected reasons without any evidence ever being made, unlike
> in case of fence-agents.  Also "no metadata" and "issue with their
> extraction" will allegedly yield the same outcome, so it is reflected
> in the comments being emitted to the schema for both sorts of agents.
> 
> Signed-off-by: Jan Pokorný 
> ---
>  config/tools/xml/ccs_update_schema.in | 7 +--
>  1 file changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/config/tools/xml/ccs_update_schema.in 
> b/config/tools/xml/ccs_update_schema.in
> index 98ed885..b63c987 100644
> --- a/config/tools/xml/ccs_update_schema.in
> +++ b/config/tools/xml/ccs_update_schema.in
> @@ -215,6 +215,9 @@ generate_ras() {
>   lecho " ras: processing $(basename $i)"
>   $i meta-data 2>/dev/null | xsltproc $rngdir/ra2rng.xsl - >> \
>   "$outputdir/resources.rng.cache" 2>/dev/null
> + [ "$?" != 0 ] && \
> + echo "  " >> "$outputdir/resources.rng.cache"
>   done
>   cat $rngdir/resources.rng.mid >> "$outputdir/resources.rng.cache"
>   lecho " ras: generating ref data"
> @@ -301,8 +304,8 @@ generate_fas() {
>   xsltproc $rngdir/fence2rng.xsl - >> \
>   "$outputdir/fence_agents.rng.cache" 2>/dev/null
>   [ "$?" != 0 ] && \
> - echo "  " >> \
> - "$outputdir/fence_agents.rng.cache"
> + echo "  " >> "$outputdir/fence_agents.rng.cache"
>   done
>   cat $rngdir/fence.rng.tail >> "$outputdir/fence_agents.rng.cache"
>  }
> 



[Cluster-devel] [cluster.git/STABLE32][PATCH] xml: ccs_update_schema: be verbose about extraction fail

2014-04-29 Thread Jan Pokorný
Previously, the distillation of resource-agents' metadata could fail
from unexpected reasons without any evidence ever being made, unlike
in case of fence-agents.  Also "no metadata" and "issue with their
extraction" will allegedly yield the same outcome, so it is reflected
in the comments being emitted to the schema for both sorts of agents.

Signed-off-by: Jan Pokorný 
---
 config/tools/xml/ccs_update_schema.in | 7 +--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/config/tools/xml/ccs_update_schema.in 
b/config/tools/xml/ccs_update_schema.in
index 98ed885..b63c987 100644
--- a/config/tools/xml/ccs_update_schema.in
+++ b/config/tools/xml/ccs_update_schema.in
@@ -215,6 +215,9 @@ generate_ras() {
lecho " ras: processing $(basename $i)"
$i meta-data 2>/dev/null | xsltproc $rngdir/ra2rng.xsl - >> \
"$outputdir/resources.rng.cache" 2>/dev/null
+   [ "$?" != 0 ] && \
+   echo "  " >> "$outputdir/resources.rng.cache"
done
cat $rngdir/resources.rng.mid >> "$outputdir/resources.rng.cache"
lecho " ras: generating ref data"
@@ -301,8 +304,8 @@ generate_fas() {
xsltproc $rngdir/fence2rng.xsl - >> \
"$outputdir/fence_agents.rng.cache" 2>/dev/null
[ "$?" != 0 ] && \
-   echo "  " >> \
-   "$outputdir/fence_agents.rng.cache"
+   echo "  " >> "$outputdir/fence_agents.rng.cache"
done
cat $rngdir/fence.rng.tail >> "$outputdir/fence_agents.rng.cache"
 }
-- 
1.9.0