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

Multiple bug fixes #5

Merged
merged 3 commits into from
Nov 6, 2014
Merged

Multiple bug fixes #5

merged 3 commits into from
Nov 6, 2014

Conversation

scottanderson
Copy link

No description provided.

@cloudbees-pull-request-builder

plugins » parameterized-remote-trigger-plugin #19 SUCCESS
This pull request looks good

@jenkinsadmin
Copy link
Member

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

@scottanderson scottanderson changed the title Do not encode the entire URL, only the job name Multiple bug fixes Oct 29, 2014
scottanderson referenced this pull request Oct 30, 2014
parameterized. Previously, a remote job with spaces in the name would
cause a failure while attempting to check for job parameters. This
caused each build trigger step to retry the parameterization check 5
times and then swallow the resulting AbortException.
Save TRIGGERED_BUILD_NUMBER and, if blocking, TRIGGERED_BUILD_RESULT to
the build environment. Prior to this change, it was necessary to parse
the log file, which was very hacky and only possible on the master. This
can be used by later build steps and downstream jobs.
If multiple jobs are started at the same time, it can be challenging to
determine the correct build numbers for each, as Jenkins does not
respond with the build number. To help mitigate this risk, check the
expected build parameters, and if they do not match then scan +-2 build
numbers looking for the correct build.
morficus added a commit that referenced this pull request Nov 6, 2014
@morficus morficus merged commit a8c7aa6 into jenkinsci:master Nov 6, 2014
cashlalala pushed a commit that referenced this pull request May 11, 2018
 Prevent ConcurrentModificationException in BuildInfoExporterAction.build
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.

4 participants