Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

git_pillar mountpoint ignores the root parameter #50364

Closed
sathieu opened this issue Nov 2, 2018 · 1 comment

Comments

Projects
None yet
2 participants
@sathieu
Copy link
Contributor

commented Nov 2, 2018

Description of Issue/Question

Setup

Consider the following master config:

ext_pillar:
  - git:
    - master https://mydomain.tld/top.git:
      - root: pillar
      - mountpoint: ''
    - master https://mydomain.tld/myapp1.git:
      - root: pillar/myapp1
      - mountpoint: myapp1
    - master https://mydomain.tld/myapp2.git:
      - root: pillar/myapp2
      - mountpoint: myapp2

Steps to Reproduce Issue

The root parameter is ignored.

Longer story: I want all my git repos to be virtually relative to /srv. The top.git repo only contains pillar/top.sls which loads the proper "app" pillar corresponding to the minion id. This prevents an app repo to inject pillar data to minions of another app (each app repo can have different commiters).

Versions Report

(Provided by running salt --versions-report. Please also mention any differences in master/minion versions.)

@Ch3LL

This comment has been minimized.

Copy link
Contributor

commented Nov 5, 2018

ping @terminalmage can you comment on if this is expected behavior or just something we need to add?

@Ch3LL Ch3LL added this to the Approved milestone Nov 5, 2018

sathieu added a commit to sathieu/salt that referenced this issue Nov 24, 2018

sathieu added a commit to sathieu/salt that referenced this issue Nov 24, 2018

sathieu added a commit to sathieu/salt that referenced this issue Nov 24, 2018

sathieu added a commit to sathieu/salt that referenced this issue Nov 28, 2018

sathieu added a commit to sathieu/salt that referenced this issue Nov 28, 2018

sathieu added a commit to sathieu/salt that referenced this issue Dec 5, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.