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

Use real mount points #77

Merged
merged 2 commits into from
Nov 12, 2018
Merged

Use real mount points #77

merged 2 commits into from
Nov 12, 2018

Conversation

tipabu
Copy link
Collaborator

@tipabu tipabu commented Nov 8, 2018

Previously, we would create one loopback device with some subdirs to represent "drives" and symlink them into place under /srv/node*

Now, use N loopback devices. Note that you may want to crank down LOOPBACK_GB as it applies to each sparse file and the default may allow you to fill your VM's root disk. (Not immediately -- the files are still sparse -- but PUT enough data and you'll fill up.)

Two side benefits (beyond getting closer to "real" system behavior):

  • we can now enable mount_check, and
  • we can have per-node-and-service rsync modules.

Previously, we would create one loopback device with some subdirs
to represent "drives" and symlink them into place under /srv/node*

Now, use N loopback devices. Note that you may want to crank down
LOOPBACK_GB as it applies to each sparse file and the default may
allow you to fill your VM's root disk.

Two side benefits (beyond getting closer to "real" system behavior):
we can now enable mount_check, and we can have per-node-and-service
rsync modules.

Addresses #43.
@tipabu
Copy link
Collaborator Author

tipabu commented Nov 8, 2018

Tested on Bionic -- haven't tried some of the other base images we might want to support.

@tipabu tipabu mentioned this pull request Nov 9, 2018
@tipabu
Copy link
Collaborator Author

tipabu commented Nov 9, 2018

Oh hey, there's also #21...

@clayg
Copy link
Collaborator

clayg commented Nov 12, 2018

I'd say this would close #43

I tested it and it works good for me on xenial & bionic :shipit:

@tipabu tipabu merged commit df2283a into master Nov 12, 2018
@tipabu tipabu deleted the real-disks branch November 12, 2018 20:33
@tipabu tipabu mentioned this pull request Nov 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants