Skip to content
This repository has been archived by the owner on Mar 22, 2023. It is now read-only.

common: s/NVML/PMDK/ #443

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

common: s/NVML/PMDK/ #443

wants to merge 1 commit into from

Conversation

GBuella
Copy link
Contributor

@GBuella GBuella commented May 21, 2018

This change is Reviewable

@codecov
Copy link

codecov bot commented May 22, 2018

Codecov Report

Merging #443 into master will decrease coverage by 29.51%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master     #443       +/-   ##
===========================================
- Coverage   83.91%   54.39%   -29.52%     
===========================================
  Files          71       56       -15     
  Lines        8815     7017     -1798     
  Branches      269        0      -269     
===========================================
- Hits         7397     3817     -3580     
- Misses       1414     3200     +1786     
+ Partials        4        0        -4
Flag Coverage Δ
#ltp_tests 54.39% <100%> (+0.69%) ⬆️
#sqlite_tests ?
#tests_antool ?
#tests_posix_multi_threaded ?
#tests_posix_single_threaded ?
#tests_preload ?
Impacted Files Coverage Δ
src/libpmemfile-posix/out.c 43.67% <100%> (-9.5%) ⬇️
src/libpmemfile-posix/mknod.c 0% <0%> (-100%) ⬇️
src/libpmemfile-posix/copy_file_range.c 0% <0%> (-100%) ⬇️
src/libpmemfile-posix/access.c 0% <0%> (-100%) ⬇️
src/libpmemfile-posix/flock.c 0% <0%> (-100%) ⬇️
src/libpmemfile-posix/mmap.c 0% <0%> (-100%) ⬇️
src/libpmemfile-posix/link.c 0% <0%> (-98.97%) ⬇️
src/libpmemfile-posix/timestamps.c 0% <0%> (-96.86%) ⬇️
src/libpmemfile-posix/fallocate.c 0% <0%> (-96.39%) ⬇️
src/libpmemfile-posix/rename.c 0% <0%> (-95.15%) ⬇️
... and 51 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a1b9897...e781bb5. Read the comment docs.

@GBuella
Copy link
Contributor Author

GBuella commented May 22, 2018

Although this docker script still installs NVML, from before the rename has happened:
utils/docker/images/install-pmdk.sh

@GBuella
Copy link
Contributor Author

GBuella commented May 22, 2018

Oh well.

Rebuilding the Docker image for the Dockerfile.ubuntu-17.04

Reading package lists...
W: The repository 'http://security.ubuntu.com/ubuntu zesty-security Release' does not have a Release file.
W: The repository 'http://archive.ubuntu.com/ubuntu zesty Release' does not have a Release file.
W: The repository 'http://archive.ubuntu.com/ubuntu zesty-updates Release' does not have a Release file.
W: The repository 'http://archive.ubuntu.com/ubuntu zesty-backports Release' does not have a Release file.
E: Failed to fetch http://security.ubuntu.com/ubuntu/dists/zesty-security/universe/source/Sources  404  Not Found [IP: 91.189.88.152 80]
E: Failed to fetch http://archive.ubuntu.com/ubuntu/dists/zesty/universe/source/Sources  404  Not Found [IP: 91.189.88.149 80]
E: Failed to fetch http://archive.ubuntu.com/ubuntu/dists/zesty-updates/universe/source/Sources  404  Not Found [IP: 91.189.88.149 80]
E: Failed to fetch http://archive.ubuntu.com/ubuntu/dists/zesty-backports/multiverse/binary-amd64/Packages  404  Not Found [IP: 91.189.88.149 80]
E: Some index files failed to download. They have been ignored, or old ones used instead.
The command '/bin/sh -c apt-get update' returned a non-zero code: 100

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant