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

[release-v0.14.0] bump kubevirt to v0.34.2 #254

Conversation

RamLavi
Copy link
Member

@RamLavi RamLavi commented Nov 26, 2020

This PR bumps kubevirt to v0.34.2 on the stable branch, to support disk.io configuration.

What this PR does / why we need it:

Special notes for your reviewer:

Release note:

NONE

The kubevirt currently used doesn't support disk.io support,
which causes it to be removed from the vm instance when
reconciled by kubemacpool.
Bumped to v0.34.2 to support disk.io in the vm object

Signed-off-by: Ram Lavi <ralavi@redhat.com>
Signed-off-by: Ram Lavi <ralavi@redhat.com>
Signed-off-by: Ram Lavi <ralavi@redhat.com>
@kubevirt-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign ramlavi
You can assign the PR to them by writing /assign @ramlavi in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@@ -25,7 +25,7 @@ source ./cluster/kubevirtci.sh
CNAO_VERSIOV=0.35.0
#use kubevirt latest z stream release

KUBEVIRT_VERSION=$(getLatestPatchVersion v0.30)
KUBEVIRT_VERSION=$(getLatestPatchVersion v0.34)
Copy link
Member

Choose a reason for hiding this comment

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

The bug https://bugzilla.redhat.com/show_bug.cgi?id=1901480 is for 2.6 which is yet to be branched out. This fix should be on the master.

In case we decide to fix the bug in a 2.5 z-stream, we would then needs a change similar to this (but I believe we should pin to v0.30.7)

Copy link
Member Author

Choose a reason for hiding this comment

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

closing the PR

@RamLavi RamLavi closed this Nov 29, 2020
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

3 participants