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

Support multiple filesystems anywhere files can be specified #13971

Closed
ryan-lane opened this issue Jul 3, 2014 · 9 comments · Fixed by #53462
Closed

Support multiple filesystems anywhere files can be specified #13971

ryan-lane opened this issue Jul 3, 2014 · 9 comments · Fixed by #53462
Labels
Feature new functionality including changes to functionality and code refactors, etc. stale
Milestone

Comments

@ryan-lane
Copy link
Contributor

Current some modules support multiple filesystems, like http, ftp, s3, etc.. It would be really awesome if this support was available everywhere. For instance, it would be really awesome to be able to call state.template on an S3 URL, or to be able to provide an S3 URL to pkg.installed in the sources section.

@cachedout
Copy link
Contributor

Good suggestion! I agree that would be awfully handy.

@xenophonf
Copy link
Contributor

I too would like to be able to write a pkg.installed state that looked like:

jdk8:
  pkg:
    - installed
    - sources:
      - jdk1.8.0_40: s3://ibrsp/depot/jdk8/jdk-8u40-linux-x64.rpm

Of course, such a state currently results in errors similar to the following (posted here for the sake of others' web searches):

[INFO    ] Running state [jdk8] at time 19:03:21.571871
[INFO    ] Executing state pkg.installed for jdk8
[INFO    ] Executing command 'yum -q clean expire-cache' in directory '/root'
[INFO    ] Executing command 'yum -q check-update ' in directory '/root'
[INFO    ] Executing command 'rpm -qp --queryformat "\'%{NAME}_|-%{VERSION}_|-%{RELEASE}_|-%{ARCH}_|-none\'" s3://ibrsp/depot/jdk8/jdk-8u40-linux-x64.rpm' in directory '/root'
[INFO    ] Executing command 'yum -y     install s3://ibrsp/depot/jdk8/jdk-8u40-linux-x64.rpm' in directory '/root'
[ERROR   ] Command 'yum -y     install s3://ibrsp/depot/jdk8/jdk-8u40-linux-x64.rpm' failed with return code: 1
[ERROR   ] output: Loaded plugins: fastestmirror
Loading mirror speeds from cached hostfile
 * base: ftp.heanet.ie
 * epel: ftp.heanet.ie
 * epel-debuginfo: ftp.heanet.ie
 * epel-source: ftp.heanet.ie
 * extras: ftp.heanet.ie
 * nux-dextop: mirror.li.nux.ro
 * updates: ftp.heanet.ie
No package s3://ibrsp/depot/jdk8/jdk-8u40-linux-x64.rpm available.
Error: Nothing to do

@stale
Copy link

stale bot commented Oct 21, 2017

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

If this issue is closed prematurely, please leave a comment and we will gladly reopen the issue.

@stale stale bot added the stale label Oct 21, 2017
@stale
Copy link

stale bot commented Oct 21, 2017

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

If this issue is closed prematurely, please leave a comment and we will gladly reopen the issue.

2 similar comments
@stale
Copy link

stale bot commented Oct 21, 2017

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

If this issue is closed prematurely, please leave a comment and we will gladly reopen the issue.

@stale
Copy link

stale bot commented Oct 21, 2017

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

If this issue is closed prematurely, please leave a comment and we will gladly reopen the issue.

@stale stale bot closed this as completed Oct 28, 2017
@lorengordon
Copy link
Contributor

Please reopen

@lorengordon
Copy link
Contributor

Turned out to be reasonably easy, I think... #53462

lorengordon added a commit to lorengordon/salt that referenced this issue Dec 21, 2019
@lorengordon
Copy link
Contributor

Hurrah! PR was merged!

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. stale
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants