Skip to content
This repository was archived by the owner on May 19, 2023. It is now read-only.

Gpuciscripts clean and update - #285

Closed
msadang wants to merge 11 commits into
rapidsai:branch-0.17from
msadang:gpuciscripts-update
Closed

msadang wants to merge 11 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:59
@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)

@dillon-cullinan

Copy link
Copy Markdown
Contributor

Duplicate of: #276

@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 commented Nov 19, 2020

Copy link
Copy Markdown
Contributor Author

rerun tests

@msadang msadang changed the title [WIP] - Gpuciscripts clean and update Gpuciscripts clean and update Nov 19, 2020
@msadang msadang added 2 - In Progress enhancement New feature or request non-breaking Non-breaking change labels Dec 3, 2020
@ajschmidt8 ajschmidt8 closed this Dec 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

2 - In Progress enhancement New feature or request non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants