Skip to content

Update readme to document rawdisk #173

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

Merged
merged 1 commit into from
Apr 20, 2017

Conversation

dalehamel
Copy link
Collaborator

@zchee readme update as requested for #169

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@zchee zchee merged commit 0b0f220 into machine-drivers:master Apr 20, 2017
@zchee
Copy link
Member

zchee commented Apr 20, 2017

@dalehamel Thanks! merged.

@alafon
Copy link

alafon commented Jun 12, 2017

@zchee I won't open a new issue to ask a possible simple question but could you explain a bit more what using this feature implies ? It is said that there is a potential cost of data durability, what does that mean ? Can mounted volumes be affected/corrupted ? Or is it just about the containers inner data ?

Thanks in advance

@dalehamel
Copy link
Collaborator Author

@alafon since I added the support for this, I'll answer.

Basically, if you are running something where data integrity really matters (MySQL perhaps), then an unclean shutdown of the VM could result in some bytes being unwritten when using this driver.

Most likely though, since you're probably using this for development, this probably doesn't matter.

This is because the virtio-blk is much more "raw" than the ahci approach, which attempts to more closely mimic how an actual SCSI device would operate.

We use virtio-blk because it is faster and lower overhead, and we make sure everything is disposable anyways (this is kubernetes after all), so we don't really care if there are some edge cases where data could be corrupted.

@alafon
Copy link

alafon commented Jun 13, 2017

Thanks for the answer, pretty clear :)

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

3 participants