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-13053 Lustre <p>conf-sanity test_87 calls umount_ldiskfs() and fails to find the function</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>/usr/lib64/lustre/tests/conf-sanity.sh: line 6334: umount_ldiskfs: command not found </pre> </div></div> <p>The problem is that umount_ldiskfs() does not exist, but unmount_ldiskfs() does exist. conf-sanity test 87 is the only test and only test suite that calls unmount_ldiskfs(). </p> <p>umount_client() is another function that exists and is used throughout conf-sanity. Since the unmount command is umount, let’s be consistent and change unmount_ldiskfs() to umount_ldiskfs().</p> LU-13053

conf-sanity test 87 call to umount_ldiskfs() fails

Bug Minor Resolved Fixed James Nunez James Nunez easy tests Fri, 6 Dec 2019 16:28:24 +0000 Thu, 23 Jan 2020 13:30:41 +0000 Thu, 23 Jan 2020 13:30:41 +0000 Lustre 2.13.0 Lustre 2.14.0 Lustre 2.12.4 Lustre 2.14.0 0 2 <p>James Nunez (jnunez@whamcloud.com) uploaded a new patch: <a href="https://review.whamcloud.com/36949" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/36949</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-13053" title="conf-sanity test 87 call to umount_ldiskfs() fails" class="issue-link" data-issue-key="LU-13053"><del>LU-13053</del></a> tests: rename unmount_ldiskfs to umount<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 1f24cd0e0c460e0ff2e079ee1ad16d0d71554ab2</p> <p>Reviewing all the unmount commands in test-framework, we have the following that use 'umount' in the name:<br/> umount_client()<br/> zconf_umount_clients()</p> <p>We have the following that use 'unmount' in the name:<br/> unmount_ldiskfs()<br/> unmount_zfs()<br/> unmount_fstype()</p> <p>It looks like we are being a bit inconsistent in naming conventions between clients and servers.</p> <p>In the latest version of the patch, I just changed conf-sanity test 87 to conform to what is in test-framework.sh. </p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/36949/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/36949/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-13053" title="conf-sanity test 87 call to umount_ldiskfs() fails" class="issue-link" data-issue-key="LU-13053"><del>LU-13053</del></a> tests: fix conf-sanity call to umount_ldiskfs<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 4cccb265c5c2148cc4d1089fa1bdaa8db606d57e</p> <p>Landed for 2.14</p> Development Rank 1|i00qkv: Rank (Obsolete) 9223372036854775807 Severity