Adjust/generateconfs auto user group home #75
Conversation
jonasbardino
force-pushed
the
adjust/generateconfs-auto-user-group-home
branch
from
55f2132 to
e3075d2
Compare
…reater flexibility on non-deployment setups and in order to eliminate the dynamic values from PR72 in mig.shared.defaults . Please note that this is a manual rebase of the changes originally implemented on July 1st to 3rd, which got really messy when merging edge updates.
… and update the fixture files accordingly. Please note that this is a manual rebase of the changes originally implemented on July 1st to 3rd, which got really messy when merging edge updates
jonasbardino
force-pushed
the
adjust/generateconfs-auto-user-group-home
branch
from
f048d4d to
5dcb788
Compare
|
Manually rebased to current |
|
Between the CWD related hardening done in #89 and the changes that were needed to get local configs reliably written in support of the work in #77, it ended up that removal of those defaults was basically already arranged. I pulled in the changes you made here to mig_user/mig_group into PR 77 and think that effectively subsumes this PR. Result here: https://github.com/ucphhpc/migrid-sync/pull/77/files#diff-13b2a8d0b68628063fa8692dfa94986a499e335369a21e9234a9ebd1923ddc1dL80 |
|
Yes and no, I commented on PR77 now to point out a couple of details from this PR75, which it doesn't seem to handle. If we agree I'll retire this PR and merge PR77 with those adjustments. |
|
Folded into PR77 - retiring. |
A follow-up to PR72 to switch more
generateconfsarguments over tokeyword_autowith dynamic expansion of user/group and related paths to fit the active environment. This is particularly useful for local development where neither the mig user nor the code and state dirs can be expected to be in the root of the home directory. It additionally allows elimination of the dynamic expansion of values in mig.shared.defaults and keeps it local to the install helpers.