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

Revert "Change disk bus from scsi to virtio to gain performance" #10735

Merged
merged 1 commit into from
Jun 17, 2022

Conversation

vulkoingim
Copy link
Contributor

This reverts commit 9552b60.

Description

Related Issue(s)

Fixes #

How to test

Release Notes

NONE

Documentation

@vulkoingim
Copy link
Contributor Author

I think this change caused preview envs to fail (well, the vm to fail to start)

@vulkoingim
Copy link
Contributor Author

vulkoingim commented Jun 17, 2022

/werft run github -j .werft/build.yaml -a updateGitHubStatus=gitpod-io/gitpod -a no-preview=true

👍 started the job as gitpod-build-revert-10721-jenting-bus-from-scsi-to-virtio.1
(with .werft/ from main)

@jenting
Copy link
Contributor

jenting commented Jun 17, 2022

I think this change caused preview envs to fail (well, the vm to fail to start)

🤔 I thought the change worked, but it seems to be no.

@roboquat roboquat merged commit d6cc17a into main Jun 17, 2022
@roboquat roboquat deleted the revert-10721-jenting/bus-from-scsi-to-virtio branch June 17, 2022 15:24
@mads-hartmann
Copy link
Contributor

mads-hartmann commented Jun 17, 2022

I think what might have happened is that job protection tricked us again ☺️ the werft job we tested on in your PR would have run from main unless you manually triggered the job using the CLI. Could that have been what happened?

@jenting
Copy link
Contributor

jenting commented Jun 17, 2022

Absolutely yes 🤦🏻🤦🏻🤦🏻

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.

None yet

4 participants