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

Make integration test script executable #521

Merged
merged 3 commits into from
Aug 8, 2022

Conversation

karlhigley
Copy link
Contributor

Forgot to chmod +x test_integration.sh

@karlhigley karlhigley added chore Infrastructure update ci labels Aug 8, 2022
@karlhigley karlhigley added this to the Merlin 22.08 milestone Aug 8, 2022
@karlhigley karlhigley requested a review from nv-alaiacano August 8, 2022 16:44
@karlhigley karlhigley self-assigned this Aug 8, 2022
@github-actions
Copy link

github-actions bot commented Aug 8, 2022

Documentation preview

https://nvidia-merlin.github.io/Merlin/review/pr-521

@nvidia-merlin-bot
Copy link
Contributor

Click to view CI Results
GitHub pull request #521 of commit 3fcdbb9293bd272c646eef0ae1c27b993387c01c, no merge conflicts.
Running as SYSTEM
Setting status of 3fcdbb9293bd272c646eef0ae1c27b993387c01c to PENDING with url https://10.20.13.93:8080/job/merlin_merlin/325/console and message: 'Pending'
Using context: Jenkins
Building on master in workspace /var/jenkins_home/workspace/merlin_merlin
using credential systems-login
 > git rev-parse --is-inside-work-tree # timeout=10
Fetching changes from the remote Git repository
 > git config remote.origin.url https://github.com/NVIDIA-Merlin/Merlin # timeout=10
Fetching upstream changes from https://github.com/NVIDIA-Merlin/Merlin
 > git --version # timeout=10
using GIT_ASKPASS to set credentials login for merlin-systems
 > git fetch --tags --force --progress -- https://github.com/NVIDIA-Merlin/Merlin +refs/pull/521/*:refs/remotes/origin/pr/521/* # timeout=10
 > git rev-parse 3fcdbb9293bd272c646eef0ae1c27b993387c01c^{commit} # timeout=10
Checking out Revision 3fcdbb9293bd272c646eef0ae1c27b993387c01c (detached)
 > git config core.sparsecheckout # timeout=10
 > git checkout -f 3fcdbb9293bd272c646eef0ae1c27b993387c01c # timeout=10
Commit message: "Make integration test script executable"
 > git rev-list --no-walk 11cead933631b01c75fc6831a755aa693cb84236 # timeout=10
[merlin_merlin] $ /bin/bash /tmp/jenkins4439901767127779299.sh
============================= test session starts ==============================
platform linux -- Python 3.8.10, pytest-7.1.2, pluggy-1.0.0
rootdir: /var/jenkins_home/workspace/merlin_merlin/merlin
plugins: anyio-3.6.1, xdist-2.5.0, forked-1.4.0, cov-3.0.0
collected 3 items

tests/unit/test_version.py . [ 33%]
tests/unit/examples/test_building_deploying_multi_stage_RecSys.py . [ 66%]
tests/unit/examples/test_scaling_criteo_merlin_models.py . [100%]

======================== 3 passed in 235.59s (0:03:55) =========================
Performing Post build task...
Match found for : : True
Logical operation result is TRUE
Running script : #!/bin/bash
cd /var/jenkins_home/
CUDA_VISIBLE_DEVICES=1 python test_res_push.py "https://api.GitHub.com/repos/NVIDIA-Merlin/Merlin/issues/$ghprbPullId/comments" "/var/jenkins_home/jobs/$JOB_NAME/builds/$BUILD_NUMBER/log"
[merlin_merlin] $ /bin/bash /tmp/jenkins16201972171468518940.sh

@nvidia-merlin-bot
Copy link
Contributor

Click to view CI Results
GitHub pull request #521 of commit 792f14a5f1b0ef07690ba865698bd1a675dab632, no merge conflicts.
Running as SYSTEM
Setting status of 792f14a5f1b0ef07690ba865698bd1a675dab632 to PENDING with url https://10.20.13.93:8080/job/merlin_merlin/326/console and message: 'Pending'
Using context: Jenkins
Building on master in workspace /var/jenkins_home/workspace/merlin_merlin
using credential systems-login
 > git rev-parse --is-inside-work-tree # timeout=10
Fetching changes from the remote Git repository
 > git config remote.origin.url https://github.com/NVIDIA-Merlin/Merlin # timeout=10
Fetching upstream changes from https://github.com/NVIDIA-Merlin/Merlin
 > git --version # timeout=10
using GIT_ASKPASS to set credentials login for merlin-systems
 > git fetch --tags --force --progress -- https://github.com/NVIDIA-Merlin/Merlin +refs/pull/521/*:refs/remotes/origin/pr/521/* # timeout=10
 > git rev-parse 792f14a5f1b0ef07690ba865698bd1a675dab632^{commit} # timeout=10
Checking out Revision 792f14a5f1b0ef07690ba865698bd1a675dab632 (detached)
 > git config core.sparsecheckout # timeout=10
 > git checkout -f 792f14a5f1b0ef07690ba865698bd1a675dab632 # timeout=10
Commit message: "Merge branch 'main' into fix/merlin-integration-tests"
 > git rev-list --no-walk 3fcdbb9293bd272c646eef0ae1c27b993387c01c # timeout=10
[merlin_merlin] $ /bin/bash /tmp/jenkins10563996887088906904.sh
============================= test session starts ==============================
platform linux -- Python 3.8.10, pytest-7.1.2, pluggy-1.0.0
rootdir: /var/jenkins_home/workspace/merlin_merlin/merlin
plugins: anyio-3.6.1, xdist-2.5.0, forked-1.4.0, cov-3.0.0
collected 3 items

tests/unit/test_version.py . [ 33%]
tests/unit/examples/test_building_deploying_multi_stage_RecSys.py . [ 66%]
tests/unit/examples/test_scaling_criteo_merlin_models.py . [100%]

======================== 3 passed in 225.08s (0:03:45) =========================
Performing Post build task...
Match found for : : True
Logical operation result is TRUE
Running script : #!/bin/bash
cd /var/jenkins_home/
CUDA_VISIBLE_DEVICES=1 python test_res_push.py "https://api.GitHub.com/repos/NVIDIA-Merlin/Merlin/issues/$ghprbPullId/comments" "/var/jenkins_home/jobs/$JOB_NAME/builds/$BUILD_NUMBER/log"
[merlin_merlin] $ /bin/bash /tmp/jenkins5497225560246496830.sh

@karlhigley karlhigley merged commit 1903c8c into main Aug 8, 2022
@nv-alaiacano nv-alaiacano deleted the fix/merlin-integration-tests branch August 8, 2022 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Infrastructure update ci
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants