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

acceptance: support running acceptance tests on ARM64 #89232

Merged
merged 1 commit into from
Oct 11, 2022

Conversation

healthy-pod
Copy link
Contributor

@healthy-pod healthy-pod commented Oct 3, 2022

This code change makes it possible to run acceptance tests
on ARM64.

Release note: None
Epic: CRDB-1463

@cockroach-teamcity
Copy link
Member

This change is Reviewable

@healthy-pod healthy-pod force-pushed the acceptance-support-arm branch 22 times, most recently from 740ec1d to a6c1bc2 Compare October 7, 2022 03:02
@healthy-pod healthy-pod marked this pull request as ready for review October 7, 2022 03:52
@healthy-pod healthy-pod requested review from a team, rickystewart and otan October 7, 2022 03:53
@@ -34,6 +35,7 @@ func TestComposeGSSPython(t *testing.T) {
}

func TestComposeFlyway(t *testing.T) {
skip.UnderArm64(t)
Copy link
Collaborator

Choose a reason for hiding this comment

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

would be nice to have a comment or description here for why it's skipped.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agree, actually I initially skipped it because flyway:flyway6 image didn't support linux/arm64. Now I see that linux/arm64 is supported since 8.4.3 so I will update the image and unskip the test instead

This code change makes it possible to run acceptance tests
on ARM64.

Release note: None
Epic: CRDB-1463
@healthy-pod healthy-pod force-pushed the acceptance-support-arm branch from a6c1bc2 to 60a19e4 Compare October 7, 2022 17:17
@healthy-pod
Copy link
Contributor Author

bors r=otan,rafiss

@craig craig bot merged commit 2a9f158 into cockroachdb:master Oct 11, 2022
@craig
Copy link
Contributor

craig bot commented Oct 11, 2022

Build succeeded:

@healthy-pod
Copy link
Contributor Author

blathers backport release-22.2

@healthy-pod
Copy link
Contributor Author

blathers backport release-22.2.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants