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

Add Compute Library tests to Jenkins for AArch64 CI #8394

Merged
merged 1 commit into from
Jul 4, 2021
Merged

Add Compute Library tests to Jenkins for AArch64 CI #8394

merged 1 commit into from
Jul 4, 2021

Conversation

u99127
Copy link
Contributor

@u99127 u99127 commented Jul 2, 2021

Now that we have hardware testing on by default, Actually run this.

leandron
leandron approved these changes Jul 3, 2021
View reviewed changes
Copy link
Contributor

@leandron leandron left a comment

Choose a reason for hiding this comment

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

LGTM

@masahi masahi merged commit a00d211 into apache:main Jul 4, 2021
@masahi
Copy link
Member

masahi commented Jul 4, 2021

@u99127 @leandron @tqchen ARM compute lib tests are failing in CI:
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/main/1255/pipeline
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-8366/16/pipeline

@u99127
Copy link
Contributor Author

u99127 commented Jul 4, 2021 •
edited
Loading

@u99127 @leandron @tqchen ARM compute lib tests are failing in CI:
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/main/1255/pipeline
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-8366/16/pipeline

That is very weird. The same tests were passing locally on my machine that is running Ubuntu 18.04 and pretty much the same environment as in upstream CI for a few days.

@u99127 @leandron @tqchen ARM compute lib tests are failing in CI:
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/main/1255/pipeline
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-8366/16/pipeline

Maybe we should revert this till new docker images are spun up as in #8177 before merging this in ?

That is the only difference between my environment and the CI environment that is currently running that I can spot. Sorry about the breakage.

u99127 pushed a commit to u99127/tvm that referenced this pull request Jul 4, 2021
Revert "Actually add Compute Library tests to the Jenkins File (apach…
20cedcd 
…e#8394)"

This reverts commit a00d211 till we
can understand why this is failing in CI but not on developer machines.
@u99127
Copy link
Contributor Author

u99127 commented Jul 4, 2021

@u99127 @leandron @tqchen ARM compute lib tests are failing in CI:
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/main/1255/pipeline
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-8366/16/pipeline

That is very weird. The same tests were passing locally on my machine that is running Ubuntu 18.04 and pretty much the same environment as in upstream CI for a few days.

@u99127 @leandron @tqchen ARM compute lib tests are failing in CI:
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/main/1255/pipeline
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-8366/16/pipeline

Maybe we should revert this till new docker images are spun up as in #8177 before merging this in ?

That is the only difference between my environment and the CI environment that is currently running that I can spot. Sorry about the breakage.

Since this is failing -I've put up a revert at #8400

@leandron leandron mentioned this pull request Jul 5, 2021
Merged
u99127 pushed a commit to u99127/tvm that referenced this pull request Jul 7, 2021
Revert "Actually add Compute Library tests to the Jenkins File (apach…
c9b47d1 
…e#8394)"

This reverts commit a00d211 till we
can understand why this is failing in CI but not on developer machines.
jroesch pushed a commit that referenced this pull request Jul 7, 2021
Revert "Actually add Compute Library tests to the Jenkins File (#8394)…
8fb4cdf 
…" (#8400)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leandron leandron leandron 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.
3 participants
@u99127 @masahi @leandron