From c05794510dcb5d25de13f1f84ca2d84b2123bbcb Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Mariusz=20J=C3=B3zala?= <377355+jozala@users.noreply.github.com> Date: Thu, 26 Jun 2025 12:57:02 +0200 Subject: [PATCH] Use ES_VERSION to resolve manifest for ML and Beats (#130084) Previously BRANCH was used in all cases, but if there is a newer version of the dependency, the incorrect version could be used. BRANCH is still used for snapshot DRA builds because full version is not available in artifacts-snapshot API. Additionally, added log what URL is used to fetch manifest and make the curl command fail in case of error like 404 response. --- .ci/scripts/resolve-dra-manifest.sh | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/.ci/scripts/resolve-dra-manifest.sh b/.ci/scripts/resolve-dra-manifest.sh index 37ede297fdc0..67d86dbe5aba 100755 --- a/.ci/scripts/resolve-dra-manifest.sh +++ b/.ci/scripts/resolve-dra-manifest.sh @@ -6,7 +6,8 @@ strip_version() { } fetch_build() { - curl -sS https://artifacts-$1.elastic.co/$2/latest/$3.json \ + >&2 echo "Checking for build id: https://artifacts-$1.elastic.co/$2/latest/$3.json" + curl -sSf https://artifacts-$1.elastic.co/$2/latest/$3.json \ | jq -r '.build_id' } @@ -15,7 +16,15 @@ BRANCH="${BRANCH:-$2}" ES_VERSION="${ES_VERSION:-$3}" WORKFLOW=${WORKFLOW:-$4} -LATEST_BUILD=$(fetch_build $WORKFLOW $ARTIFACT $BRANCH) +if [[ "$WORKFLOW" == "staging" ]]; then + LATEST_BUILD=$(fetch_build $WORKFLOW $ARTIFACT $ES_VERSION) +elif [[ "$WORKFLOW" == "snapshot" ]]; then + LATEST_BUILD=$(fetch_build $WORKFLOW $ARTIFACT $BRANCH) +else + echo "Unknown workflow: $WORKFLOW" + exit 1 +fi + LATEST_VERSION=$(strip_version $LATEST_BUILD) # If the latest artifact version doesn't match what we expect, try the corresponding version branch.