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-6045 Lustre <p>when adding an already existing net, then we fail with -EEXIST but when we add an already existing route we ignore the entry. We should also fail with -EEXIST in this case as well.</p> LU-6045

DLC: Address a minor inconsistency between net and route adding behavior

Bug Major Resolved Fixed Amir Shehata Amir Shehata Thu, 18 Dec 2014 03:02:03 +0000 Mon, 12 Jan 2015 18:33:41 +0000 Mon, 12 Jan 2015 18:33:41 +0000 Lustre 2.7.0 Lustre 2.7.0 0 6 <p>Amir Shehata (amir.shehata@intel.com) uploaded a new patch: <a href="http://review.whamcloud.com/13116" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13116</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6045" title="DLC: Address a minor inconsistency between net and route adding behavior" class="issue-link" data-issue-key="LU-6045"><del>LU-6045</del></a> lnet: return appropriate errno when adding route<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: f7ce83ee0e5ee34b255d13ee2d914ca6be749706</p> <p>I have a higher level question for end-users of lnetctl, and possibly yaml configs (as it relates <a href="https://jira.whamcloud.com/browse/LU-6043" title="DLC: lnetctl import does not configure any networks" class="issue-link" data-issue-key="LU-6043"><del>LU-6043</del></a>) - does it make more sense to return -EEXIST when trying to configure a device already configured, or should it silently succeed if it is already configured, or possibly have a userspace-level option to decide (e.g. like <tt>mkdir -p</tt> or <tt>rm -f</tt> do)?</p> <p>The way it works is that lnetctl doesn't stop it's batch process when a single operation fails.</p> <p>For example say you are provisioning N number of routes in a batch, and one of these already exists. This single operation fails, but the rest of the operations succeed. When the batch processing is complete the returned YAML block reports which operations in the batch failed and for what reason. In this case it's useful to specify a seq_no for each operation to be able to associate the YAML report with the batch request.</p> <p>The main idea here is to off load the decision whether to fail or not to fail from the kernel. The kernel always reports back the actual state of the operation requested. User space can decide what to do with this return code. I believe this is useful since the DLC model never uses the Kernel to operate on batches. It only requests one singe operation to be completed at a tiem. Batch processing always occurs in User space. In my opinion, this model is superior to getting the kernel to perform batch processing since it entails having to parse the YAML batch description in the kernel, which is not preferred.</p> <p>Another option is to parse in user space and form a batch data structure, but again this is not optimal because it limits the number of operation per batch to the max size data structure passed to the kernel.</p> <p>As I mentioned currently the DLC API processes all the batch requests and reports which ones failed without halting the batch. If it is needed we can further enhance the API to allow it to halt batch processing or continue batch processing on failure, depending on what type of flexibility we would like to expose to the user.</p> <p>As of now, I don't see a real use case to allow batch processing to be halted.</p> <p>Note: the current behavior when parsing modprobe route configuration is to ignore specific failures and continue provisioning the rest of the routes. This patch does not alter this behavior but transfers the decision to halt or not to halt (in case of DLC) to user space.</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/13116/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13116/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6045" title="DLC: Address a minor inconsistency between net and route adding behavior" class="issue-link" data-issue-key="LU-6045"><del>LU-6045</del></a> lnet: return appropriate errno when adding route<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 0f9f4b1b234bdc12a2604225d8c6398a355b75a4</p> Related LU-6043 Development Rank 1|hzx2rj: Rank (Obsolete) 16846 Severity