Skip to content

Change disk bus from scsi to virtio to gain performance #10721

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
Jun 17, 2022

Conversation

jenting
Copy link
Contributor

@jenting jenting commented Jun 17, 2022

Description

Currently, the Harvester VM uses scsi, we change to virtio to gain the performance.
Check the VM image supports the virtio by

$ lsmod | grep virtio_blk

Related slack discussion https://gitpod.slack.com/archives/C01KGM9EBD4/p1655444499062949

Related Issue(s)

Fixes #

How to test

Check the VM is bootable.

Release Notes

None

Documentation

None

Verify the kernel module is loaded with the VM image
$ lsmod | grep virtio_blk

Signed-off-by: JenTing Hsiao <[email protected]>
@werft-gitpod-dev-com
Copy link

started the job as gitpod-build-jenting-bus-from-scsi-to-virtio.1 because the annotations in the pull request description changed
(with .werft/ from main)

@werft-gitpod-dev-com
Copy link

started the job as gitpod-build-jenting-bus-from-scsi-to-virtio.2 because the annotations in the pull request description changed
(with .werft/ from main)

@jenting jenting requested a review from a team June 17, 2022 10:22
Copy link
Contributor

@mads-hartmann mads-hartmann left a comment

Choose a reason for hiding this comment

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

🛹 Preview env seems to work just fine 🎉

@roboquat roboquat merged commit 9552b60 into main Jun 17, 2022
@roboquat roboquat deleted the jenting/bus-from-scsi-to-virtio branch June 17, 2022 11:47
@vulkoingim vulkoingim restored the jenting/bus-from-scsi-to-virtio branch June 17, 2022 14:40
@vulkoingim vulkoingim deleted the jenting/bus-from-scsi-to-virtio branch June 17, 2022 14:40
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.

3 participants