955136b56d
It seems the deploy function causes the job to fail if it doesn't succeed. That wasn't the intent as we want to curl the Review App after the deploy finished (even if it failed) because sometimes the Review App is just a bit long to be ready. This change wraps the Review App deployment with "set +e"/"set -e" to ensure that the job doesn't fail right away if the deploy fails. Signed-off-by: Rémy Coutable <remy@rymai.me> |
||
|---|---|---|
| .. | ||
| frontend | ||
| review_apps | ||
| build_assets_image | ||
| clean-old-cached-assets | ||
| create_mysql_user.sh | ||
| create_postgres_user.sh | ||
| detect-new-flaky-examples | ||
| gitaly-test-build | ||
| gitaly-test-spawn | ||
| gitaly_test.rb | ||
| insert-rspec-profiling-data | ||
| lint-changelog-yaml | ||
| lint-conflicts.sh | ||
| lint-doc.sh | ||
| lint-rugged | ||
| merge-reports | ||
| merge-simplecov | ||
| no-ee-check | ||
| prepare_build.sh | ||
| prune-old-flaky-specs | ||
| schema_changed.sh | ||
| security-harness | ||
| static-analysis | ||
| sync-reports | ||
| trigger-build | ||
| trigger-build-docs | ||
| utils.sh | ||