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

Command-line option for specifying Node.Mode (and handling for it on the plugin side) #3

Merged
merged 1 commit into from
May 16, 2013

Conversation

timurstrekalov
Copy link
Contributor

  • Added a command-line option to the Client to specify the mode to be
    sent to the master node (either 'normal' or 'exclusive' which
    correspond to values in hudson.model.Node.Mode)
  • Added a specific OptionHandler to parse those
  • Added another @QueryParameter to hudson.plugins.swarm.PluginImpl to
    make sure the mode gets correctly applied
  • Added Mode as a parameter to the SwarmSlave constructor that's being
    called from PluginImpl

- Added a command-line option to the Client to specify the mode to be
  sent to the master node (either 'normal' or 'exclusive' which
  correspond to values in hudson.model.Node.Mode)
- Added a specific OptionHandler to parse those
- Added another @QueryParameter to hudson.plugins.swarm.PluginImpl to
  make sure the mode gets correctly applied
- Added Mode as a parameter to the SwarmSlave constructor that's being
  called from PluginImpl
@buildhive
Copy link

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

@timurstrekalov
Copy link
Contributor Author

I guess the biggest reason I would like this feature is that one should be able to make the ad-hoc Swarm slaves "exclusive" so that they do not accidentally interfere with other jobs (which you might not want).

ssogabe added a commit that referenced this pull request May 16, 2013
Command-line option for specifying Node.Mode (and handling for it on the plugin side)
@ssogabe ssogabe merged commit ac1d57b into jenkinsci:master May 16, 2013
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.

3 participants