Skip to content

Improve review apps

Rémy Coutable requested to merge improve-review-apps into master

What does this MR do?

Done:

  • Enable autoscaling for the review-apps-ee cluster, with maximum 30 preemptible nodes
  • Merge the review-app-image job into the review job
  • Made the review job automatic
  • Soft cleanup review apps before setting up a review app
  • Hard cleanup stale (last updated > 7 days) Helm releases
    • This ensures that if review app stopping failed at the step above, we still clear the actual Helm release (and the environment will also be cleared in GitLab thanks to "delete the environment" step above

To do:

  • Make sure review app deployment works, currently blocked by Error: error when upgrading: current Tiller version is newer, use --force-upgrade to downgrade
  • Make sure review app deployment works, currently blocked by Error: release review-improve-re-ffvbep failed: timed out waiting for the condition
    • That may be caused by the self-signed TLS certificate, which prevents the runner from connecting properly:

      E  [31;1mERROR: Registering runner... failed               [0;m  [31;1mrunner[0;m=rE3Vn6af [31;1mstatus[0;m=couldn't execute POST against https://gitlab-review-improve-re-ffvbep.gitlab-review.app/api/v4/runners: Post https://gitlab-review-improve-re-ffvbep.gitlab-review.app/api/v4/runners: x509: certificate is valid for ingress.local, not gitlab-review-improve-re-ffvbep.gitlab-review.app
      
      E  [31;1mPANIC: Failed to register this runner. Perhaps you are having network problems[0;m 

      I've created https://gitlab.com/gitlab-com/infrastructure/issues/4735 to ask for a proper certificate.

  • Move the review apps cleanup to a scheduled pipeline (twice per day should be enough) instead of before setting up a review app
  • Disable the review job for forks

Are there points in the code the reviewer needs to double check?

  • What do you think about stopping GitLab review/* environments after 5 days?
  • What do you think about deleting GitLab review/* environments after 6 days?
  • What do you think about cleaning up Helm releases after 7 days?

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Fixes https://gitlab.com/gitlab-org/gitlab-ee/issues/6875.

Edited by Marin Jankovski

Merge request reports