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.

Sign up for GitHub

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

Jump to bottom

remote: fix tests #86

Closed
piotrrojek opened this issue Oct 14, 2016 · 2 comments
Closed

remote: fix tests #86

piotrrojek opened this issue Oct 14, 2016 · 2 comments
Assignees
@piotrrojek
Labels
wontfix
Milestone
Open source release

Comments

@piotrrojek
Copy link
Contributor

piotrrojek commented Oct 14, 2016 •
edited by matryer
Loading

Remote implementation currently has no tests.
@piotrrojek piotrrojek mentioned this issue Oct 14, 2016
Merged
@piotrrojek piotrrojek self-assigned this Oct 14, 2016
@matryer matryer changed the title remote: create Mounter interface + create tests for remote remote: fix tests Oct 14, 2016
@piotrrojek
Copy link
Contributor Author

Problems and questions on those tests:

  • we need to have fileserver with (e.g.) NFS shares up on build with jenkins; as I know @jasonhancock is looking into making docker image with nfs
  • how do we handle mounting? do we mount once and run all tests? or do we mount on every tests to be able to run every tests separately?
  • what about unmounting? remote doesn't handle that
  • seperate, but related: mounting from OSX (in my case to docker nfs server image) doesn't work (it hangs)

thoughts? @matryer @ernesto-jimenez

@matryer matryer added this to the Open source release milestone Oct 17, 2016
@matryer
Copy link
Contributor

matryer commented Oct 17, 2016

Closing due to #91

@matryer matryer closed this as completed Oct 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@piotrrojek piotrrojek

Labels
wontfix
Projects
None yet
Milestone
Open source release
Development

No branches or pull requests
2 participants
@matryer @piotrrojek