Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ci] Attach correct build artifact link to build scan when multiple are uploaded #105530

Merged
merged 2 commits into from
Feb 23, 2024

Conversation

brianseeders
Copy link
Contributor

@brianseeders brianseeders commented Feb 14, 2024

Some jobs upload multiple artifacts, which messes up the link that gets attached to the build scan (because all of the artifact IDs get appended together).

@brianseeders brianseeders added >non-issue Team:Delivery Meta label for Delivery team :Delivery/Tooling Developer tooliing and automation labels Feb 14, 2024
@brianseeders brianseeders marked this pull request as ready for review February 14, 2024 22:27
@brianseeders brianseeders requested a review from a team as a code owner February 14, 2024 22:27
@elasticsearchmachine
Copy link
Collaborator

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

String[] partsB = b.split(" ");
// ISO-8601 timestamps can be sorted lexicographically
return partsA[1].compareTo(partsB[1]);
}).toList();
Copy link
Contributor

@mark-vieira mark-vieira Feb 14, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: I think we can simplify this somewhat by sorting in reverse order and calling findFirst() instead of toList() since we only care about the most recent one.

@brianseeders brianseeders merged commit e568f70 into elastic:main Feb 23, 2024
13 of 15 checks passed
@brianseeders brianseeders deleted the fix-multiple-artifacts branch February 23, 2024 15:28
brianseeders added a commit to brianseeders/elasticsearch that referenced this pull request Feb 23, 2024
brianseeders added a commit to brianseeders/elasticsearch that referenced this pull request Feb 23, 2024
@brianseeders
Copy link
Contributor Author

💔 Some backports could not be created

Status Branch Result
8.13
8.12
7.17 Conflict resolution was aborted by the user

Manual backport

To create the backport manually run:

backport --pr 105530

Questions ?

Please refer to the Backport tool documentation

brianseeders added a commit that referenced this pull request Feb 23, 2024
brianseeders added a commit that referenced this pull request Feb 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Delivery/Tooling Developer tooliing and automation >non-issue Team:Delivery Meta label for Delivery team v7.17.19 v8.12.3 v8.13.1 v8.14.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants