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.

Sign up for GitHub

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

Jump to bottom

feat: add new gradle workflow for ARM #229

Merged
merged 17 commits into from
Apr 19, 2021
Merged

feat: add new gradle workflow for ARM #229

merged 17 commits into from
Apr 19, 2021

Conversation

bshaffer
Copy link
Contributor

No description provided.

@bshaffer
Copy link
Contributor Author

This appears to be broken due to the conscript dependency not being available for Aarch64 (see google/conscrypt#858). The error from gradle is:

java.lang.UnsatisfiedLinkError: no conscrypt_openjdk_jni-linux-aarch_64 in java.library.path

@bshaffer
Copy link
Contributor Author

I do not know why, but the following is failing:

io.grpc.testing.integration.Http2OkHttpTest > deadlineExceeded FAILED
    java.lang.AssertionError: ClientCall started after deadline exceeded: -0.129141655s from now

@bshaffer bshaffer mentioned this pull request Apr 13, 2021
Merged
@jamesward
Copy link
Collaborator

I do not know why, but the following is failing:

io.grpc.testing.integration.Http2OkHttpTest > deadlineExceeded FAILED
    java.lang.AssertionError: ClientCall started after deadline exceeded: -0.129141655s from now

I think that test is flappy.

@jamesward
Copy link
Collaborator

BTW, I saw that Gradle 7.0 is out and has ARM support: https://docs.gradle.org/7.0/release-notes.html

@bshaffer
Copy link
Contributor Author

bshaffer commented Apr 16, 2021 •
edited
Loading

@jamesward I think you are right and the failures in part were flappy! It's now ready for review :)

EDIT: Addressed flakey tests in #246

@bshaffer bshaffer requested a review from jamesward April 19, 2021 18:46
@jamesward jamesward merged commit dcb2829 into master Apr 19, 2021
@jamesward jamesward deleted the add-arm-actions branch April 19, 2021 18:47
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jamesward jamesward jamesward approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bshaffer @jamesward