-
Bug
-
Resolution: Fixed
-
Minor
-
None
as you can see in the following code, in our case, the run is always an instance of WorkflowRun, and the runResult is always Result.SUCCESS, so no user would be add to recipient.
if (run instanceof AbstractBuild) { Set<User> users = ((AbstractBuild<?,?>)run).getCulprits(); RecipientProviderUtilities.addUsers(users, context.getListener(), env, to, cc, bcc, debug); } else if (runResult != null && runResult.isWorseThan(Result.SUCCESS)) { List<Run<?, ?>> builds = new ArrayList<>(); Run<?, ?> build = run; while (build != null) { final Result buildResult = build.getResult(); if (buildResult != null) { if (buildResult.isWorseThan(Result.SUCCESS)) { debug.send("Including build %s with status %s", build.getId(), buildResult); builds.add(build); } else { break; } } build = build.getPreviousCompletedBuild(); } Set<User> users = RecipientProviderUtilities.getChangeSetAuthors(builds, debug); RecipientProviderUtilities.addUsers(users, context.getListener(), env, to, cc, bcc, debug); }
- is blocked by
-
-
- Closed
-
[JENKINS-40653] CulpritsRecipientProvider does not work with pipeline
Makson Lee
created issue -
Makson Lee
made changes -
| Description |
New:
as you can see in the following code, in our case, the run is always an instance of WorkflowRun, and the runResult is always Result.SUCCESS, so no user would be add to recipient. https://github.com/jenkinsci/email-ext-plugin/blob/master/src/main/java/hudson/plugins/emailext/plugins/recipients/CulpritsRecipientProvider.java {code:java} if (run instanceof AbstractBuild) { Set<User> users = ((AbstractBuild<?,?>)run).getCulprits(); RecipientProviderUtilities.addUsers(users, context.getListener(), env, to, cc, bcc, debug); } else if (runResult != null && runResult.isWorseThan(Result.SUCCESS)) { List<Run<?, ?>> builds = new ArrayList<>(); Run<?, ?> build = run; while (build != null) { final Result buildResult = build.getResult(); if (buildResult != null) { if (buildResult.isWorseThan(Result.SUCCESS)) { debug.send("Including build %s with status %s", build.getId(), buildResult); builds.add(build); } else { break; } } build = build.getPreviousCompletedBuild(); } Set<User> users = RecipientProviderUtilities.getChangeSetAuthors(builds, debug); RecipientProviderUtilities.addUsers(users, context.getListener(), env, to, cc, bcc, debug); } {code} |
| Resolution | New: Fixed [ 1 ] | |
| Status | Original: Open [ 1 ] | New: Closed [ 6 ] |
| Link |
New:
This issue is blocked by |