-
Notifications
You must be signed in to change notification settings - Fork 6.2k
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
Use bazel disk cache for all CI jobs #5144
Conversation
Test PASSed. |
if [[ "$TRAVIS" == "TRAVIS" ]]; then | ||
# Use bazel disk cache if this script is running in Travis. | ||
mkdir -p $HOME/ray-bazel-cache | ||
echo "build --disk_cache=$HOME/ray-bazel-cache" >> $HOME/.bazelrc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this need to be $RAY_ROOT/.bazelrc for it to be effective.
FILE_DIR=$(cd "$(dirname "${BASH_SOURCE:-$0}")"; pwd)
RAY_ROOT = "$FILE_DIR/../.."
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bazel also reads .bazelrc
file in the home directory.
I'm rerunning the CI jobs to see if time will be shorten.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems that Travis doesn't allow modifying files in the home dir.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Test FAILed. |
Test PASSed. |
Test FAILed. |
Test FAILed. |
Test FAILed. |
Test PASSed. |
Test PASSed. |
Test FAILed. |
Test PASSed. |
Test FAILed. |
Test FAILed. |
Test PASSed. |
Test PASSed. |
Test FAILed. |
Test FAILed. |
Test PASSed. |
What do these changes do?
Previously, bazel disk cache was only enabled for Python jobs.
Related issue number
Linter
scripts/format.sh
to lint the changes in this PR.