From 21c24c235796739fa6b9719112f0910e149f3e9b Mon Sep 17 00:00:00 2001 From: Andrey Lushnikov Date: Mon, 8 Feb 2021 16:19:37 -0800 Subject: [PATCH] devops: do not check for logs existance when building browsers (#5367) In our *old world*, browser buildbots were polling repo and were trying to rebuild browsers every 5 minutes. They relied on the presence of either build or build log on the CDN to avoid continiously re-building. In the new world, we trigger builds only when we change the `BUILD_NUMBER` files, so there's no polling any more. This patch removes the check for log presence so that those builds that were failing due to misconfiguration (e.g. Visual Studio license required updated) could be restarted just from the GitHub UI. --- browser_patches/checkout_build_archive_upload.sh | 4 ---- 1 file changed, 4 deletions(-) diff --git a/browser_patches/checkout_build_archive_upload.sh b/browser_patches/checkout_build_archive_upload.sh index 4a6eade82566a..47b7627872a09 100755 --- a/browser_patches/checkout_build_archive_upload.sh +++ b/browser_patches/checkout_build_archive_upload.sh @@ -233,11 +233,7 @@ if ! [[ ($2 == '-f') || ($2 == '--force') ]]; then if ./upload.sh "${BUILD_BLOB_PATH}" --check; then echo "Build is already uploaded - no changes." exit 0 - elif ./upload.sh "${LOG_BLOB_PATH}" --check; then - echo "This build has already been attempted - skip building." - exit 0 fi - echo "Build is missing and has not been attempted - rebuilding" else echo "Force-rebuilding the build." fi