Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Browse files
Browse the repository at this point in the history
Change primary template identification to be its name, not its AMI.
Many useful configurations will have multiple slave templates that use the same AMI (but differ in other aspects). This patch changes various parts of the code to properly identify templates by their names (in the 'description' field) instead of their AMI. As an example, the manual slave provisioning button, before this patch, would allow the user to select a template from a drop-down list, but then launched the slave using the template's AMI, which could have resulted in the wrong template being used for the launch. This patch also changes the global config page so that the 'description' field is the first field in each template definition, to emphasize its importance. Addresses [JENKINS-7960] and [JENKINS-15158].
- Loading branch information
Showing
4 changed files
with
19 additions
and
17 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters