Comment # 8 on bug 1219985 from Georg Pfuetzenreuter
I now tried your suggestion.

This works:


```
c0c6bbd75dbf:/srv/formula-src # ls -ld chrony
lrwxrwxrwx 1 root root 21 Feb 16 13:24 chrony -> chrony-formula/chrony
c0c6bbd75dbf:/srv/formula-src # more /etc/salt/minion.d/roots.conf 
file_roots:
  base:
    - /srv/salt
    - /usr/share/salt-formulas/states
    - /srv/formula-src
c0c6bbd75dbf:/srv/formula-src # salt-call --local state.show_sls chrony
local:
    ----------
    chrony-package-install-pkg-installed:
....
```

However this, in my opinion slightly cleaner variant, does not:

```
c0c6bbd75dbf:/srv/formula-src # ls -ld states/chrony
lrwxrwxrwx 1 root root 24 Feb 16 13:23 states/chrony ->
../chrony-formula/chrony
c0c6bbd75dbf:/srv/formula-src # more /etc/salt/minion.d/roots.conf 
file_roots:
  base:
    - /srv/salt
    - /usr/share/salt-formulas/states
    - /srv/formula-src/states
c0c6bbd75dbf:/srv/formula-src # salt-call --local state.show_sls chrony
local:
    - No matching sls found for 'chrony' in env 'base'
```

Then again, this variant does:

```
c0c6bbd75dbf:/srv/formula-src # ls -ld chrony                      
lrwxrwxrwx 1 root root 34 Feb 16 13:32 chrony ->
repositories/chrony-formula/chrony
c0c6bbd75dbf:/srv/formula-src # more /etc/salt/minion.d/roots.conf 
file_roots:
  base:
    - /srv/salt
    - /usr/share/salt-formulas/states
    - /srv/formula-src
c0c6bbd75dbf:/srv/formula-src # salt-call --local state.show_sls chrony
local:
    ----------
    chrony-package-install-pkg-installed:
        ----------
...
```

I think I will opt for this last variant as to not clutter our repository top
level with both, submodules and symlinks.

> would still affect symlinks that are targetting files which are part of the same file root.

Thanks, makes sense.


You are receiving this mail because: