Skip to content

[CI] Enable DGL cugaph nightly CI#4573

Merged
Rhett-Ying merged 9 commits intodmlc:masterfrom
Rhett-Ying:cugraphNightlyCI
Sep 22, 2022
Merged

[CI] Enable DGL cugaph nightly CI#4573
Rhett-Ying merged 9 commits intodmlc:masterfrom
Rhett-Ying:cugraphNightlyCI

Conversation

@Rhett-Ying
Copy link
Copy Markdown
Collaborator

Description

Based on #4525 which breaks the CI. Let me open a new one to be verified by CI automatically. Probably caused bu incorrect True which should be true in Jenkinsfile. @VibhuJawa

Checklist

Please feel free to remove inapplicable items for your PR.

  • The PR title starts with [$CATEGORY] (such as [NN], [Model], [Doc], [Feature]])
  • Changes are complete (i.e. I finished coding on this PR)
  • All changes have test coverage
  • Code is well-documented
  • To the best of my knowledge, examples are either not affected by this change,
    or have been fixed to be compatible with this change
  • Related issue is referred in this PR
  • If the PR is for a new model/paper, I've updated the example index here.

Changes

@dgl-bot
Copy link
Copy Markdown
Collaborator

dgl-bot commented Sep 16, 2022

Commit ID: None

Build ID: 1

Status: ✅ CI test succeeded

Report path: link

Full logs path: link

@dgl-bot
Copy link
Copy Markdown
Collaborator

dgl-bot commented Sep 16, 2022

Commit ID: df233e5

Build ID: 2

Status: ❌ CI test failed in Stage [PyTorch Cugraph GPU Build].

Report path: link

Full logs path: link

@Rhett-Ying
Copy link
Copy Markdown
Collaborator Author

@dgl-bot

@dgl-bot
Copy link
Copy Markdown
Collaborator

dgl-bot commented Sep 19, 2022

Commit ID: d23ee9497a6fd1888f09008c72b215b6192abec0

Build ID: 3

Status: ❌ CI test failed in Stage [PyTorch Cugraph GPU Build].

Report path: link

Full logs path: link

@VibhuJawa
Copy link
Copy Markdown
Contributor

@Rhett-Ying, I pushed a fix to your PR here Rhett-Ying#2 and also verified that it runs locally.

See below test that i ran:

### Build Command
docker build -f test_cugraph_nightly_file.Dockerfile -t dgl_nightly .

### Test Command
docker run --gpus=all -it dgl_nightly bash -c 'cd /dgl && bash tests/scripts/cugraph_unit_test.sh cugraph
FROM rapidsai/cugraph_nightly_torch-cuda:11.5-base-ubuntu18.04-py3.9-pytorch1.12.0-rapids22.10
RUN cd / && git clone https://github.com/dmlc/dgl.git 
RUN cd / && cd dgl && git submodule update --init --recursive 
RUN cd / && cd dgl && git config --global user.email "vjawa@nvidia.com" && git fetch origin pull/4573/head:PR_4573 && git merge PR_4573
RUN cd /dgl && bash tests/scripts/build_dgl.sh cugraph

tests/cugraph/test_basics.py::test_dummy PASSED                                                                                                                                                                                        [ 33%]
tests/cugraph/test_basics.py::test_to_cugraph_conversion 


PASSED                                                                                                                                                                        [ 66%]
tests/cugraph/test_basics.py::test_from_cugraph_conversion PASSED                                                                                                                                                                      [100%]

@dgl-bot
Copy link
Copy Markdown
Collaborator

dgl-bot commented Sep 20, 2022

Commit ID: 671063e270e0f45692a3aca620550c06f26b6ef2

Build ID: 4

Status: ❌ CI test failed in Stage [PyTorch Cugraph GPU Build].

Report path: link

Full logs path: link

@jermainewang
Copy link
Copy Markdown
Member

One question for the team. I remember we have agreed to have cuGraph as an optional check so it will not block the development. It looks like this PR adds cuGraph to the main pipeline. Does it make cuGraph a mandatory CI check?

@Rhett-Ying
Copy link
Copy Markdown
Collaborator Author

this PR aims to update the image used for cugraph. As for making it optional check, it's probably need another PR to achieve this. I don't know how to achieve this for now.

Comment thread Jenkinsfile
@dgl-bot
Copy link
Copy Markdown
Collaborator

dgl-bot commented Sep 21, 2022

Commit ID: 5998808c52a64cbe761702f4f4271cadc8b79e0a

Build ID: 5

Status: ❌ CI test failed in Stage [PyTorch Cugraph GPU].

Report path: link

Full logs path: link

@dgl-bot
Copy link
Copy Markdown
Collaborator

dgl-bot commented Sep 21, 2022

Commit ID: 994077d

Build ID: 6

Status: ✅ CI test succeeded

Report path: link

Full logs path: link

@dgl-bot
Copy link
Copy Markdown
Collaborator

dgl-bot commented Sep 21, 2022

Commit ID: 0c4adc5

Build ID: 7

Status: ✅ CI test succeeded

Report path: link

Full logs path: link

@dgl-bot
Copy link
Copy Markdown
Collaborator

dgl-bot commented Sep 21, 2022

Commit ID: 11ce9da

Build ID: 8

Status: ✅ CI test succeeded

Report path: link

Full logs path: link

@dgl-bot
Copy link
Copy Markdown
Collaborator

dgl-bot commented Sep 22, 2022

Commit ID: 375a7ba

Build ID: 9

Status: ✅ CI test succeeded

Report path: link

Full logs path: link

@Rhett-Ying Rhett-Ying merged commit be11409 into dmlc:master Sep 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants