Skip to content

Ignore screenshot section if upload failed

Désirée Chevalier requested to merge ignore-screenshot-if-upload-failed into master

What does this MR do and why?

Fixes errors found in https://ops.gitlab.net/gitlab-org/quality/canary/-/jobs/7174527

  1. Skip setting the screenshot section when the screenshot upload fails, otherwise the failure issue is not created. Screenshots are a "nice to have" so this shouldn't block issue creation.
  2. Error Code 500 needs to be added to the list of errors whose screenshots should not be included in failure issues.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Désirée Chevalier

Merge request reports