-
Bug
-
Resolution: Fixed
-
Minor
-
None
-
Jenkins ver. 1.421
Jenkins Promoted Builds Plugin 2.2
When a promotion is renamed, all subsequent promotions of the new name will also promote it with the old name.
To reproduce, I did the following:
- Create a job test-downstream, leave all settings as-is.
- Create a job test-upstream, and set the following:
- Promote builds when ...
- Name: First promotion
- When the following downstream projects build successfully: test-downstream
- Post-build actions, build other projects: test-downstream
- Promote builds when ...
- Save settings, and then click Configure again.
- Change the name of the promotion from First promotion to Second promotion.
- Save.
- Run the build of test-upstream, wait for the build of test-upstream and test-downstream to complete (about 10s with quiet periods etc).
Now you'll observe that the new build has two gold stars: First promotion and Second promotion. Going to /job/test-upstream/1/promotion/ will show First promotion as "Pending promotion" for some reason. It shouldn't be there at all!
- is related to
-
-
- Resolved
-
-
-
- Resolved
-
-
-
- Resolved
-
[JENKINS-10423] Builds get double promotions after promotion has been renamed
Jørgen Tjernø
created issue -
Jørgen Tjernø
made changes -
| Description |
Original:
When a promotion is renamed, all subsequent promotions of the new name will also promote it with the old name. To reproduce, I did the following: Create a job test-downstream, leave all settings as-is. Create a job test-upstream, and set the following: Promote builds when ... Name: First promotion When the following downstream projects build successfully: test-downstream Post-build actions, build other projects: test-downstream Now save settings, and then click Configure again. Change First promotion to Second promotion and save. Run the build of test-upstream, wait for the build of test-upstream and test-downstream to complete (about 10s with quiet periods etc), observe that the new build has two gold stars: First promotion *and* Second promotion. Going to {{/job/test-upstream/1/promotion/}} will show First promotion as "Pending promotion" for some reason. |
New:
When a promotion is renamed, all subsequent promotions of the new name will also promote it with the old name. To reproduce, I did the following: # Create a job test-downstream, leave all settings as-is. # Create a job test-upstream, and set the following: #* Promote builds when ... #** Name: ??First promotion?? #** When the following downstream projects build successfully: ??test-downstream?? #* Post-build actions, build other projects: ??test-downstream?? # Save settings, and then click Configure again. # Change the name of the promotion from ??First promotion?? to ??Second promotion??. # Save. # Run the build of test-upstream, wait for the build of test-upstream and test-downstream to complete (about 10s with quiet periods etc). Now you'll observe that the new build has two gold stars: First promotion *and* Second promotion. Going to {{/job/test-upstream/1/promotion/}} will show First promotion as "Pending promotion" for some reason. It shouldn't be there at all! |
| Link |
New:
This issue is related to |
Jørgen Tjernø
added a comment - This seems like a hack or workaround rather than a fix - why is there disabled promotion there to begin with? Shouldn't the old one be deleted when it's renamed?
Jørgen Tjernø
added a comment - This seems like a hack or workaround rather than a fix - why is there disabled promotion there to begin with? Shouldn't the old one be deleted when it's renamed? Code changed in jenkins
User: Nicolas De Loof
Path:
src/main/java/hudson/plugins/promoted_builds/PromotionProcess.java
http://jenkins-ci.org/commit/promoted-builds-plugin/b0c0504e3d76b59389e72e6925ae15b5f93e8c70
Log:
[FIXED JENKINS-10423] don't run promotionProcess that are disabled
| Resolution | New: Fixed [ 1 ] | |
| Status | Original: Open [ 1 ] | New: Resolved [ 5 ] |
Code changed in jenkins
User: Stephen Connolly
Path:
src/main/java/hudson/plugins/promoted_builds/PromotionProcess.java
http://jenkins-ci.org/commit/promoted-builds-plugin/b2f7cedb25664a5a85f075c99b14f674105360fc
Log:
Merge pull request #14 from ndeloof/master
fix for JENKINS-10423
Compare: https://github.com/jenkinsci/promoted-builds-plugin/compare/314089c...b2f7ced
Integrated in 
plugins_promoted-builds #57
[FIXED JENKINS-10423] don't run promotionProcess that are disabled
Nicolas De Loof :
Files :
- src/main/java/hudson/plugins/promoted_builds/PromotionProcess.java
@Jørgen
It depends on why the promotion process was renamed.
If the promotion process is renamed, you may want to retain the old promotions with the old name, or you may want to rename all the old promotions too. In reality JENKINS-8783 should be fixed by adding a three button confirm rather than [Yes]/[No] it should be [Rename process and all previous promotions of this process]/[Rename process leaving all previous promotions with the old name]/[No]
That would make it clear why/how you can have disabled processes left lying around after a rename IMHO
proposed fix https://github.com/jenkinsci/promoted-builds-plugin/pull/14