You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2022/09/23 09:06:15 UTC

[GitHub] [tvm] leandron opened a new pull request, #12884: [CI] Add Zephyr-SDK binaries to PATH env. in ci_cortexm

leandron opened a new pull request, #12884:
URL: https://github.com/apache/tvm/pull/12884

   In recent test rounds with updated images, it seems Zephyr-SDK binaries such as various QEMU related files are missing from $PATH, which makes Zephyr tests to fail with, e.g. "qemu-system-i386: command not found".
   
   This PR adds those missing binaries to $PATH.
   
   cc @areusch @gromero @mehrdadh @driazati for reviews


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [tvm] gromero commented on pull request #12884: [CI] Add Zephyr-SDK binaries to PATH env. in ci_cortexm

Posted by GitBox <gi...@apache.org>.
gromero commented on PR #12884:
URL: https://github.com/apache/tvm/pull/12884#issuecomment-1256201222

   @leandron Change LGTM.
   
   Just for the records the "command not found" errors we get within the current docker images are not exactly the same `E           tvm._ffi.base.TVMError: MicroSessionTimeoutError: session start handshake failed after 10s` errors we're initially chasing in https://ci.tlcpack.ai/blue/organizations/jenkins/docker-images-ci%2Fdocker-image-run-tests/detail/docker-image-run-tests/231/tests/  However it could be related since another QEMU can be picked up since the PATH is not set correctly (picked up from host, or elsewhere in the container env).
   
   Let's see CI goes. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [tvm] leandron commented on pull request #12884: [CI] Add Zephyr-SDK binaries to PATH env. in ci_cortexm

Posted by GitBox <gi...@apache.org>.
leandron commented on PR #12884:
URL: https://github.com/apache/tvm/pull/12884#issuecomment-1256376915

   In parallel, I'm also validation this change in `ci-docker-staging`: https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/ci-docker-staging/286/pipeline


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [tvm] lhutton1 commented on pull request #12884: [CI] Add Zephyr-SDK binaries to PATH env. in ci_cortexm

Posted by GitBox <gi...@apache.org>.
lhutton1 commented on PR #12884:
URL: https://github.com/apache/tvm/pull/12884#issuecomment-1256389791

   Thanks @leandron @gromero!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [tvm] lhutton1 merged pull request #12884: [CI] Add Zephyr-SDK binaries to PATH env. in ci_cortexm

Posted by GitBox <gi...@apache.org>.
lhutton1 merged PR #12884:
URL: https://github.com/apache/tvm/pull/12884


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org