On 04/07/2014, at 7:30 AM, Santosh Pradhan wrote:
> Thanks guys for looking into this. I am just wondering how this passed the
> regression before Niels could merged this in?
It was due to stupidity on my part. ;)
Was adjusting the bash script in jenkins the other day, attempting
to get the console output nicer looking. So, added a few echo
statements in places, attempting to space things out.
Previous (working code) was like this:
...
sudo -E bash -x /opt/qa/regression.sh
RET=$?
if [ $RET = 0 ]; then
V="+1"
VERDICT="SUCCESS"
else
V="-1"
VERDICT="FAILED"
fi
...
With the brilliant addition of echo statements in exactly
the wrong place, it became:
...
echo
echo
sudo -E bash -x /opt/qa/regression.sh
echo
echo
RET=$?
if [ $RET = 0 ]; then
V="+1"
VERDICT="SUCCESS"
else
V="-1"
VERDICT="FAILED"
fi
...
... and was using the return code from the echo statements.
Not my brightest moment. ;)
+ Justin
--
GlusterFS - http://www.gluster.org
An open source, distributed file system scaling to several
petabytes, and handling thousands of clients.
My personal twitter: twitter.com/realjustinclift
_______________________________________________
Gluster-devel mailing list
Gluster-devel@gluster.org
http://supercolony.gluster.org/mailman/listinfo/gluster-devel
