Skip to content

feat(vmimport): Add KVM source#697

Draft
votdev wants to merge 2 commits intoharvester:mainfrom
votdev:issue_9948_kvm
Draft

feat(vmimport): Add KVM source#697
votdev wants to merge 2 commits intoharvester:mainfrom
votdev:issue_9948_kvm

Conversation

@votdev
Copy link
Member

@votdev votdev commented Jan 27, 2026

Summary

Add support to manage KVM Source resources from VM Importer via UI.

Related Issue

harvester/harvester#9948

Test screenshot or video

@votdev votdev self-assigned this Jan 27, 2026
@github-actions github-actions bot added the other label Jan 27, 2026
@votdev votdev mentioned this pull request Jan 27, 2026
Open
@votdev votdev force-pushed the issue_9948_kvm branch 2 times, most recently from 4fc834a to 4a1cafe Compare January 29, 2026 13:18
@a110605 a110605 added feat New feature PR and removed other labels Jan 30, 2026
@a110605
Copy link
Collaborator

a110605 commented Feb 2, 2026

I will this PR after #703 and #702 as they're v1.8.0 highlight features.

@votdev votdev mentioned this pull request Feb 2, 2026
Merged
1 task
@a110605
Copy link
Collaborator

a110605 commented Feb 6, 2026

When clicking Source KVM, there is error pop up. We may need list page like pkg/harvester/list/harvesterhci.io.migration.vmwaresource.vue.

Screenshot 2026-02-06 at 5 21 47 PM

@votdev
Copy link
Member Author

votdev commented Feb 23, 2026

When clicking Source KVM, there is error pop up. We may need list page like pkg/harvester/list/harvesterhci.io.migration.vmwaresource.vue.
Screenshot 2026-02-06 at 5 21 47 PM

I think then this file is also missing for the OVA source. I'll add it with this PR as well.

@votdev votdev mentioned this pull request Feb 24, 2026
Merged
@a110605
Copy link
Collaborator

a110605 commented Feb 25, 2026 •
edited
Loading

When clicking Source KVM, there is error pop up. We may need list page like pkg/harvester/list/harvesterhci.io.migration.vmwaresource.vue.

Screenshot 2026-02-06 at 5 21 47 PM

@votdev , all the list pages should rename to migration.harvesterhci.io.XXXX.vue to match the URL path. Otherwise the error still happen.

Screenshot 2026-02-25 at 5 24 19 PM

@votdev
Copy link
Member Author

votdev commented Feb 25, 2026

When clicking Source KVM, there is error pop up. We may need list page like pkg/harvester/list/harvesterhci.io.migration.vmwaresource.vue.
Screenshot 2026-02-06 at 5 21 47 PM

@votdev , all the list pages should rename to migration.harvesterhci.io.XXXX.vue to match the URL path. Otherwise the error still happen.
Screenshot 2026-02-25 at 5 24 19 PM

I've fixed that for the KVM page in this PR. I'll open a separate PR for the other pages because this PR (and related feature) is scheduled for 1.9.

votdev added a commit to votdev/harvester-ui-extension that referenced this pull request Feb 25, 2026
@votdev
fix(vmimport): Rename list pages according to their URL path
b231fd2 
See harvester#697 (comment)

Related to: harvester/harvester#4663

Signed-off-by: Volker Theile <vtheile@suse.com>
@votdev votdev mentioned this pull request Feb 25, 2026
Merged
a110605 pushed a commit that referenced this pull request Feb 26, 2026
@votdev
fix(vmimport): Rename list pages according to their URL path (#717)
337f918 
See #697 (comment)

Related to: harvester/harvester#4663

Signed-off-by: Volker Theile <vtheile@suse.com>
@votdev @a110605
feat(vmimport): Add KVM source
434b338 
Related to: harvester/harvester#9948

Signed-off-by: Volker Theile <vtheile@suse.com>
@a110605
fix: add labelFor page header in en-us.yaml
c5e22dc 
Signed-off-by: Andy Lee <andy.lee@suse.com>
@votdev votdev marked this pull request as draft February 26, 2026 10:04
@votdev
Copy link
Member Author

votdev commented Feb 26, 2026

The KVM feature will be scheduled to Harvester 1.9, so this UI PR is on hold now.

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

Reviewers

@a110605 a110605 Awaiting requested review from a110605

@houhoucoop houhoucoop Awaiting requested review from houhoucoop

At least 1 approving review is required to merge this pull request.

Assignees

@votdev votdev

Labels

feat New feature PR

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@votdev @a110605