Skip to content

Conversation

@mboisson
Copy link
Member

@mboisson mboisson commented Jan 31, 2024

This resolves #316

@cmd-ntrf
Copy link
Member

cmd-ntrf commented Nov 4, 2025

The PR in its current state does not provide a information to the end user about the circumstances or context in which they should enable it. In the absence of explanation about the context, an end user could be tempted to enable it and it will fail if the home directory is on NFS or it will do nothing if the users are part of the FreeIPA domain.

To merge this PR, we would need to add:

  • some safeguard or warning when the home is on NFS
  • link the mkhomecreation enabling with adding a domain to SSSD, so one only using FreeIPA as an auth provider would not consider activating it.

@cmd-ntrf cmd-ntrf added the enhancement New feature or request label Nov 4, 2025
@mboisson
Copy link
Member Author

mboisson commented Nov 6, 2025

It will only fail on NFS if it is exported with root_squash for login nodes, correct ?

@cmd-ntrf
Copy link
Member

cmd-ntrf commented Nov 6, 2025

It will only fail on NFS if it is exported with root_squash for login nodes, correct ?

Correct. The no_root_squash option is activated for instances with tags in profile::nfs::server::no_root_squash_tags:
https://github.com/ComputeCanada/puppet-magic_castle/blob/main/site/profile/manifests/nfs.pp#L71

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Supporting oddjob-mkhomedir

2 participants