Skip to content

Gpuciscripts clean and update - #92

Merged
rapids-bot[bot] merged 9 commits into
NVIDIA:branch-0.18from
msadang:gpuciscript-update
Dec 16, 2020
Merged

Gpuciscripts clean and update#92
rapids-bot[bot] merged 9 commits into
NVIDIA:branch-0.18from
msadang:gpuciscript-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:21
@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.

looks good so far. I found a few things that will need to be changed. Dillon's rmm PR below is a good reference (just ignore the project flash specific stuff).

https://github.com/rapidsai/rmm/pull/615/files

Comment thread ci/gpu/build.sh Outdated
Comment thread ci/gpu/build.sh Outdated
Comment thread ci/local/README.md Outdated
@msadang

msadang commented Nov 13, 2020

Copy link
Copy Markdown
Contributor Author

gpuci_logger and parallel level changed.

@msadang msadang added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change 3 - Ready for Review labels Dec 1, 2020
@msadang msadang changed the title [WIP] - Gpuciscripts clean and update Gpuciscripts clean and update Dec 1, 2020
@dillon-cullinan
dillon-cullinan changed the base branch from branch-0.17 to branch-0.18 December 15, 2020 19:55
@rapids-bot
rapids-bot Bot merged commit 0ac7ae2 into NVIDIA:branch-0.18 Dec 16, 2020
loulankxh pushed a commit to loulankxh/raft that referenced this pull request Oct 14, 2025
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 <env> 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

Authors:
  - Mark Sadang <msadang@nvidia.com>
  - Mark Sadang <marksadang@engineer.com>
  - Mark Sadang <63854412+msadang@users.noreply.github.com>

Approvers:
  - AJ Schmidt
  - Dillon Cullinan

URL: NVIDIA#92
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3 - Ready for Review 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