Bug 4833 - /vsmagent/make_homedir is redundant
Summary: /vsmagent/make_homedir is redundant
Status: NEW
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: VSM Agent (show other bugs)
Version: trunk
Hardware: PC Unknown
: P2 Normal
Target Milestone: MediumPrio
Assignee: Peter Åstrand
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-10-09 11:14 CEST by Karl Mikaelsson
Modified: 2020-06-09 12:41 CEST (History)
0 users

See Also:
Acceptance Criteria:


Attachments

Description Karl Mikaelsson cendio 2013-10-09 11:14:56 CEST
Some of the arguments for turning it off:

 * Principle of least surprise - don't make changes to the end user system without explicitly changing the configuration.
 * Lacks features - doesn't copy /etc/skel for example.
 * Feature overlap with autodir and/or pam_mkhomedir.
Comment 1 Karl Mikaelsson cendio 2013-10-15 11:14:21 CEST
Further arguments for turning it off:

 - Prevents session startup with NFS root_squash.

Arguments for keeping it on:

 - Demo effect/out of the box experience for cases where users but home directories doesn't exist.
Comment 2 Peter Åstrand cendio 2013-10-15 13:13:38 CEST
(In reply to comment #1)
> Further arguments for turning it off:
> 
>  - Prevents session startup with NFS root_squash.

This could be solved by making this feature a "best effort" feature. Currently, we do not fail upon EEXIST. We could additionally allow EACCES or even allow session startup on any error from mkdir. 

See also bug 4831.
Comment 3 Pierre Ossman cendio 2020-06-09 12:41:19 CEST
There is a bigger argument against this feature; it is redundant with functionality already present on the system, i.e. pam_mkhomedir. That implementation also doesn't have the same limitations that our stuff does.

Note You need to log in before you can comment on or make changes to this bug.