Buildgraph - required/after when node has errors

Development Pipeline & Plugins
, ,
1

Hi,

we are trying to create some log postprocess when horde build is finished. So we modified buildgraph, for another agent and node that will execute some python scripts. We need to execute this node when some other node is finished, so there is Requires or After attribute for it.

But both seems like to not execute when required/after node throws error. Is there some posibility, how to create dependency, that will execute when dependant node has errors?

Thanks and regards

Lubos Suk

[Attachment Removed]

2

Hi there,

I was doing some testing over the buildgraph and I noticed that the “after” parameter on the node alone, isnt enough for it to use the dependacy chain when it builds the graph which might be an oversight, I am aunsure if that is inteded functionality. The documentaiton hints that Requires must complete and after should always trigger. However after might simply be for ordering. I’ll submit a bug report and see if its expected functionality or not.

Regarding downstream jobs,

The way Build Graph is structured, it throws exception upon a failure preventing any downstream nodes from ever being considered.

There are 2 ways I could think you could achieve this off the top of my head:

* Create a custom task in C# within buildgraph that does the task that you care about and does some clean up before positing failures.

* If you are using Horde, Use the horde API located at (HordeURL/swagger/index.html) and have another service poll for job updates.

If you would like I can reassign this to the horde team as a feature request.

Kind Regards

Keegan Gibson

[Attachment Removed]

3

Hi there,

Here is a link to where you can find the bug report mentioned above once it has been reflected publicly.

An engine dev will investigate the bug at a later date. We don’t provide updates on EPS, but progress can be followed on that public issue tracker page.

Keegan has given some good advice for where to look next, so I will close this case, but feel free to respond here if you have any follow-up questions.

Thanks,

Hayden

[Attachment Removed]