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 remote sources in a source list #32691

Merged
merged 2 commits into from
Apr 21, 2016

Conversation

terminalmage
Copy link
Contributor

This commit modifies the source_list check so that remote sources
(http(s), ftp, etc.) are not fetched more than once. To do so, it
adds the use of __context__ in cp.cache_file and
file.source_list to prevent multiple fetches of a single file in the
same salt run.

@terminalmage
Copy link
Contributor Author

terminalmage commented Apr 19, 2016

Some tests need to be updated, working on this now.

This commit modifies the source_list check so that remote sources
(http(s), ftp, etc.) are not fetched more than once. To do so, it
adds the use of ``__context__`` in ``cp.cache_file`` and
``file.source_list`` to prevent multiple fetches of a single file in the
same salt run.
Added __context__ to test cases to reflect usage of __context__, and
also added file.source_list to mocked funcs for archive.extracted unit
test.
@cachedout
Copy link
Contributor

Very nice.

@cachedout cachedout merged commit 67d0c81 into saltstack:2015.5 Apr 21, 2016
TheBigBear added a commit to TheBigBear/salt that referenced this pull request Apr 22, 2016
Support remote sources in a source list (saltstack#32691)
@terminalmage terminalmage deleted the issue30151 branch July 7, 2016 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants