[LU-12514] separate out the lustre mount code for server and clients Created: 05/Jul/19 Updated: 17/Jun/23 |
|
| Status: | Open |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Minor |
| Reporter: | James A Simmons | Assignee: | James A Simmons |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||||||||||||||||||||||
| Rank (Obsolete): | 9223372036854775807 | ||||||||||||||||||||||||||||
| Description |
|
During LUG it was discussed how Lustre uses the same file system type for both servers and clients. This is something the kernel VFS maintainer would rather see handled differently. This ticket tracks the changes upstream for the client as well as implement the needed server side changes. |
| Comments |
| Comment by Gerrit Updater [ 05/Jul/19 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/35424 |
| Comment by Gerrit Updater [ 05/Jul/19 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/35425 |
| Comment by Gerrit Updater [ 05/Jul/19 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/35426 |
| Comment by Gerrit Updater [ 05/Jul/19 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/35427 |
| Comment by Andreas Dilger [ 06/Jul/19 ] |
|
Given that we are past feature freeze for 2.13 and want to keep changes to a minimum, I think it makes sense to register a "lustre_tgt" filesystem type that is exactly the same as "lustre" for now, but would allow for forward compatibility and users could start adding "lustre_tgt" to /etc/fstab and/or if they upgrade and start using "lustre_tgt" they could downgrade without problems. If the patch is simple, maybe it could be backported to 2.12 as well. |
| Comment by Andreas Dilger [ 06/Jul/19 ] |
|
Another possibility is "lustre_osd" instead of "lustre_tgt". |
| Comment by James A Simmons [ 07/Jul/19 ] |
|
Agree. I only pushed the 'simple' patches so far. The other work is much more involved and I expect that is for 2.14 |
| Comment by Andreas Dilger [ 08/Jul/19 ] |
|
I'm not suggesting to actually separate the client and server mount code for 2.13, just register a second filesystem_type with the existing code, just for forward compatibility for the new mount type. |
| Comment by Gerrit Updater [ 12/Jul/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/35424/ |
| Comment by Gerrit Updater [ 24/Jul/19 ] |
|
Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/35608 |
| Comment by Gerrit Updater [ 11/Sep/19 ] |
|
Oleg Drokin (green@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/36165 |
| Comment by Gerrit Updater [ 17/Sep/19 ] |
|
Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/36202 |
| Comment by Gerrit Updater [ 20/Sep/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/36165/ |
| Comment by Gerrit Updater [ 06/Dec/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/35425/ |
| Comment by Gerrit Updater [ 20/Dec/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/35426/ |
| Comment by Gerrit Updater [ 20/Dec/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/35427/ |
| Comment by James A Simmons [ 20/Dec/19 ] |
|
Theirs more |
| Comment by Gerrit Updater [ 24/Feb/20 ] |
|
James Simmons (jsimmons@infradead.org) uploaded a new patch: https://review.whamcloud.com/37693 |
| Comment by Gerrit Updater [ 02/Jun/20 ] |
|
James Simmons (jsimmons@infradead.org) uploaded a new patch: https://review.whamcloud.com/38799 |
| Comment by Gerrit Updater [ 10/Jul/20 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/38799/ |
| Comment by Gerrit Updater [ 10/Mar/21 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37693/ |
| Comment by Gerrit Updater [ 27/Apr/22 ] |
|
"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/47159 |
| Comment by Gerrit Updater [ 27/Apr/22 ] |
|
"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/47160 |
| Comment by Gerrit Updater [ 11/Jun/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/47159/ |
| Comment by Gerrit Updater [ 01/Sep/22 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/47160/ |
| Comment by James A Simmons [ 01/Sep/22 ] |
|
One more patch to develop and another to test the change. |
| Comment by Gerrit Updater [ 15/Nov/22 ] |
|
"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/49165 |