Skip to content

WIP: Implement showing CI bridge error messages

Furkan Ayhan requested to merge 200016-show-bridge-error-reason into master

WARNING

This is just an approach. Also I wrote some # TODO: comments in code changes to emphasize some wrong approaches.

The proposal in the issue is not suitable for cases when no downstream pipeline created. So in this MR, I suggest to store failure reason, instead of reading from downstream pipeline.

What does this MR do?

Related to: #200016 (closed)

When a bridge job fails to create downstream pipeline, we only show if it is failed or not.

With this, we will start showing error messages.

Screenshots

Example of similar error case stated in #204378 (closed)

Screen_Shot_2020-03-26_at_17.49.17

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