Conditional publisher stops working (Bug #240)


Added by Dmitry Repkin almost 5 years ago. Updated over 4 years ago.


Status:Closed Start date:12/13/2012
Priority:Normal Due date:
Assignee:- % Done:

0%

Category:- Spent time: -
Target version:1.8.4
Affected version:1.8.2

Description

After updating from 1.6 to 1.8.2 I found conditionalPublisher stops working in my projects.

Test config is attached.


TestComponent.config (1.3 kB) Dmitry Repkin, 12/13/2012 07:48 pm


Related issues

related to CruiseControl.NET - Bug #214: forcebuild task failed when run in coditional task New 10/12/2012

History

Updated by Olivier Sannier almost 5 years ago

Could it be that your exec task is failing and thus the condition "success" is not met ?
I'm asking because running a batch file directly like you did is not what the documentation is recommending.
Rather, you should use cmd /C

Updated by Dmitry Repkin almost 5 years ago

Condition exactly returns "success". If I change return code in batch file to return "error", CCNet finely shows an "Fail" for task.

Downgrading to 1.6 resolves the problem.

Updated by Sergey Enns almost 5 years ago

IntegrationResult check in BuildPublisher, but miss in result.Clone() within ConditionalPublisher
Here the patch: https://github.com/kyser/CruiseControl.NET/commit/2635cf3da5ef7d56c6ddcf11d7090de0c9755caf

Updated by Ruben Willems over 4 years ago

I added a test on the integration result.clone.
this did not exist yet. :-(
And I found some other issues also :-( :-( :-(

for example every integration result as it's own ID (a guid)
should a clone have the same id or not?

following items are also NOT mapped
° CCNetBuildDate
° CCNetBuildTime
° Failure users
° Failure tasks

I would expect a clone to map these also, correct or not?
if a clone is used in a parallel task, what gives in all those combinations?

Updated by Ruben Willems over 4 years ago

  • Status changed from New to In Progress

Updated by Ruben Willems over 4 years ago

  • Target version set to 1.8.4

Updated by Ruben Willems over 4 years ago

  • Status changed from In Progress to Resolved

Updated by Ruben Willems over 4 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF