Skip to content

Conversation

@quic-sanirudh
Copy link
Contributor

@quic-sanirudh quic-sanirudh commented Jun 6, 2022

This patch updates the docker image tag used for ci_hexagon to a recent build that uses LLVM 14.0.0 release.

cc @Mousius @areusch @driazati @mehrdadh

@mehrdadh
Copy link
Member

mehrdadh commented Jun 6, 2022

I will retag the image and push the docker image so we can test this PR. Here is the commands for reference:

docker pull tlcpackstaging/ci_hexagon:20220603-203325-cee74c9f8
docker tag tlcpackstaging/ci_hexagon:20220603-203325-cee74c9f8 tlcpack/ci-hexagon:20220603-203325-cee74c9f8
docker push tlcpack/ci-hexagon:20220603-203325-cee74c9f8

@mehrdadh
Copy link
Member

mehrdadh commented Jun 6, 2022

image is here: https://hub.docker.com/layers/ci-hexagon/tlcpack/ci-hexagon/20220603-203325-cee74c9f8/images/sha256-2cf1e317957c21dfdc692a11006af9b118615b60e6097dc2e5fb9d3ffa27f43b?context=repo

@driazati could you help us to test this PR at docker staging?

@driazati
Copy link
Member

driazati commented Jun 7, 2022

Submitted #11607 which will run CI with the new docker image (and apparently since it's the same commit as this PR even gets linked to the CI runs here)

@mehrdadh
Copy link
Member

mehrdadh commented Jun 7, 2022

Please don't merge this since hexagon tests are skipped in the CI at the moment.

@quic-sanirudh
Copy link
Contributor Author

@mehrdadh Do you think it's okay to merge this PR now?

@quic-sanirudh quic-sanirudh force-pushed the ci_hexagon_docker_tag branch from 4a1391d to b7fd874 Compare June 13, 2022 14:52
@mehrdadh
Copy link
Member

@driazati confirmed that new image has passed the CI.
thanks @driazati and @quic-sanirudh!!

@mehrdadh mehrdadh merged commit d92a773 into apache:main Jun 14, 2022
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.

3 participants