Skip to content

Conversation

@jozala
Copy link
Contributor

@jozala jozala commented Jun 26, 2025

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.

Previously BRANCH was used, but if there is newer version of the
dependency, the incorrect version could be used.
@jozala jozala self-assigned this Jun 26, 2025
@jozala jozala requested a review from a team as a code owner June 26, 2025 09:37
@jozala jozala added :Delivery/Build Build or test infrastructure v8.19.0 v9.1.0 v8.17.8 v9.0.4 v8.18.4 auto-backport Automatically create backport pull requests when merged labels Jun 26, 2025
@elasticsearchmachine elasticsearchmachine added Team:Delivery Meta label for Delivery team v9.2.0 labels Jun 26, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-delivery (Team:Delivery)

@jozala jozala requested a review from breskeby June 26, 2025 09:37
@jozala jozala added auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) and removed auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) labels Jun 26, 2025
Need to use BRANCH for snapshot DRA builds because full version is not
available in artifacts-snapshot API.
jozala added 2 commits June 26, 2025 12:43
Log what URL is used to fetch manifest and make the curl command fail in
 case of error like 404 response.
@jozala jozala added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Jun 26, 2025
@jozala jozala merged commit c057945 into elastic:main Jun 26, 2025
5 of 8 checks passed
@jozala jozala deleted the dra-resolve-manifest-with-version branch June 26, 2025 10:57
jozala added a commit to jozala/elasticsearch that referenced this pull request Jun 26, 2025
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.
jozala added a commit to jozala/elasticsearch that referenced this pull request Jun 26, 2025
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.
jozala added a commit to jozala/elasticsearch that referenced this pull request Jun 26, 2025
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.
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
8.19
8.17
9.0
8.18
7.17 Commit could not be cherrypicked due to conflicts
9.2 The branch "9.2" is invalid or doesn't exist

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 130084

jozala added a commit to jozala/elasticsearch that referenced this pull request Jun 26, 2025
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.
@jozala
Copy link
Contributor Author

jozala commented Jun 26, 2025

💚 All backports created successfully

Status Branch Result
7.17

Questions ?

Please refer to the Backport tool documentation

elasticsearchmachine pushed a commit that referenced this pull request Jun 26, 2025
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.
elasticsearchmachine pushed a commit that referenced this pull request Jun 26, 2025
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.
elasticsearchmachine pushed a commit that referenced this pull request Jun 26, 2025
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.
elasticsearchmachine pushed a commit that referenced this pull request Jun 26, 2025
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.

(cherry picked from commit c057945)

# Conflicts:
#	.ci/scripts/resolve-dra-manifest.sh
elasticsearchmachine pushed a commit that referenced this pull request Jun 26, 2025
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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport pending :Delivery/Build Build or test infrastructure >non-issue Team:Delivery Meta label for Delivery team v7.17.30 v8.17.8 v8.18.4 v8.19.0 v9.0.4 v9.1.0 v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants