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 mounts for --volume=HOME #977

Merged
merged 2 commits into from
Aug 26, 2019
Merged

Fix mounts for --volume=HOME #977

merged 2 commits into from
Aug 26, 2019

Conversation

atemate
Copy link
Contributor

@atemate atemate commented Aug 22, 2019

Closes #975

@atemate atemate requested a review from asvetlov August 22, 2019 12:54
@@ -1284,8 +1284,8 @@ neuro run [OPTIONS] IMAGE [CMD]...

# Starts a container pytorch:latest on a machine with smaller GPU resources
# (see exact values in `neuro config show`) and with two volumes mounted:
# storage://~ --> /var/storage/home (in read-write mode),
# storage://neuromation --> /var/storage/neuromation (in read-only mode).
# storage://<home-directory> --> /var/storage/home (in read-write mode),
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

side change aiming improvement of the readability

@asvetlov
Copy link
Contributor

LGTM when green.
P.S.
e2e port forward test fails in approximately 30-50% of all CI jobs :(

@codecov
Copy link

codecov bot commented Aug 23, 2019

Codecov Report

Merging #977 into master will increase coverage by 4.6%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #977     +/-   ##
=========================================
+ Coverage   87.69%   92.29%   +4.6%     
=========================================
  Files          37       37             
  Lines        4259     4259             
  Branches      640      640             
=========================================
+ Hits         3735     3931    +196     
+ Misses        409      221    -188     
+ Partials      115      107      -8
Impacted Files Coverage Δ
neuromation/cli/job.py 89.76% <ø> (+6.48%) ⬆️
neuromation/api/jobs.py 96.38% <0%> (+1.2%) ⬆️
neuromation/api/config_factory.py 90.38% <0%> (+1.28%) ⬆️
neuromation/cli/formatters/jobs.py 97.94% <0%> (+2.46%) ⬆️
neuromation/api/url_utils.py 92.42% <0%> (+4.54%) ⬆️
neuromation/cli/utils.py 89.29% <0%> (+5.35%) ⬆️
neuromation/cli/main.py 69.8% <0%> (+5.44%) ⬆️
neuromation/cli/formatters/config.py 100% <0%> (+11.11%) ⬆️
neuromation/api/images.py 94.21% <0%> (+23.96%) ⬆️
... and 2 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 9b5e44f...5d0ac60. Read the comment docs.

1 similar comment
@codecov
Copy link

codecov bot commented Aug 23, 2019

Codecov Report

Merging #977 into master will increase coverage by 4.6%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #977     +/-   ##
=========================================
+ Coverage   87.69%   92.29%   +4.6%     
=========================================
  Files          37       37             
  Lines        4259     4259             
  Branches      640      640             
=========================================
+ Hits         3735     3931    +196     
+ Misses        409      221    -188     
+ Partials      115      107      -8
Impacted Files Coverage Δ
neuromation/cli/job.py 89.76% <ø> (+6.48%) ⬆️
neuromation/api/jobs.py 96.38% <0%> (+1.2%) ⬆️
neuromation/api/config_factory.py 90.38% <0%> (+1.28%) ⬆️
neuromation/cli/formatters/jobs.py 97.94% <0%> (+2.46%) ⬆️
neuromation/api/url_utils.py 92.42% <0%> (+4.54%) ⬆️
neuromation/cli/utils.py 89.29% <0%> (+5.35%) ⬆️
neuromation/cli/main.py 69.8% <0%> (+5.44%) ⬆️
neuromation/cli/formatters/config.py 100% <0%> (+11.11%) ⬆️
neuromation/api/images.py 94.21% <0%> (+23.96%) ⬆️
... and 2 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 9b5e44f...5d0ac60. Read the comment docs.

@atemate atemate merged commit f6bca6d into master Aug 26, 2019
@atemate atemate deleted the ay/fix-volume-home branch August 26, 2019 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Resource storage://neuromation is not public anymore?
3 participants