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

Fix ci routes #331

Merged
merged 18 commits into from
May 26, 2022
Merged

Fix ci routes #331

merged 18 commits into from
May 26, 2022

Conversation

jperez999
Copy link
Collaborator

This fixes the CI routes so they all leverage the test scripts in the corresponding repos. allowing each repo to control how unit testing is performed. current setup only works if the code is installed at the root level.

@jperez999 jperez999 requested a review from benfred May 24, 2022 03:54
@github-actions
Copy link

Documentation preview

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

@jperez999
Copy link
Collaborator Author

rerun tests

@nvidia-merlin-bot
Copy link
Contributor

Click to view CI Results
GitHub pull request #331 of commit 32beacb1b04538a2a7b50619b2fcc3fc343ac095, no merge conflicts.
Running as SYSTEM
Setting status of 32beacb1b04538a2a7b50619b2fcc3fc343ac095 to PENDING with url https://10.20.13.93:8080/job/merlin_merlin/103/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/331/*:refs/remotes/origin/pr/331/* # timeout=10
 > git rev-parse 32beacb1b04538a2a7b50619b2fcc3fc343ac095^{commit} # timeout=10
Checking out Revision 32beacb1b04538a2a7b50619b2fcc3fc343ac095 (detached)
 > git config core.sparsecheckout # timeout=10
 > git checkout -f 32beacb1b04538a2a7b50619b2fcc3fc343ac095 # timeout=10
Commit message: "remove root slash"
 > git rev-list --no-walk 930361dac226ce387d85060af805123244ba3e95 # timeout=10
[merlin_merlin] $ /bin/bash /tmp/jenkins7593456078902872892.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.5.0, xdist-2.5.0, forked-1.4.0, cov-3.0.0
collected 1 item

tests/unit/test_version.py . [100%]

============================== 1 passed in 0.01s ===============================
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://github.com/gitapi/repos/NVIDIA-Merlin/Merlin/issues/$ghprbPullId/comments" "/var/jenkins_home/jobs/$JOB_NAME/builds/$BUILD_NUMBER/log"
[merlin_merlin] $ /bin/bash /tmp/jenkins2986927351180948074.sh

@benfred benfred merged commit e2e8e72 into NVIDIA-Merlin:main May 26, 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.

3 participants