liquidmetal-dev / liquidmetal-dev/flintlock

cloud-hypervisor: adopt /vm.add-generic-vhost-user endpoint

Open Beginner friendly
#1,184 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Go
Stars
1.5k
Forks
73
Avg merge
17h 24m
Merged PRs (30d)
30

Description

Cloud Hypervisor v53.0 added a /vm.add-generic-vhost-user REST endpoint for attaching a generic vhost-user device to a running VM. It's not currently supported by flintlock's cloud-hypervisor client.

Work

  • Add PathAddGenericVhostUser to pkg/cloudhypervisor/client.go
  • Add a GenericVhostUserConfig struct to pkg/cloudhypervisor/types.go
  • Add an AddGenericVhostUser method to the Client interface and implementation

Filed as part of the v53.0 compatibility review. Deliberately scoped out of the minimal v53.0 version-bump PR.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with pkg/cloudhypervisor/client.go and pkg/cloudhypervisor/types.go, then review the existing Cloud Hypervisor client methods for the v53.0 endpoint patterns. Done means the path constant, GenericVhostUserConfig type, and AddGenericVhostUser method are available through both the Client interface and its implementation.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
api
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Active
Clarity
Clearly specified
Newbie friendliness
86/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.