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

Docker fix #563

Merged
merged 7 commits into from
Aug 26, 2022
Merged

Docker fix #563

merged 7 commits into from
Aug 26, 2022

Conversation

jperez999
Copy link
Collaborator

The adds pyarrow build because feast requires pyarrow s3 support to run.

@jperez999 jperez999 self-assigned this Aug 26, 2022
@jperez999 jperez999 added the ci label Aug 26, 2022
@github-actions
Copy link

Documentation preview

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

@nvidia-merlin-bot
Copy link
Contributor

Click to view CI Results
GitHub pull request #563 of commit c3317409a99e4fc53b0ba471cb39da74582d8138, no merge conflicts.
Running as SYSTEM
Setting status of c3317409a99e4fc53b0ba471cb39da74582d8138 to PENDING with url https://10.20.13.93:8080/job/merlin_merlin/376/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/563/*:refs/remotes/origin/pr/563/* # timeout=10
 > git rev-parse c3317409a99e4fc53b0ba471cb39da74582d8138^{commit} # timeout=10
Checking out Revision c3317409a99e4fc53b0ba471cb39da74582d8138 (detached)
 > git config core.sparsecheckout # timeout=10
 > git checkout -f c3317409a99e4fc53b0ba471cb39da74582d8138 # timeout=10
Commit message: "build pyarrow for s3 support for feast - required"
 > git rev-list --no-walk f390f3aca3d0344033d7e7c0a8ff8cec991c8b68 # timeout=10
[merlin_merlin] $ /bin/bash /tmp/jenkins12076903533112380029.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 s [ 66%]
tests/unit/examples/test_scaling_criteo_merlin_models.py . [100%]

=============================== warnings summary ===============================
../../../../../usr/local/lib/python3.8/dist-packages/dask_cudf/core.py:33
/usr/local/lib/python3.8/dist-packages/dask_cudf/core.py:33: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
DASK_VERSION = LooseVersion(dask.version)

../../../.local/lib/python3.8/site-packages/setuptools/_distutils/version.py:346: 34 warnings
/var/jenkins_home/.local/lib/python3.8/site-packages/setuptools/_distutils/version.py:346: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
other = LooseVersion(other)

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
============ 2 passed, 1 skipped, 35 warnings in 113.77s (0:01:53) =============
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/jenkins1283253688214877272.sh

@jperez999 jperez999 requested a review from karlhigley August 26, 2022 17:32
@jperez999 jperez999 merged commit b3f58bb into NVIDIA-Merlin:main Aug 26, 2022
@viswa-nvidia viswa-nvidia added this to the Merlin 22.09 milestone Sep 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants