Skip to content

Commit

Permalink
Set sccache timeout (#432)
Browse files Browse the repository at this point in the history
For local builds we don't want sccache timing out so quickly. This
number matches [what we use for other CI
builds](https://github.com/rapidsai/gha-tools/blob/main/tools/rapids-configure-sccache#L9).

This PR also adds some documentation of how include/exclude files are
handled by `rapids-make-conda-env`.

---------

Co-authored-by: Paul Taylor <[email protected]>
Co-authored-by: James Lamb <[email protected]>
  • Loading branch information
3 people authored Dec 31, 2024
1 parent 71649b5 commit e1168d7
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 2 deletions.
1 change: 1 addition & 0 deletions .devcontainer/rapids.Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,7 @@ ENV PYTHONDONTWRITEBYTECODE="1"

ENV SCCACHE_REGION="us-east-2"
ENV SCCACHE_BUCKET="rapids-sccache-devs"
ENV SCCACHE_IDLE_TIMEOUT=900
ENV AWS_ROLE_ARN="arn:aws:iam::279114543810:role/nv-gha-token-sccache-devs"

ENV HISTFILE="/home/coder/.cache/._bash_history"
Expand Down
2 changes: 1 addition & 1 deletion features/src/rapids-build-utils/devcontainer-feature.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"name": "NVIDIA RAPIDS devcontainer build utilities",
"id": "rapids-build-utils",
"version": "25.2.3",
"version": "25.2.4",
"description": "A feature to install the RAPIDS devcontainer build utilities",
"containerEnv": {
"BASH_ENV": "/etc/bash.bash_env"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,9 @@
#
# Options that require values:
# -e,--exclude <file> Path(s) to requirement files of packages to exclude.
# Can also be a file descriptor like '<(echo libucx)'.
# -i,--include <file> Path(s) to requirement files of packages to include.
# Can also be a file descriptor like '<(echo libucx)'.
# -k,--key <key> Only include the key(s)
# --matrix-entry <entry> Matrix entries, in the form 'key=value' to be added to the '--matrix' arg
# of rapids-dependency-file-generator.
Expand Down Expand Up @@ -48,11 +50,13 @@ make_conda_dependencies() {

local -a _exclude=();
local exc; for exc in "${exclude[@]}"; do
# append '-f' so each file's contents will be treated as a list of patterns for 'grep'
_exclude+=(-f "${exc}");
done

local -a _include=();
local inc; for inc in "${include[@]}"; do
# append '-f' so each file's contents will be treated as a list of patterns for 'grep'
_include+=(-f "${inc}");
done

Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
#!/usr/bin/env bash

# Usage:
# rapids-make-conda-dependencies [OPTION]...
# rapids-make-pip-dependencies [OPTION]...
#
# Generate a combined pip requirements file for all repos.
#
Expand All @@ -11,7 +11,9 @@
#
# Options that require values:
# -e,--exclude <file> Path(s) to requirement files of packages to exclude.
# Can also be a file descriptor like '<(echo libucx)'.
# -i,--include <file> Path(s) to requirement files of packages to include.
# Can also be a file descriptor like '<(echo libucx)'.
# -k,--key <key> Only include the key(s)
# --matrix-entry <entry> Matrix entries, in the form 'key=value' to be added to the '--matrix' arg
# of rapids-dependency-file-generator.
Expand Down Expand Up @@ -49,11 +51,13 @@ make_pip_dependencies() {

local -a _exclude=();
local exc; for exc in "${exclude[@]}"; do
# append '-f' so each file's contents will be treated as a list of patterns for 'grep'
_exclude+=(-f "${exc}");
done

local -a _include=();
local inc; for inc in "${include[@]}"; do
# append '-f' so each file's contents will be treated as a list of patterns for 'grep'
_include+=(-f "${inc}");
done

Expand Down

0 comments on commit e1168d7

Please sign in to comment.