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] Fix Rust permissions for wasmtime and sccache #10015

Merged
merged 1 commit into from
Jan 27, 2022

Conversation

Mousius
Copy link
Member

@Mousius Mousius commented Jan 21, 2022

Previously this was ran as part of ubuntu_install_rust.sh, as we now have multiple scripts which write as the container build user we have to fix up each time to ensure future users don't break.

See failure here: https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/ci-docker-staging/210/pipeline/58

@Mousius Mousius force-pushed the rust-permissions branch 3 times, most recently from 24d9191 to bc2a7a1 Compare January 25, 2022 09:00
Previously this was ran as part of `ubuntu_install_rust.sh`, as we now have multiple scripts which write as the container build user we have to fix up each time to ensure future users don't break.
@Mousius
Copy link
Member Author

Mousius commented Jan 27, 2022

Ping @leandron / @areusch / @jroesch 😸

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 thanks @Mousius!

@leandron leandron merged commit f161bc2 into apache:main Jan 27, 2022
sunggg pushed a commit to sunggg/tvm that referenced this pull request Jan 29, 2022
Previously this was ran as part of `ubuntu_install_rust.sh`, as we now have multiple scripts which write as the container build user we have to fix up each time to ensure future users don't break.
ylc pushed a commit to ylc/tvm that referenced this pull request Feb 16, 2022
Previously this was ran as part of `ubuntu_install_rust.sh`, as we now have multiple scripts which write as the container build user we have to fix up each time to ensure future users don't break.
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.

2 participants