Skip to content

eve-k: update to kubevirt 1.7.3#5865

Merged
rene merged 3 commits intolf-edge:masterfrom
andrewd-zededa:eve-k-kubevirt-1.7.3
Apr 27, 2026
Merged

eve-k: update to kubevirt 1.7.3#5865
rene merged 3 commits intolf-edge:masterfrom
andrewd-zededa:eve-k-kubevirt-1.7.3

Conversation

@andrewd-zededa
Copy link
Copy Markdown
Contributor

@andrewd-zededa andrewd-zededa commented Apr 24, 2026

Description

Incorporate bug fixes in the 1.7.3 release regarding VMI reporting stuck in Pending due to an issue in the virt-handler pod notification server.

KUBE_VERSION moved to 3 to trigger updates. Downgrade blocking gate added in cluster-update.sh: just exit if applied-kube-version is greater than current script version KUBE_VERSION.

Large PR due to all the go mod dependency updates needed to update the kubevirt go mod to 1.7.3 as well

Go Mod Changes:

kubevirt.io/api and kubevirt.io/client-go bumped from v1.6.0 -> v1.7.3
in pkg/pillar, and from v1.2.0 -> v1.7.3 in pkg/kube/update-component.

kubevirt v1.7.3 requires k8s.io/client-go v0.33.5 which introduced the
k8s.io/client-go/kubernetes/typed/resource/v1beta2 package for the DRA
(Dynamic Resource Allocation) v1beta2 API. That package does not exist
in v0.32.5, so go mod tidy failed until the k8s.io/* pins were raised.

pkg/pillar changes:

  • kubevirt.io/api: v1.6.0 -> v1.7.3
  • kubevirt.io/client-go: v1.6.0 -> v1.7.3
  • All k8s.io/* replace directives: v0.32.5 -> v0.33.5
  • kubevirt.io/containerized-data-importer-api updated to v1.63.1
  • go.sum and vendor/ regenerated

pkg/kube/update-component changes:

  • kubevirt.io/client-go: v1.2.0 -> v1.7.3 (larger jump)
  • All k8s.io/* replace directives: v0.26.3 -> v0.33.5
  • k8s.io/kube-openapi pseudo-version updated to align with kubevirt pin
  • kubevirt.io/containerized-data-importer-api updated to v1.63.1
  • go.sum and vendor/ regenerated

API breakage fix in pkg/kube/update-component/kubevirt.go:

  • KubeVirt().Get() signature changed in v1.7.3: context.Context is now
    required as the first argument, and GetOptions is passed by value
    instead of by pointer. Updated the single call site accordingly.

PR dependencies

None

How to test and validate this PR

  • upgrade eve base os of an eve-k node, it will incorporate this update after booting into the next image

Changelog notes

Update kubevirt to 1.7.3 to incorporate VMI ready fix

PR Backports

  • 16.0-stable: ? might be a large backport
  • 14.5-stable: No, as the feature is not available there.
  • 13.4-stable: No, as the feature is not available there.

Checklist

  • I've provided a proper description
  • I've added the proper documentation
  • I've tested my PR on amd64 device
  • I've tested my PR on arm64 device
  • I've written the test verification instructions
  • I've set the proper labels to this PR

And the last but not least:

  • I've checked the boxes above, or I've provided a good reason why I didn't
    check them.

Please, check the boxes above after submitting the PR in interactive mode.

andrewd-zededa and others added 2 commits April 24, 2026 11:45
Update stale expected_versions for update path up to the
expected running versions defined in kube currently.

Signed-off-by: Andrew Durbin <andrewd@zededa.com>
kubevirt.io/api and kubevirt.io/client-go bumped from v1.6.0 -> v1.7.3
in pkg/pillar, and from v1.2.0 -> v1.7.3 in pkg/kube/update-component.

kubevirt v1.7.3 requires k8s.io/client-go v0.33.5 which introduced the
k8s.io/client-go/kubernetes/typed/resource/v1beta2 package for the DRA
(Dynamic Resource Allocation) v1beta2 API. That package does not exist
in v0.32.5, so go mod tidy failed until the k8s.io/* pins were raised.

pkg/pillar changes:
- kubevirt.io/api: v1.6.0 -> v1.7.3
- kubevirt.io/client-go: v1.6.0 -> v1.7.3
- All k8s.io/* replace directives: v0.32.5 -> v0.33.5
- kubevirt.io/containerized-data-importer-api updated to v1.63.1
- go.sum and vendor/ regenerated

pkg/kube/update-component changes:
- kubevirt.io/client-go: v1.2.0 -> v1.7.3 (larger jump)
- All k8s.io/* replace directives: v0.26.3 -> v0.33.5
- k8s.io/kube-openapi pseudo-version updated to align with kubevirt pin
- kubevirt.io/containerized-data-importer-api updated to v1.63.1
- go.sum and vendor/ regenerated

API breakage fix in pkg/kube/update-component/kubevirt.go:
- KubeVirt().Get() signature changed in v1.7.3: context.Context is now
  required as the first argument, and GetOptions is passed by value
  instead of by pointer. Updated the single call site accordingly.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: Andrew Durbin <andrewd@zededa.com>
@zedi-pramodh
Copy link
Copy Markdown

Nothing to review here all clean and good :) BTW, did you test cluster upgrade with this PR ?

@andrewd-zededa
Copy link
Copy Markdown
Contributor Author

Nothing to review here all clean and good :) BTW, did you test cluster upgrade with this PR ?

I did upgrade my dev cluster from master with 1.6.0 to this build including 1.7.3. The kubevirt component is updated after the first node in the cluster boots into the new eve baseos version.

Watch kubevirt component update process in k3s-install.log:

$ tail -f /persist/kubelog/k3s-install.log
...
2026-04-24 21:12:35 : update_HandleCluster: version:3 appliedversion:2 continuing
...
2026-04-24 21:12:35 : Component: kubevirt ready on existing version
2026-04-24 21:12:35 : publishUpdateStatus() <nodename> kubevirt in_progress
2026-04-24 21:14:38 : publishUpdateStatus() <nodename> kubevirt completed

and confirm kubevirt new version like this in 'eve enter kube':

[kube] root@63ed13ac-9d6c-457d-b558-6b6314be1b69:/$ kubectl -n kubevirt get kubevirt/kubevirt -o json | jq .status.operatorVersion
"v1.7.3"

@zedi-pramodh
Copy link
Copy Markdown

LGTM. @rene can we get this merged by Monday so that we can provide build to a customer, thanks

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 17.03%. Comparing base (2281599) to head (d5c052b).
⚠️ Report is 589 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5865      +/-   ##
==========================================
- Coverage   19.52%   17.03%   -2.50%     
==========================================
  Files          19      477     +458     
  Lines        3021    85762   +82741     
==========================================
+ Hits          590    14611   +14021     
- Misses       2310    69641   +67331     
- Partials      121     1510    +1389     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@eriknordmark eriknordmark requested a review from rene April 25, 2026 13:33
@eriknordmark
Copy link
Copy Markdown
Contributor

LGTM. @rene can we get this merged by Monday so that we can provide build to a customer, thanks

I can't be merged while it is marked as a draft @naiming-zededa

@naiming-zededa
Copy link
Copy Markdown
Contributor

LGTM. @rene can we get this merged by Monday so that we can provide build to a customer, thanks

I can't be merged while it is marked as a draft @naiming-zededa

@andrewd-zededa can you remove the draft, is this done?

@andrewd-zededa
Copy link
Copy Markdown
Contributor Author

One small line addition to make, will push in about an hour

Exit if KUBE_VERSION below current applied version.
Don't try downgrades, unsupported.

Signed-off-by: Andrew Durbin <andrewd@zededa.com>
@andrewd-zededa andrewd-zededa marked this pull request as ready for review April 27, 2026 15:20
@github-actions github-actions Bot requested a review from zedi-pramodh April 27, 2026 15:39
Copy link
Copy Markdown
Contributor

@rene rene left a comment

Choose a reason for hiding this comment

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

Kicking off tests....

@rene rene merged commit a67d55c into lf-edge:master Apr 27, 2026
29 of 32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants