Skip to content

Conversation

@breskeby
Copy link
Contributor

@breskeby breskeby commented Feb 7, 2021

This adds some coverage for our build tools dealing with aarch64.

@breskeby breskeby self-assigned this Feb 7, 2021
@breskeby breskeby added :Delivery/Build Build or test infrastructure Team:Delivery Meta label for Delivery team v7.0.0 v8.0.0 labels Feb 7, 2021
@breskeby breskeby changed the title Improve build tools test coverage on aarch64 Ignore BwcVersionsTests on aarch64 Feb 7, 2021
@breskeby breskeby marked this pull request as ready for review February 7, 2021 22:45
@elasticmachine
Copy link
Collaborator

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

@breskeby breskeby added v7.12.0 and removed v7.0.0 labels Feb 7, 2021
@breskeby breskeby marked this pull request as draft February 7, 2021 22:48
@breskeby breskeby force-pushed the ignore-BwcVersionsTest-on-aarch branch from 050ab49 to 4c70698 Compare February 8, 2021 09:59
@breskeby breskeby marked this pull request as ready for review February 8, 2021 19:50
@breskeby breskeby changed the title Ignore BwcVersionsTests on aarch64 build-tools check are fixed on aarch64 Feb 9, 2021

@BeforeClass
public static void setupAll() {
Assume.assumeFalse(Architecture.current() == Architecture.AARCH64);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

For now we just ignore the unit tests on aarch64 as we filter out there anything < 7.12

@breskeby breskeby requested review from mark-vieira and pugnascotia and removed request for pugnascotia February 9, 2021 08:31
@breskeby breskeby force-pushed the ignore-BwcVersionsTest-on-aarch branch from a0ecd3c to a042870 Compare February 9, 2021 12:40
Copy link
Contributor

@mark-vieira mark-vieira left a comment

Choose a reason for hiding this comment

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

LGTM

@breskeby breskeby merged commit bae65dd into elastic:master Feb 10, 2021
@breskeby breskeby deleted the ignore-BwcVersionsTest-on-aarch branch February 10, 2021 08:27
breskeby added a commit to breskeby/elasticsearch that referenced this pull request Feb 15, 2021
* Igmore BwcVersionsTests on aarch64
* Introduce classifier field to Architecture
* Add test coverage for downloading adoptjdk aarch64 jdks
breskeby added a commit that referenced this pull request Feb 16, 2021
* build-tools check are fixed on aarch64  (#68630)
* Ignore BwcVersionsTests on aarch64
* Introduce classifier field to Architecture
* Add test coverage for downloading adoptjdk aarch64 jdks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Delivery/Build Build or test infrastructure Team:Delivery Meta label for Delivery team v7.12.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants