Don't use virtio for dogwood or eucalyptus boxes #3590

Merged
merged 1 commit into from Jan 5, 2017

Projects

None yet

4 participants

@nedbat
Member
nedbat commented Jan 5, 2017 edited

The "virtio" nictype is faster, but needs support built into the box file. Before Ficus, we didn't have support, so don't use it for older boxes.

@jlajoie @staubina @gsong


Make sure that the following steps are done before merging

  • @devops team member has commented with 👍
  • are you adding any new default values that need to be overridden when this goes live?
    • Open a ticket (DEVOPS) to make sure that they have been added to secure vars.
    • Add an entry to the CHANGELOG.
@nedbat nedbat Don't use virtio for dogwood or eucalyptus boxes
1bcec28
@gsong
gsong approved these changes Jan 5, 2017 View changes
@nedbat nedbat merged commit 911ce6a into master Jan 5, 2017

2 checks passed

VersionEye All software dependencies are fine. You are awesome!
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@nedbat nedbat deleted the ned/conditional-virtio branch Jan 5, 2017
@nedbat
Member
nedbat commented Jan 5, 2017

This is now cherry-picked onto ned/test-ficus.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment