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-13224 Lustre <p>Allow applications that need to access the various lustre parameters in sysfs, profs, etc. to call a function from lustreapi.h that behaves similarly to "lctl get_param &lt;param name&gt;" and has the option to return data to any FILE*, so that it can be further processed by the calling application.</p> <p>The goal is to use the same infrastructure that allows lctl to know where the correct files are without having to create an lctl process for each get, set, and list operation.</p> <p>The target applications are those that need to continuously monitor lustre, such as LMT.</p> RHEL 7.5 LU-13224

Expose lctl get/set/list param functionality in lustreapi

Improvement Minor Resolved Fixed Gian-Carlo Defazio Gian-Carlo Defazio llnl Sat, 8 Feb 2020 00:39:12 +0000 Thu, 23 Apr 2020 16:24:38 +0000 Tue, 21 Apr 2020 15:31:11 +0000 Lustre 2.10.8 Lustre 2.14.0 Lustre 2.12.5 0 7 <p>There are already functions in liblustreapi_param.c that do this, they just haven't been exposed publicly yet.</p> <p>Please take a look at <tt>get_lustre_param_path()</tt> and <tt>get_lustre_param_value()</tt>. If they seem suitable for your use, we can expose them as proper <tt>llapi_&#42;</tt> functions. Otherwise, feedback is welcome.</p> <p><tt>get_lustre_param_path()</tt> and <tt>get_lustre_param_value()</tt> do most of what I'm looking for. However, <tt>get_lustre_param_value()</tt> reads only the first line of the first file that matches some pattern. I'd like a more flexible interface that can do some of what lctl does, such as reading all lines of all files that match. The solution I've come up with moves, and slightly modifies, some functions from lustre_cfg.c to liblustreapi.c and then exposes them using lustreapi.h.}}</p> <p>I'm OK with changing the <tt>get_lustre_param_{path,value}()</tt> interface slightly, since those are not officially supported APIs. The new function names should preferably be something like <tt>llapi_param_get_{path,value}()</tt> so the names are more closely related. Alternately, you could add a third interface like <tt>llapi_param_get_values()</tt> or <tt>llapi_param_get_all()</tt> that returns all of the fields. What might be even more helpful would be <tt>llapi_param_get_value_matching()</tt> that can parse out matching subfields of YAML data and return only the data from one field.</p> <p>Gian-Carlo DeFazio (defazio1@llnl.gov) uploaded a new patch: <a href="https://review.whamcloud.com/37545" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/37545</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-13224" title="Expose lctl get/set/list param functionality in lustreapi" class="issue-link" data-issue-key="LU-13224"><del>LU-13224</del></a> utils: expose lctl functionality via lustreapi<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 4641f2b945b7f51fdd8783f0b8238a0f7c99551f</p> <p>Gian-Carlo DeFazio (defazio1@llnl.gov) uploaded a new patch: <a href="https://review.whamcloud.com/38011" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/38011</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-13224" title="Expose lctl get/set/list param functionality in lustreapi" class="issue-link" data-issue-key="LU-13224"><del>LU-13224</del></a> utils: insignificant changes to compare tests.<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: a96243e315bede22a1bfbe934e21badf2e315e6e</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/37545/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/37545/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-13224" title="Expose lctl get/set/list param functionality in lustreapi" class="issue-link" data-issue-key="LU-13224"><del>LU-13224</del></a> utils: expose llapi_param* functions<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 9b44cf70a95b2ee97a17697dc97fbe462ad1f5b9</p> <p>Gian-Carlo DeFazio (defazio1@llnl.gov) uploaded a new patch: <a href="https://review.whamcloud.com/38239" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/38239</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-13224" title="Expose lctl get/set/list param functionality in lustreapi" class="issue-link" data-issue-key="LU-13224"><del>LU-13224</del></a> utils: expose llapi_param* functions<br/> Project: fs/lustre-release<br/> Branch: b2_12<br/> Current Patch Set: 1<br/> Commit: bbb9485b7d63c5aea42ad65e664b88d55a40da76</p> <p>Gian-Carlo DeFazio (defazio1@llnl.gov) uploaded a new patch: <a href="https://review.whamcloud.com/38299" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/38299</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-13224" title="Expose lctl get/set/list param functionality in lustreapi" class="issue-link" data-issue-key="LU-13224"><del>LU-13224</del></a> utils: expose llapi_param* functions<br/> Project: fs/lustre-release<br/> Branch: b2_10<br/> Current Patch Set: 1<br/> Commit: a3f90f10a23e14b07c9250e07b3aacc21757979d</p> <p>Can this be closed?</p> <p>Closing this issue. The backports can land independently. </p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/38239/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/38239/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-13224" title="Expose lctl get/set/list param functionality in lustreapi" class="issue-link" data-issue-key="LU-13224"><del>LU-13224</del></a> utils: expose llapi_param* functions<br/> Project: fs/lustre-release<br/> Branch: b2_12<br/> Current Patch Set: <br/> Commit: c1ff1110d090686833196918ccf289cf62a280d6</p> Development Rank 1|i00tfr: Rank (Obsolete) 9223372036854775807