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-10460 Lustre <p>In <a href="https://jira.whamcloud.com/browse/LU-10392" title="LustreError: 82980:0:(fid_handler.c:329:__seq_server_alloc_meta()) srv-lglossy-MDT0002: Allocated super-sequence failed: rc = -115" class="issue-link" data-issue-key="LU-10392"><del>LU-10392</del></a>, a list of local LLNL patches applied to 2.10.2 during our testing came up, and Peter requested we consider pushing them upstream.</p> <p><a href="https://jira.hpdd.intel.com/browse/LU-10392?focusedCommentId=216657&amp;page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-216657" class="external-link" target="_blank" rel="nofollow noopener">https://jira.hpdd.intel.com/browse/LU-10392?focusedCommentId=216657&amp;page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-216657</a></p> <p>This ticket is to track that.  The two patches are: </p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>* 7823f77 LU-4009 osd-zfs: Add tunables to disable sync (DEBUG) * 1826c76 Don't install lustre init script on systemd systems (AKA https://review.whamcloud.com/#/c/7761/) </pre> </div></div> LU-10460

Evaluate two LLNL patches for upstream

Task Minor Resolved Fixed Peter Jones Olaf Faaland Thu, 4 Jan 2018 22:22:29 +0000 Mon, 30 Jul 2018 17:55:26 +0000 Thu, 25 Jan 2018 18:06:37 +0000 Lustre 2.11.0 Lustre 2.10.4 0 4 <p>Let's discuss these one at a time.</p> <p>I propose the patch re: lustre init scripts is not appropriate for upstream.</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>* 1826c76 Don't install lustre init script on systemd systems </pre> </div></div> <p>All that patch does, is avoid including the following files in the rpm if with_systemd is not set:</p> <ul> <li>lustre/scripts/lustre</li> <li>lustre/conf/lustre</li> </ul> <p>This works for us, because we use pacemaker and a resource agent to start, stop, and query the status of lustre targets on our servers. The resource agent contains the start/stop/query logic and pacemaker keeps track of what is running where.</p> <p>However this seems inappropriate for the general user running under systemd, who might be merely testing lustre, or might be using diskful nodes without shared storage, where ordinary systemctl commands seem like the right mechanism for managing these functions.</p> <p>We don't need this ourselves, though, and so don't have plans to allocate time to work out the right service definitions.</p> <p>Olaf</p> <p>We discussed this and agree with your assessment that the lustre-init scripts change is more suitable to keep as an LLNL-only change but we will move forward to land adding <a href="https://review.whamcloud.com/#/c/7761/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/#/c/7761/</a> using this tracking reference.</p> <p>Peter</p> <p>I can't find systemd patch in our Gerrit repo, so it would be useful to get a version pushed for review, with a description of what problem it is solving. These patches were added for a reason, so we need to understand why they are harmful just being installed on systemd systems, and if we can somehow make them conditional/deactivated when HA agents are involved. </p> <p>Olaf Faaland-LLNL (faaland1@llnl.gov) uploaded a new patch: <a href="https://review.whamcloud.com/30830" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/30830</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-10460" title="Evaluate two LLNL patches for upstream" class="issue-link" data-issue-key="LU-10460"><del>LU-10460</del></a> rpm: Skip packaging lustre init for systemd<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 236ff56515c4263a3af61d316644f44f4a553b20</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="https://review.whamcloud.com/7761/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/7761/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-10460" title="Evaluate two LLNL patches for upstream" class="issue-link" data-issue-key="LU-10460"><del>LU-10460</del></a> osd-zfs: Add tunables to disable sync<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 00c4bd85b372f68c3e75fdab9658c8f0074113be</p> <p>IIUC we do not intend to land the systemd patch. If you agree can you abandon the patch and we'll close out this ticket now that the sync patch has landed.</p> <blockquote><p>IIUC we do not intend to land the systemd patch. If you agree can you abandon the patch and we'll close out this ticket now that the sync patch has landed.</p></blockquote> <p>I agree. I abandoned the systemd/systemV init script patch.</p> <p>Thanks Olaf</p> <p>Minh Diep (minh.diep@intel.com) uploaded a new patch: <a href="https://review.whamcloud.com/31163" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/31163</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-10460" title="Evaluate two LLNL patches for upstream" class="issue-link" data-issue-key="LU-10460"><del>LU-10460</del></a> osd-zfs: Add tunables to disable sync<br/> Project: fs/lustre-release<br/> Branch: b2_10<br/> Current Patch Set: 1<br/> Commit: b5a8310d211a3a1d0e917b04a61eadc2f7fa3d42</p> <p>John L. Hammond (john.hammond@intel.com) merged in patch <a href="https://review.whamcloud.com/31163/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/31163/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-10460" title="Evaluate two LLNL patches for upstream" class="issue-link" data-issue-key="LU-10460"><del>LU-10460</del></a> osd-zfs: Add tunables to disable sync<br/> Project: fs/lustre-release<br/> Branch: b2_10<br/> Current Patch Set: <br/> Commit: 0394b38784cabd3c73fed1b9f4d6224e151303a3</p> Related Development Rank 1|hzzqjj: Rank (Obsolete) 9223372036854775807