Skip to content

Improve the way we check for Review App availability

Rémy Coutable requested to merge test-review-app-deployment into master

What does this MR do and why?

The new way of verifying deployment is a bit more verbose and directly checks the /users/sign_in: https://gitlab.com/gitlab-org/gitlab/-/jobs/3339490661#L179.

Also, we actually keep the curl logs as artifacts.

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 Rémy Coutable

Merge request reports