Skip to content
This repository was archived by the owner on Jul 7, 2026. It is now read-only.

Gpuciscripts clean and update - #215

Merged
msadang merged 6 commits into
rapidsai:branch-0.17from
msadang:gpuciscripts-update
Dec 9, 2020
Merged

msadang merged 6 commits into
rapidsai:branch-0.17from
msadang:gpuciscripts-update

Conversation

@msadang

@msadang msadang commented Nov 10, 2020

Copy link
Copy Markdown
Contributor

In CI folder the following changes have been made:

  • Remove logger function and replace all logger calls with gpuci_logger

  • Removed all ellipses ... from logger messages

  • Prepend /opt to the conda path in PATH variable

  • Replace conda with gpuci_conda_retry for build and install calls (Did not replace conda activate with gpuci_conda_retry)

  • Replace source activate with the following:
    . /opt/conda/etc/profile.d/conda.sh
    conda activate rapids

  • Replace conda list with more verbose information:
    conda info
    conda config --show-sources
    conda list --show-channel-urls

  • Update Copyright year in the top of scripts to include 2020 if applicable

  • Set PARALLEL_LEVEL to ${PARALLEL_LEVEL:-4}

  • Set gpuci_conda_retry flags in cpu/build.sh
    Setup 'gpuci_conda_retry' for build retries (results in 2 total attempts)
    export GPUCI_CONDA_RETRY_MAX=1
    export GPUCI_CONDA_RETRY_SLEEP=30

  • Replace calls to gcc and g++ with $CC and $CXX respectively

@msadang
msadang requested a review from a team as a code owner November 10, 2020 01:53
@GPUtester

Copy link
Copy Markdown
Contributor

Please update the changelog in order to start CI tests.

View the gpuCI docs here.

@ajschmidt8 ajschmidt8 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like my review comments from the raft PR below will apply here as well (remove gpuci_logger function and fix export PARALLEL_LEVEL=${PARALLEL_LEVEL:-4}).

NVIDIA/raft#92 (review)

@msadang

msadang commented Nov 13, 2020

Copy link
Copy Markdown
Contributor Author

It looks like my review comments from the raft PR below will apply here as well (remove gpuci_logger function and fix export PARALLEL_LEVEL=${PARALLEL_LEVEL:-4}).

rapidsai/raft#92 (review)

gpuci_logger and parallel level changed.

@msadang msadang added 2 - In Progress Currenty a work in progress improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Dec 2, 2020
@msadang msadang added 3 - Ready for Review Ready for review by team and removed 2 - In Progress Currenty a work in progress labels Dec 2, 2020
@msadang msadang changed the title [WIP] - Gpuciscripts clean and update Gpuciscripts clean and update Dec 3, 2020
@msadang
msadang merged commit 9291331 into rapidsai:branch-0.17 Dec 9, 2020
@msadang
msadang deleted the gpuciscripts-update branch December 9, 2020 19:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

3 - Ready for Review Ready for review by team improvement Improvement / enhancement to an existing function non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants