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-13359 Lustre <p>With <a href="https://review.whamcloud.com/35615" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/35615</a> "lfs quota --pool" shows correct soft/hard limit and grace time. However used is still the same with used without --pool option, i.e. it prints used for all OSTs in a system. print_obd_quota should be changed to take into account only OSTs from appropriate quota pool. I would like to do this in a separate ticket to make 35615 as small as possible.</p> LU-13359

change lfs quota --pool to print used based only on OSTs from the pool

Task Critical Resolved Fixed Sergey Cheremencev Sergey Cheremencev Fri, 13 Mar 2020 13:52:18 +0000 Sat, 12 Sep 2020 15:43:58 +0000 Thu, 13 Aug 2020 13:16:32 +0000 Lustre 2.14.0 Lustre 2.14.0 0 4 <p>Sergey,</p> <p>A thought: I would do this as a patch on top of 35615, and I think it could use the same LU.  I did this sort of approach for SEL - Generated a patch series, all associated with one ticket.</p> <p>This is a fix that is not relevant except with 35615, right?  If so, I think it's clearer to just put another patch associated with the original LU.  If I've misunderstood and this fix makes sense <b>without</b> 35615, then never mind. <img class="emoticon" src="https://jira.whamcloud.com/images/icons/emoticons/smile.png" height="16" width="16" align="absmiddle" alt="" border="0"/></p> <p>Hi Patrick!</p> <p>Glad to see you hear <img class="emoticon" src="https://jira.whamcloud.com/images/icons/emoticons/smile.png" height="16" width="16" align="absmiddle" alt="" border="0"/>.</p> <blockquote><p>A thought: I would do this as a patch on top of 35615, and I think it could use the same LU.  I did this sort of approach for SEL - Generated a patch series, all associated with one ticket.</p></blockquote> <p>That would be nice and yes a patch on top of 35615 is a correct place as there is already an option to show/set quota for the pol.<br/> If so, I will close this ticket - I created it mostly to don't forget about this problem.</p> <p>Sergey Cheremencev (sergey.cheremencev@hpe.com) uploaded a new patch: <a href="https://review.whamcloud.com/39298" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/39298</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-13359" title="change lfs quota --pool to print used based only on OSTs from the pool" class="issue-link" data-issue-key="LU-13359"><del>LU-13359</del></a> quota: make used for pool correct<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 58ebb595ee9f1e28cd4ef8ce2e3d26dcd92c9860</p> <p>While Patrick's comments were OK <em>before</em> the <a href="https://jira.whamcloud.com/browse/LU-11023" title="OST Pool Quotas" class="issue-link" data-issue-key="LU-11023"><del>LU-11023</del></a> patches were landed, it is better to use separate tickets for issues found <em>after</em> the <a href="https://jira.whamcloud.com/browse/LU-11023" title="OST Pool Quotas" class="issue-link" data-issue-key="LU-11023"><del>LU-11023</del></a> patches are landed and the ticket closed, as was done with this patch. Otherwise, it is difficult to track patches that <em>have</em> to land to 2.14 in order to make this feature work properly. I've changed this over to "<tt>Fix Version: 2.14.0</tt> and "<tt>Critical</tt>" so that it is tracked properly.</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/39298/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/39298/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-13359" title="change lfs quota --pool to print used based only on OSTs from the pool" class="issue-link" data-issue-key="LU-13359"><del>LU-13359</del></a> quota: make used for pool correct<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 6b9f849fd5f49ce68e6102f9c79f52eb11bf949d</p> <p>Landed for 2.14</p> <p>Neil Brown (neilb@suse.de) uploaded a new patch: <a href="https://review.whamcloud.com/39676" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/39676</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-13359" title="change lfs quota --pool to print used based only on OSTs from the pool" class="issue-link" data-issue-key="LU-13359"><del>LU-13359</del></a> quota: call rhashtable_lookup near params decl<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 11f22d965757d0535165ef58fc01d5c4d3aeb327</p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/39676/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/39676/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-13359" title="change lfs quota --pool to print used based only on OSTs from the pool" class="issue-link" data-issue-key="LU-13359"><del>LU-13359</del></a> quota: call rhashtable_lookup near params decl<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 1d116c8ff68fc784141b647b790e4b4861460797</p> Related LU-11023 Development Epic/Theme quota Rank 1|i00vfr: Rank (Obsolete) 9223372036854775807