/minimega

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

minimega/ron/miniccc: add ufs to miniccc #1108

Merged
merged 14 commits into from Jun 21, 2018

Conversation

Reviewers

@djfritz djfritz

Assignees

@djfritz djfritz

Labels
None yet
Projects
None yet
Milestone
  2.5
2 participants
@jcrussell @djfritz
@jcrussell
Contributor

jcrussell commented Jun 4, 2018 •
edited

Exposes the guest filesystem to the host over the cc connection.

jcrussell added some commits May 31, 2018

@jcrussell
vendor: add ninep 
For `cc mount`.
9672893
@jcrussell
minimega/ron/miniccc: initial work on ufs 
Initial work to expose the guest's filesystem via the cc connection and
9p. Requires tunneling yet another thing over the cc connection.

Tested and hung on the mount command... need to investigate.
88b7a97
@jcrussell
vendor: bump ninep 
Windows support.
0a71d0f
@jcrussell
minimega/ron/miniccc: somewhat working 
Seems to work, need to do some refactoring on the ninep side and then
refactor this too.
653dbe7
@jcrussell
various: continued refactoring for ufs 
Bump the vendored ninep code to post-refactored version. Update
minimega/ron/miniccc accordingly.
21cc3cd
@jcrussell
minimega: improved `cc mount` 
Mount a guest filesystem across the network to the node that runs the
command via `cc mount <uuid> ...`. Need to fix `cc mount` listing and
probably move the ccMounts into the namespace struct so that they are
namespace-aware.
3d7c211
@jcrussell
minimega/ron: cc mount improvements 
Move ccMounts into the namespace struct so that cc mount is
namespace-aware. Add `clear cc mount` to unmount things. Adding locking
to prevent potential races.
1ded362
@jcrussell
Merge remote-tracking branch 'origin/master' into issue-1107
25b202a
@jcrussell
minimega: fixing clear cc 
Make clear cc clean up cc mounts. Change mount to read-only. Abort if
the destination for the mount does not exist.
3ebbeda
@jcrussell
minimega/doc: update docs for `cc mount` 
Not really integrated into tutorial...
ccddc83

@jcrussell jcrussell added this to the 2.5 milestone Jun 12, 2018

@jcrussell jcrussell self-assigned this Jun 12, 2018

jcrussell added some commits Jun 14, 2018

@jcrussell
ninep,miniccc: bump version 
Merge in the latest ninep code, update miniccc accordingly.
f901a3a
@jcrussell
minimega: minor `cc mount` tweaks 
Drop the read-only flag since we can now write (and it didn't seem to
work). Clean up mounts when we destroy the namespace.

Probably need to also clean up the mounts when we kill a VM...
e3bdc33
@jcrussell

This comment has been minimized.

Sign in to view
Contributor

jcrussell commented Jun 14, 2018

TODO: unmount if the VM is killed?

jcrussell added some commits Jun 19, 2018

@jcrussell
ninep: bump version 
To fix Windows build.
1497ef8
@jcrussell
minimega: add note about cc mount 
Make it clear that mounts should be unmounted before the VM is killed.
14fb518

@jcrussell jcrussell assigned djfritz and unassigned jcrussell Jun 21, 2018

@djfritz

djfritz approved these changes Jun 21, 2018
View changes

LGTM - what could go wrong?

@djfritz djfritz merged commit 85cf7f7 into sandia-minimega:master Jun 21, 2018
1 check passed

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@jcrussell jcrussell deleted the jcrussell:issue-1107 branch Jun 21, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment