Skip to content

[FIXED JENKINS-4949] Add ability to deploy to multiple containers#8

Merged
kohsuke merged 1 commit intojenkinsci:masterfrom
randommoocow:JENKINS-4949
Jul 3, 2014
Merged

[FIXED JENKINS-4949] Add ability to deploy to multiple containers#8
kohsuke merged 1 commit intojenkinsci:masterfrom
randommoocow:JENKINS-4949

Conversation

@randommoocow
Copy link

Change allows multiple deployment targets to be specified through
job configuration screen.

Change allows multiple deployment targets to be specified through
job configuration screen.
@buildhive
Copy link

Jenkins » deploy-plugin #10 SUCCESS
This pull request looks good
(what's this?)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thinking about catching the exception on redeploy inside the loop (and rethrow it later?) to update as many deploy targets as possible!? Or leave it this way for fail fast behavior?

@randommoocow
Copy link
Author

One reason to leave it as fail-fast would be because if multiple deploy targets fail for different reasons, it could be difficult to throw a single reason (exception) for the failures.

@h0nIg
Copy link
Member

h0nIg commented Mar 22, 2013

is there any progress on this issue? i would like to deploy my war files instead of using ssh for my deploy :(

@frekele
Copy link
Member

frekele commented Mar 22, 2013

It is possible to manually download and install yourself.
https://buildhive.cloudbees.com/job/jenkinsci/job/deploy-plugin/10/

@bflad
Copy link

bflad commented Jun 10, 2013

Any movement on getting this merged? Thanks.

@jenkinsadmin
Copy link
Member

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

@kohsuke kohsuke merged commit 1fe03c8 into jenkinsci:master Jul 3, 2014
kohsuke added a commit that referenced this pull request Jul 3, 2014
Conflicts:
	pom.xml
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.

8 participants