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

[FEATURE REQUEST] salt-cloud does not support using using salt file server for file_map #58178

Open
ggiesen opened this issue Aug 12, 2020 · 1 comment
Labels
Feature new functionality including changes to functionality and code refactors, etc. File Servers Salt-Cloud
Projects
Milestone

Comments

@ggiesen
Copy link
Contributor

ggiesen commented Aug 12, 2020

Is your feature request related to a problem? Please describe.
I started using the file_map feature of salt-cloud (on vmware specifically) and it appears it does not support using the salt file server as the file source (ie salt://scripts/some_script.sh). This means I have to use yet another mechanism to do version control/CI/CD for these types of scripts, instead of using gitfs.

This works:

file_map:
  /srv/scripts/some_script.sh: /root/some_script.sh

This does not:

file_map:
  salt://scripts/some_script.sh: /root/some_script.sh

Describe the solution you'd like
Add support for salt file server script sources for file_map in salt-cloud

Describe alternatives you've considered
Use local files with a separate synchronization mechanism (apart from salt's gitfs)

@ggiesen ggiesen added the Feature new functionality including changes to functionality and code refactors, etc. label Aug 12, 2020
@Akm0d Akm0d added this to the Aluminium milestone Aug 12, 2020
@ggiesen
Copy link
Contributor Author

ggiesen commented Nov 30, 2020

I would love to see this apply more broadly to any code that is executed by the master (reactor scripts, etc). It would be nice if the master had access to the salt:// filesystem just as the minions do.

@sagetherage sagetherage added the Aluminium Release Post Mg and Pre Si label Dec 7, 2020
@sagetherage sagetherage removed the Aluminium Release Post Mg and Pre Si label Feb 24, 2021
@sagetherage sagetherage modified the milestones: Aluminium, Approved Feb 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature new functionality including changes to functionality and code refactors, etc. File Servers Salt-Cloud
Projects
No open projects
Magnesium
  
Awaiting triage
Development

No branches or pull requests

3 participants