Whamcloud Community JIRA https://jira.whamcloud.com This file is an XML representation of an issue en-us 9.4.14 940014 05-12-2023 [LU-9318] sanity-lsnapshot should be skipped for all non-ZFS configs https://jira.whamcloud.com/browse/LU-9318 Lustre <p><a href="https://testing.hpdd.intel.com/test_sessions/1ef8dc97-8487-4c21-ad37-fe03e2430916" class="external-link" target="_blank" rel="nofollow noopener">https://testing.hpdd.intel.com/test_sessions/1ef8dc97-8487-4c21-ad37-fe03e2430916</a></p> Interop:<br/> server: RHEL7.3, 2.9 Lustre, ldiskfs <br/> client: RHEL7.3, 2.10 Lustre<br/> LU-9318 sanity-lsnapshot should be skipped for all non-ZFS configs Bug Major Resolved Fixed Saurabh Tandan James Casper Tue, 11 Apr 2017 15:32:16 +0000 Wed, 19 Apr 2017 20:58:54 +0000 Wed, 19 Apr 2017 12:44:28 +0000 Lustre 2.10.0 Lustre 2.10.0 0 5 <p>James, could you tell us the reason for skipping? and shouldn't this be in lustre tree, not auto test?</p> <p>sanity-lsnapshot should only be run on zfs configs.</p> <p>Of our 7 regression and interop configs, only 2 use ZFS. It was correctly skipped or <br/> absent for 4 of the 5 ldiskfs configs.</p> <p>The problem here is that the check for a ZFS file system is after the script tries to add snapshot to the debug list.</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre> 00:27:14:CMD: onyx-47vm7 /usr/sbin/lctl set_param debug=+snapshot 00:27:14:onyx-47vm7: error: set_param: setting /proc/sys/lnet/debug=+snapshot: Invalid argument 00:27:14:sanity-lsnapshot returned 0 </pre> </div></div> <p>This is a script issue and I'll convert ticket to LU.</p> <p>This just needs a check at the start of the script to skip the whole thing if not running with ZFS, or if the version is too old:</p> <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java">[ $(lustre_version_code mds1) -lt $(version_code 2.9.55) || $(lustre_version_code ost1) -lt $(version_code 2.9.55) ] &amp;&amp; skip <span class="code-quote">"Need server version at least 2.9.55"</span> &amp;&amp; exit 0 [ $(facet_fstype mds1) = <span class="code-quote">"ldiskfs"</span> ] || $(facet_fstype ost1) = <span class="code-quote">"ldiskfs"</span> ] &amp;&amp; skip <span class="code-quote">"can't test snapshots with ldiskfs"</span> &amp;&amp; exit 0 </pre> </div></div> <p>Saurabh Tandan (saurabh.tandan@intel.com) uploaded a new patch: <a href="https://review.whamcloud.com/26600" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/26600</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-9318" title="sanity-lsnapshot should be skipped for all non-ZFS configs" class="issue-link" data-issue-key="LU-9318"><del>LU-9318</del></a> test: sanity-lsnapshot skipped for all non-zfs configs<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 57433a0b141b5791d0347d9e6c22d21dd1d82c6d</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="https://review.whamcloud.com/26600/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/26600/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-9318" title="sanity-lsnapshot should be skipped for all non-ZFS configs" class="issue-link" data-issue-key="LU-9318"><del>LU-9318</del></a> test: sanity-lsnapshot skipped for all non-zfs configs<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: eb3379162c9c3044759013390cdba7b53783b31d</p> <p>Landed for 2.10</p> Related LU-9320 Development Rank 1|hzz9tj: Rank (Obsolete) 9223372036854775807 Severity