Skip to content
This repository has been archived by the owner on Apr 28, 2020. It is now read-only.

Backport 557, BZ 1756328: Use volumeDevices for Block volumeMode #565

Conversation

mareklibra
Copy link
Contributor

To make flows in the conversion pod easier, the temporary disk is requested with
the Filesystem volumeMode no matter of the kubevirt-storage-class-defaults values.

The temporary space for the conversion pod is requested on the default storage class.
The "Filesystem" volumeMode seems to be safe, supported by all storage classes.

The target disks for the v2v stay untouched, this change affects just the temporary space.

Signed-off-by: Marek Libra <[email protected]>

Conflicts:
  - src/k8s/requests/v2v/importVmware.js
@mareklibra mareklibra force-pushed the backport.557.v2v.volumeDevices.Block branch from df2860d to 5407d37 Compare October 18, 2019 08:04
@coveralls
Copy link

coveralls commented Oct 18, 2019

Pull Request Test Coverage Report for Build 1987

  • 6 of 17 (35.29%) changed or added relevant lines in 4 files are covered.
  • 3 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-0.2%) to 82.556%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/components/Wizard/CreateVmWizard/providers/VMwareImportProvider/VMWareImportProvider.js 0 1 0.0%
src/k8s/requests/v2v/startV2VvmwareController.js 0 1 0.0%
src/k8s/requests/v2v/importVmware.js 5 14 35.71%
Files with Coverage Reduction New Missed Lines %
src/components/Wizard/CreateVmWizard/stateUpdate/providers/prefillVmStateUpdate.js 1 8.57%
src/k8s/requests/v2v/importVmware.js 2 10.91%
Totals Coverage Status
Change from base Build 2399: -0.2%
Covered Lines: 4206
Relevant Lines: 4871

💛 - Coveralls

@mareklibra mareklibra changed the title Backport.557.v2v.volume devices.block Backport 557, BZ 1756328: Use volumeDevices for Block volumeMode Oct 18, 2019
@jelkosz jelkosz merged commit a4cb831 into kubevirt:release-v0.1.44 Oct 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants