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

Discussion on migrating remote_file module to voxpupuli #187

Open
TraGicCode opened this issue Nov 19, 2018 · 5 comments
Open

Discussion on migrating remote_file module to voxpupuli #187

TraGicCode opened this issue Nov 19, 2018 · 5 comments

Comments

@TraGicCode
Copy link
Contributor

TraGicCode commented Nov 19, 2018

Hey Guys,

I ran into this amazing module and noticed @reidmv reached out the to maintainer who is no longer actively working on this module. He suggested transferring ownership to voxpupuli so this approved module which is cross-platform and built with types and providers ( which also has 1 million downloads ) doesn't die. It also does some advanced things like automatically creating file resources and embedding them in the catalog during compilation.

https://forge.puppet.com/lwf/remote_file

While we already have a "download_file" module ( which isn't cross platform ) and the archive module ( which just does alot ). I would like to get some feedback about the fate of this module.

Thanks

@bastelfreak
Copy link
Member

Has the remote_file module any features that our archive one hasn't?

@reidmv
Copy link
Member

reidmv commented Nov 19, 2018

@bastelfreak remote_file is laser-focused on describing files that should be present, sourced from a remote HTTP(S) source. It is very much NOT about downloading-and-extracting zip files, tarballs, etc. It's a "do one thing and do it well" unix philosophy kind of module.

The main utility it currently provides beyond what is available in Puppet's file type is proxy support and ability to set SSL no verify (verify_peer parameter).

Compared to archive, its pros are clearer user experience and readability of Puppet code for the target use case (which is not archive files), and a rough optional ensure=latest capability based on last-modified headers.

@TraGicCode
Copy link
Contributor Author

To tack on with what reid said, the file resource still isn’t Quite full featured. For example it has issues downloading github releases from github.com and I’ve run into problems elsewhere like downloading signed urls from google and azure cloud.

The archive module does handle a lot of file downloading functionality ( unfortunately archive is turning into a Swiss Army knife ) so there is overlap. this remote_file module has cross platform types and providers and in my eyes would be a great opportunity to deprecate something like the “download” file module under vox which would be a win in regards to quality and cross-platform functionality

@ekohl
Copy link
Member

ekohl commented Mar 12, 2024

This is an old issue by now. As part of a triage we're going through them. Is this still relevant or can we close it?

@kenyon
Copy link
Member

kenyon commented Mar 12, 2024

I didn't know about this proposal to migrate remote_file when I replaced remote_file with archive in voxpupuli/puppet-ca_cert#69, so I'd say we might as well just use the archive module instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

5 participants