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

Swarm 3.3 download URL doesn't exist #739

Closed
mmarseglia opened this issue Mar 17, 2017 · 4 comments
Closed

Swarm 3.3 download URL doesn't exist #739

mmarseglia opened this issue Mar 17, 2017 · 4 comments

Comments

@mmarseglia
Copy link

in manifests/slave.pp the client jar file name is defined as:
$client_jar = "swarm-client-${version}-jar-with-dependencies.jar"

There is no swarm client 3.3 with that name at https://repo.jenkins-ci.org/releases/org/jenkins-ci/plugins/swarm-client/3.3/

../
swarm-client-3.3-javadoc.jar        10-Feb-2017 08:20  80.47 KB
swarm-client-3.3-javadoc.jar.md5    10-Feb-2017 08:20  32 bytes
swarm-client-3.3-javadoc.jar.sha1   10-Feb-2017 08:20  40 bytes
swarm-client-3.3-sources.jar        10-Feb-2017 08:20  17.78 KB
swarm-client-3.3-sources.jar.md5    10-Feb-2017 08:20  32 bytes
swarm-client-3.3-sources.jar.sha1   10-Feb-2017 08:20  40 bytes
swarm-client-3.3.jar                10-Feb-2017 08:20  1.50 MB
swarm-client-3.3.jar.md5            10-Feb-2017 08:20  32 bytes
swarm-client-3.3.jar.sha1           10-Feb-2017 08:20  40 bytes
swarm-client-3.3.pom                10-Feb-2017 08:20  3.89 KB
swarm-client-3.3.pom.md5            10-Feb-2017 08:20  32 bytes
swarm-client-3.3.pom.sha1           10-Feb-2017 08:20  40 bytes
@SpencerMalone
Copy link

It seems that without announcing it in the changelogs, they removed the dependency-less jar and renamed the jar-with-dependencies to swarm-client-3.3.jar, so hopefully we just have to support that new URL name if version >= 3.3. See: https://issues.jenkins-ci.org/browse/JENKINS-42138

@jantman
Copy link

jantman commented Apr 11, 2017

Yeah, that's what happened. I suppose the right way would be to change jenkins::slave::client_jar based on $version, but for future-proofing, it would probably be good if that was exposed as a param as well.

@jantman
Copy link

jantman commented Apr 11, 2017

It looks like #746 includes a code fix for this, but no test updates.

@jhoblitt
Copy link
Member

jhoblitt commented Feb 8, 2018

This has indeed been fixed on master for sometime, so I'm going to close this ticket.

@jhoblitt jhoblitt closed this as completed Feb 8, 2018
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

No branches or pull requests

4 participants