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][ADRENO] Enhancements to Adreno specific CI utils #15991

Merged
merged 1 commit into from Nov 27, 2023

Conversation

srkreddy1238
Copy link
Contributor

RPC seem to be hanging while running bunch of tests with in single session. This enables running them individually.
Also we don't need rerun with in same session that leads to hang some times. Adding PLATFORM which is needed for run_pytest to summarize the failures.

RPC seem to be hanging while running bunch of tests with in single session.
This enables running them individually.
Also we don't need rerun with in same session that leads to hang some times.
Adding PLATFORM which is needed for run_pytest to summarize the failures.

# cleanup pycache
find . -type f -path "*.pyc" | xargs rm -f
# Test TVM
make cython3

# The RPC to remote Android device has issue of hang after few tests with in CI environments.
Copy link
Member

Choose a reason for hiding this comment

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

Thanks for looking into this! This workaround should be good if it does not significantly increase the overall CI time. It would be great to identify the root cause of the hang issue as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

True, individual runs increase the times significantly that too running on remote device.

I did a bit of a debug on this and it waits infinitely on socket read on host where as device has nothing to write.
Didn't perused further at the moment. I will definitely revisit the RPC issue a bit later.

@yongwww
Copy link
Member

yongwww commented Oct 31, 2023

cc: @driazati @leandron @lhutton1

@srkreddy1238
Copy link
Contributor Author

@yongwww Can u review and merge this ? We have few follow up PR's on this.

The RPC issue I will investigate a little later.

@yongwww yongwww merged commit e5c6f74 into apache:main Nov 27, 2023
20 checks passed
@yongwww
Copy link
Member

yongwww commented Nov 27, 2023

@srkreddy1238 it is landed now.

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.

None yet

2 participants