https://jira.whamcloud.com This file is an XML representation of an issue en-us 9.4.14 940014 05-12-2023 https://jira.whamcloud.com/browse/LU-2763 Lustre <p>I was trying to create a test patch with a meaningful Test-Paramaters. </p> <p>The patch can be seen here <a href="http://review.whamcloud.com/5275" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/5275</a></p> <p>The paramaters in question are</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>Test-Parameters: nettypes=o2ib envdefinitions=SLOW=yes \ clientdistro=el6 serverdistro=el5 testlist=racer </pre> </div></div> <p>The b1_8 build/commit-msg does not allow lines over 70 long. "\" escaping did not work does not work as it combines them together. </p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>error: commit message has line longer than 70 columns. </pre> </div></div> <p>The error message is seen regardless of the escaping used. I tired sever thing and in the end a used the "-r" change. </p> <p>In the above patch I was able to work through the issue by adding a "-r" to the read command but is has been suggested that we bring back the entire commit-msg from current Master. </p> The current Git Tree and Ubunut 12.04 desktop. LU-2763

Build: 1.8 git commit fails to allow long Test Paramters

Bug Minor Resolved Not a Bug Keith Mannthey Keith Mannthey Tue, 5 Feb 2013 20:53:19 +0000 Wed, 22 May 2013 22:37:24 +0000 Wed, 22 May 2013 22:37:24 +0000 Lustre 2.4.0 Lustre 2.1.5 Lustre 1.8.9 0 5 <p><a href="http://review.whamcloud.com/5278" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/5278</a> is the initial patch. </p> <p>Hi Keith,</p> <p>We found that while editing commit message for a Lustre b2_1 patch with the following test parameters:</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>Test-Parameters: envdefinitions=DURATION=14400 \ clientdistro=el5 serverdistro=el5 clientarch=x86_64 \ serverarch=x86_64 clientcount=4 osscount=2 mdscount=2 \ austeroptions=-R failover=true useiscsi=true \ testlist=recovery-mds-scale </pre> </div></div> <p>we got the following one after committing:</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>Test-Parameters: envdefinitions=DURATION=14400 clientdistro=el5 serverdistro=el5 clientarch=x86_64 serverarch=x86_64 clientcount=4 osscount=2 mdscount=2 austeroptions=-R failover=true useiscsi=true testlist=recovery-mds-scale </pre> </div></div> <p>E.g., <a href="http://review.whamcloud.com/5462" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/5462</a></p> <p>On b1_8 branch, we can use two '\'s to escape '\', but this does not work on b2_1 and master branches. </p> <p>I find the double escape "<br class="atl-forced-newline" />" an ok solution.</p> <p>There is a good workaround. </p> Development Rank 1|hzvio7: Rank (Obsolete) 6696 Severity