Sonatype Nexus support (archive::nexus) about puppet-archive HOT 9 CLOSED

The other two define types use a function because the checksum file needs processing. The archive parameter checksum_source was intended for this, but not yet implemented. I think checksum_url might be a better name?

from puppet-archive.

@nanliu If you're working with a remote source (http, ftp, s3), checksum_url is probably a better name.

from puppet-archive.

@jairojunior please review latest changes and provide feedback if you run into any issues. Also I'm happy to take a PR for a define type wrapper for nexus.

from puppet-archive.

Sorry didn't mean to close issue. Per last comment, I will accept a nexus define type wrapper. I'll list it as experimental along with the go/artifactory wrapper in the readme.

from puppet-archive.

Thank you.

I was able to finish archive::nexus with the latest changes, but I have a small concern about performance.

Check this gist: https://gist.github.com/jairojunior/071a5256abeb1d39e54a

In the second run, with HTTP checksum URL it tooks 12s, with HTTPS, 6s, and hardcoded checksum only takes 0.4s

I have a module that uses net/http to query a HTTP API and the request are really fast (~100ms). Do you have any idea of what is causing this?

Meanwhile, I'll work on rspec tests for archive::nexus.

from puppet-archive.

I'm not able to reproduce the difference (hardcode v.s. checksum_url difference is negligible for multiple runs). Can you time curl to the url in comparison:
https://gist.github.com/nanliu/0d3f5bf4a53431f81ed2

from puppet-archive.

My bad, it's something with my puppetlabs/centos-6.6-64-puppet vagrant setup:

time curl -i -X GET 'https://oss.sonatype.org/service/local/artifact/maven/content?g=io.hawt&a=hawtio-web&v=1.4.36&p=war.md5&r=releases'

Took 5.5s there and only 0.4s in my machine

puppet apply --debug --trace --profile site.pp --modulepath '/home/jjunior/wildfly-vagrant/deploy/modules'

Is doing fine, less than a second.Thanks again.

from puppet-archive.

@nanliu I think we should close this issue.

from puppet-archive.

Fix by #54

from puppet-archive.