Skip to content

WIP: Solve inconsistent DAG pipeline behaviors

Furkan Ayhan requested to merge 213080-resolve-dag-inconsistent-behaviors into master

What does this MR do?

Related to #213080 (closed) and #31264 (closed)

This MR is created with the inspiration of this comment:

needs here serves to have test starting as soon as all the needs completes

It means that, needs should not care about whether dependent jobs are failed or succeeded. The goal of needs is whether dependent jobs are completed or not. when keyword of the job should decide if dependent jobs need to be success or failure.

To this end, I've changed the expected behaviors of yaml test cases, and added some new basic tests.

Question

Is this a breaking change ?

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Furkan Ayhan

Merge request reports