<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=koi8-r">
<style type="text/css" style="display:none;"><!-- P {margin-top:0;margin-bottom:0;} --></style>
</head>
<body dir="ltr">
<div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Arial,Helvetica,sans-serif;">
<p><span style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 16px;">Well, yes, it's possible to add that. But our testing lib has no mechanism to properly set env vars, so we'll have to pass them in each system() call along with make command to
start tests.</span><br>
</p>
<br>
<br>
<div style="color: rgb(0, 0, 0);">
<div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>От:</b> Pavel Emelianov<br>
<b>Отправлено:</b> 25 ноября 2016 г. 18:00<br>
<b>Кому:</b> Cyrill Gorcunov; crml<br>
<b>Копия:</b> Vitaly Ostrosablin; Andrey Vagin<br>
<b>Тема:</b> Re: [CRIU] [PATCH 3/3] test: Handle parseargs for VZ7 qa envinronment</font>
<div> </div>
</div>
</div>
<font size="2"><span style="font-size:10pt;">
<div class="PlainText">On 11/25/2016 05:20 PM, Cyrill Gorcunov wrote:<br>
> From: Vitaly Ostrosablin <vostrosablin@virtuozzo.com><br>
> <br>
> Test seem to assume that zdtm lib directory is on PATH, but if it's not<br>
<br>
... then it should be added there by the caller ;)<br>
<br>
> - calling for parseargs.sh would fail (if cwd is static). Therefore,<br>
> this patch adds a envvar, which, if set, uses old behavior, and else<br>
> goes to ../lib/parseargs.sh<br>
> <br>
> Signed-off-by: Vitaly Ostrosablin <vostrosablin@virtuozzo.com><br>
> Signed-off-by: Cyrill Gorcunov <gorcunov@openvz.org><br>
> ---<br>
> test/zdtm/static/conntracks | 7 ++++++-<br>
> test/zdtm/static/route_rules | 7 ++++++-<br>
> 2 files changed, 12 insertions(+), 2 deletions(-)<br>
> <br>
> diff --git a/test/zdtm/static/conntracks b/test/zdtm/static/conntracks<br>
> index 54898c76f157..9c5be1be4de7 100755<br>
> --- a/test/zdtm/static/conntracks<br>
> +++ b/test/zdtm/static/conntracks<br>
> @@ -45,8 +45,13 @@ do_stop()<br>
> echo "PASS" > $outfile<br>
> }<br>
> <br>
> +if [ -z $VZENV ]; then<br>
> + parseargs=../lib/parseargs.sh<br>
> +else<br>
> + parseargs=parseargs.sh<br>
> +fi<br>
> <br>
> -tmpargs="$(parseargs.sh --name=$0 \<br>
> +tmpargs="$($parseargs --name=$0 \<br>
> --flags-req=statefile,outfile \<br>
> --flags-opt="start,stop" -- "$@")" ||<br>
> die "can't parse command line"<br>
> diff --git a/test/zdtm/static/route_rules b/test/zdtm/static/route_rules<br>
> index ea44537c78b9..139fbb9e05a6 100755<br>
> --- a/test/zdtm/static/route_rules<br>
> +++ b/test/zdtm/static/route_rules<br>
> @@ -60,8 +60,13 @@ do_stop()<br>
> echo "PASS" > $outfile<br>
> }<br>
> <br>
> +if [ -z $VZENV ]; then<br>
> + parseargs=../lib/parseargs.sh<br>
> +else<br>
> + parseargs=parseargs.sh<br>
> +fi<br>
> <br>
> -tmpargs="$(parseargs.sh --name=$0 \<br>
> +tmpargs="$($parseargs --name=$0 \<br>
> --flags-req=statefile,outfile \<br>
> -- "$@")" ||<br>
> die "can't parse command line"<br>
> <br>
<br>
</div>
</span></font></div>
</div>
</body>
</html>