Skip to content
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

Gpuciscripts clean and update #627

Merged
merged 7 commits into from
Dec 1, 2020
Merged

Gpuciscripts clean and update #627

merged 7 commits into from
Dec 1, 2020

Conversation

msadang
Copy link
Contributor

@msadang msadang commented Nov 10, 2020

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:51
@GPUtester
Copy link
Contributor

Please update the changelog in order to start CI tests.

View the gpuCI docs here.

@msadang
Copy link
Contributor Author

msadang commented Nov 13, 2020

gpuci_logger and parallel level changed.

@msadang msadang changed the title [WIP] - Gpuciscripts clean and update Gpuciscripts clean and update Nov 30, 2020
@msadang msadang added 2 - In Progress Currently a work in progress gpuCI labels Nov 30, 2020
@msadang msadang self-assigned this Dec 1, 2020
@msadang msadang added 3 - Ready for review Ready for review by team 5 - Ready to Merge Testing and reviews complete, ready to merge improvement Improvement / enhancement to an existing function non-breaking Non-breaking change and removed 2 - In Progress Currently a work in progress labels Dec 1, 2020
@kkraus14 kkraus14 added 6 - Okay to Auto-Merge and removed 3 - Ready for review Ready for review by team labels Dec 1, 2020
@rapids-bot rapids-bot bot merged commit 62ee3c7 into rapidsai:branch-0.17 Dec 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5 - Ready to Merge Testing and reviews complete, ready to merge gpuCI 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.

5 participants