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-15625 Lustre <p>Only <tt>lctl conf_param</tt> must be used to set root squash parameters. If <tt>lctl set_param -P</tt> is used, the value is set only on server side permanently, and any other value set via conf_param is ignored. Plus this value is not forwarded to client side.</p> LU-15625

set_param -P must not be used for root squash

Bug Minor Resolved Fixed Sebastien Buisson Sebastien Buisson sec Mon, 7 Mar 2022 17:36:20 +0000 Sun, 21 Aug 2022 13:39:04 +0000 Mon, 27 Jun 2022 15:55:45 +0000 Lustre 2.16.0 Lustre 2.15.2 0 3 <p>"Sebastien Buisson &lt;sbuisson@ddn.com&gt;" uploaded a new patch: <a href="https://review.whamcloud.com/46728" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/46728</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-15625" title="set_param -P must not be used for root squash" class="issue-link" data-issue-key="LU-15625"><del>LU-15625</del></a> sec: set_param -P must not be used for root squash<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: fa37f40d246748937f8f0bf459c34b7c72e75c44</p> <p>It isn't really clear why "<tt>set_param -P</tt>" is not working in this case? Is there a different path or parameter name in use on the client compared to the server?</p> <p>It looks like that is the root of the problem:</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre># lctl list_param -R "*" | grep squash llite.myth-ffff90f574676000.nosquash_nids llite.myth-ffff90f574676000.root_squash mdt.myth-MDT0000.nosquash_nids mdt.myth-MDT0000.root_squash nodemap.default.squash_gid nodemap.default.squash_uid </pre> </div></div> <p>It looks like the squash parameter needs to be set like:</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre># lctl set_param -P *.myth*.root_squash=x:y # lctl set_param -P *.myth*.nodquash_nids=xxx </pre> </div></div> <p>So that it includes the fsname but not the device name, so that it applies on both the client and server. </p> <p>It would really be better for usability if "<tt>set_param -P</tt>" just worked properly in this case, since users are very unlikely to read the manual. One option would be for the config llog parsing to detect the case of <tt>mdt.&#42;.root_squash</tt> and replaced it with <tt>llite.&#42;</tt> so that it "just worked".</p> <p>OK, after discussion with Andreas, approach taken in <a href="https://review.whamcloud.com/46728" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/46728</a> is not the best way to go.</p> <p>The patch for this problem should consist in fixing <tt>set_param -P</tt> for <tt>root_squash</tt> and <tt>nosquash_nids</tt> parameters, so that this setting is correctly propagated to clients, and we do not end up with clients with inconsistent or no values at all for these parameters.</p> <p>"Sebastien Buisson &lt;sbuisson@ddn.com&gt;" uploaded a new patch: <a href="https://review.whamcloud.com/46739" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/46739</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-15625" title="set_param -P must not be used for root squash" class="issue-link" data-issue-key="LU-15625"><del>LU-15625</del></a> sec: fix set_param -P for root squash<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 9dcb2e477ad58f35b9d63e689350e0fb927a0168</p> <p>"Oleg Drokin &lt;green@whamcloud.com&gt;" merged in patch <a href="https://review.whamcloud.com/46739/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/46739/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-15625" title="set_param -P must not be used for root squash" class="issue-link" data-issue-key="LU-15625"><del>LU-15625</del></a> sec: fix set_param -P for root squash<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 7b8449e8f9ed310376a191ec8dfa2f1cb6233377</p> <p>"Sebastien Buisson &lt;sbuisson@ddn.com&gt;" uploaded a new patch: <a href="https://review.whamcloud.com/47792" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/47792</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-15625" title="set_param -P must not be used for root squash" class="issue-link" data-issue-key="LU-15625"><del>LU-15625</del></a> sec: fix set_param -P for root squash<br/> Project: fs/lustre-release<br/> Branch: b2_15<br/> Current Patch Set: 1<br/> Commit: 6db1730e59e574bd5e6cb5537248eaf9e760322c</p> <p>Landed for 2.16</p> <p>"Oleg Drokin &lt;green@whamcloud.com&gt;" merged in patch <a href="https://review.whamcloud.com/47792/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/47792/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-15625" title="set_param -P must not be used for root squash" class="issue-link" data-issue-key="LU-15625"><del>LU-15625</del></a> sec: fix set_param -P for root squash<br/> Project: fs/lustre-release<br/> Branch: b2_15<br/> Current Patch Set: <br/> Commit: 8ea47747c9fcf3ba1d0bca8456f2ad978f8dd2d5</p> Development Rank 1|i02k4f: Rank (Obsolete) 9223372036854775807 Severity