Skip to content

Group and limit pipeline warnings

Payton Burdette requested to merge refactor-pipeline-warnings-ux into master

What does this MR do?

This MR changes the UX around pipeline warnings.

We now will group the warning instead of using separate callouts for each warning. For now we are using a summary element to have collapsible warnings for a temporary solution. We also set a cap on the warnings to show no more than 25 warnings for a pipeline. There are use cases where a lot of warnings can arise for pipeline configurations.

These warnings are shown in 3 places, CI Lint view, Run manual pipeline view, Pipeline graph view.

Follow up for: #219431 (closed)

Screenshots

Screen_Shot_2020-08-18_at_9.46.41_AM

Screen_Shot_2020-08-18_at_9.46.49_AM

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 Payton Burdette

Merge request reports