[LU-8132] tests: llmount.sh does not use fs-specific paths for the ost and mdt it mounts Created: 12/May/16 Updated: 05/Aug/20 Resolved: 05/Aug/20 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.9.0 |
| Type: | Bug | Priority: | Minor |
| Reporter: | CEA | Assignee: | James Nunez (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | patch | ||
| Issue Links: |
|
||||||||
| Severity: | 3 | ||||||||
| Rank (Obsolete): | 9223372036854775807 | ||||||||
| Description |
|
When using llmount.sh with its default configuration, the paths to the ost(s) and mdt(s) looks like this: Because those paths are not specific to the filesystem they are related to, when mounting two different filesystems with llmount.sh, the first one's ost(s) and mdt(s) are replaced by the second's. |
| Comments |
| Comment by Quentin Bouget [ 12/May/16 ] |
|
This could be corrected by changing the paths to something like: |
| Comment by Peter Jones [ 12/May/16 ] |
|
Do you intend to push a patch to fix this? |
| Comment by Gerrit Updater [ 13/May/16 ] |
|
Quentin Bouget (quentin.bouget.ocre@cea.fr) uploaded a new patch: http://review.whamcloud.com/20188 |
| Comment by Quentin Bouget [ 13/May/16 ] |
|
Actually since the ultimate goal is to be able to mount two different filesystems in local mode with llmount.sh, would it be ok for this patch to introduce some other modifications? For that, the changes in the current fix are not enough. I would need to add an option "NOFORMAT_MGS" and "NOSETUP_MGS" (similar to the NOFORMAT and NOSETUP variables used in llmount.sh) to protect the mgs created by the first llmount.sh execution from the second one. |
| Comment by James Nunez (Inactive) [ 13/May/16 ] |
|
Quentin - Thank you for the patch. You may certainly add functionality to the existing patch, but I prefer patches that do a single thing since this is easier for reviewers and may help with the landing process. My request would be to upload a second patch with your proposed change of introducing NOFORMAT_MGS and NOSETUP_MGS. |
| Comment by Quentin Bouget [ 13/May/16 ] |
|
Ok, thanks. Here is the related ticket: LU-8140. |
| Comment by Gerrit Updater [ 02/Jun/16 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/20188/ |
| Comment by James Nunez (Inactive) [ 29/Jun/16 ] |
|
Patch landed to master (pre-2.9.0). Work related to this ticket is tracked under LU-8140. |