<div dir="ltr"><div>The idea is great!</div>I&#39;d love to do it.<div><br></div><div>Abandon this one then.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Aug 12, 2013 at 1:02 AM, Andrew Vagin <span dir="ltr">&lt;<a href="mailto:avagin@parallels.com" target="_blank">avagin@parallels.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Fri, Aug 09, 2013 at 08:08:11AM -0700, Yicheng Qin wrote:<br>
&gt; From: Yicheng Qin &lt;<a href="mailto:yichengq@google.com">yichengq@google.com</a>&gt;<br>
&gt; Subject: [PATCH] test/zdtm: add -n option to run in-namespace tests only<br>
&gt;<br>
&gt; Support -n option in zdtm, which means that only in-namespace tests<br>
&gt; will be run.<br>
<br>
</div>zdtm.sh [OPTIONS] [TEST NAME]<br>
What do you think, if we change [TEST NAME] on [TEST PATTER]?<br>
<br>
For example:<br>
bash test/zdtm.sh &#39;ns/*&#39;<br>
This command runs in-namespace tests only<br>
<br>
I would prefer this variant, because it&#39;s more functional.<br>
<br>
Could you implement this functionality?<br>
<br>
Thanks.<br>
<div><div class="h5"><br>
&gt;<br>
&gt; Signed-off-by: Yicheng Qin &lt;<a href="mailto:yichengq@google.com">yichengq@google.com</a>&gt;<br>
&gt; ---<br>
&gt;  test/zdtm.sh | 12 ++++++++++++<br>
&gt;  1 file changed, 12 insertions(+)<br>
&gt;<br>
&gt; diff --git a/test/zdtm.sh b/test/zdtm.sh<br>
&gt; index 5b064f7..6db2bf0 100755<br>
&gt; --- a/test/zdtm.sh<br>
&gt; +++ b/test/zdtm.sh<br>
&gt; @@ -173,6 +173,7 @@ CLEANUP=0<br>
&gt;  PAGE_SERVER=0<br>
&gt;  PS_PORT=12345<br>
&gt;  TCPDUMP_PID=<br>
&gt; +NS_TEST_ONLY=0<br>
&gt;<br>
&gt;  check_mainstream()<br>
&gt;  {<br>
&gt; @@ -358,6 +359,11 @@ run_test()<br>
&gt;   expr &quot;$test&quot; : &#39;ns/&#39; &gt; /dev/null &amp;&amp; PIDNS=1 || PIDNS=&quot;&quot;<br>
&gt;   test=${ZP}/${test#ns/}<br>
&gt;<br>
&gt; + # option &#39;-n&#39; to skip tests without namespace<br>
&gt; + if [[ $NS_TEST_ONLY -eq 1 ]] &amp;&amp; [[ &quot;$PIDNS&quot; == &quot;&quot; ]]; then<br>
&gt; + return<br>
&gt; + fi<br>
&gt; +<br>
&gt;   shift<br>
&gt;   local args=$*<br>
&gt;   local tname=`basename $test`<br>
&gt; @@ -629,6 +635,11 @@ while :; do<br>
&gt;   shift<br>
&gt;   continue;<br>
&gt;   fi<br>
&gt; + if [ &quot;$1&quot; = &quot;-n&quot; ]; then<br>
&gt; + shift<br>
&gt; + NS_TEST_ONLY=1<br>
&gt; + continue<br>
&gt; + fi<br>
&gt;   if [ &quot;$1&quot; = &quot;-s&quot; ]; then<br>
&gt;   SNAPSHOT=1<br>
&gt;   shift<br>
&gt; @@ -682,6 +693,7 @@ Options:<br>
&gt;   -t : mount tmpfs for dump files<br>
&gt;   -a &lt;FILE&gt;.tar.gz : save archive with dump files and logs<br>
&gt;   -g : Generate executables only<br>
&gt; + -n : Run tests in namespace only<br>
&gt;  EOF<br>
&gt;  elif [ &quot;${1:0:1}&quot; = &#39;-&#39; ]; then<br>
&gt;   echo &quot;unrecognized option $1&quot;<br>
&gt; --<br>
&gt; 1.8.3<br>
&gt;<br>
<br>
</div></div>&gt; _______________________________________________<br>
&gt; CRIU mailing list<br>
&gt; <a href="mailto:CRIU@openvz.org">CRIU@openvz.org</a><br>
&gt; <a href="https://lists.openvz.org/mailman/listinfo/criu" target="_blank">https://lists.openvz.org/mailman/listinfo/criu</a><br>
<br>
</blockquote></div><br></div>