-
Notifications
You must be signed in to change notification settings - Fork 122
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
Add run timestamp to data #415
Conversation
Click to view CI ResultsGitHub pull request #415 of commit 8164ebec5bf5089ac546ad3936f4ec1d825e0045, no merge conflicts. Running as SYSTEM Setting status of 8164ebec5bf5089ac546ad3936f4ec1d825e0045 to PENDING with url https://10.20.13.93:8080/job/merlin_merlin/200/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/415/*:refs/remotes/origin/pr/415/* # timeout=10 > git rev-parse 8164ebec5bf5089ac546ad3936f4ec1d825e0045^{commit} # timeout=10 Checking out Revision 8164ebec5bf5089ac546ad3936f4ec1d825e0045 (detached) > git config core.sparsecheckout # timeout=10 > git checkout -f 8164ebec5bf5089ac546ad3936f4ec1d825e0045 # timeout=10 Commit message: "Add run timestamp to data" > git rev-list --no-walk 511f01d799bc585e7317226429260dea96fde61f # timeout=10 [merlin_merlin] $ /bin/bash /tmp/jenkins10333641494730659327.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 2 items |
Documentation preview |
8164ebe
to
2f7845b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
min-similarity-lines=20 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good catch
The timestamp seems important if this is to work with nightly containers and harmless for the production containers.
Small linting changes too.
This was initiated as part of #388.