Skip to content

[Bug]: VM spec uses deprecated Running: true field instead of RunStrategy #9

Description

@mrhillsman

Bug Description

`internal/vm/vm.go` (line 112) sets the deprecated `Running: &running` boolean pointer on the VirtualMachine spec. KubeVirt 1.0+ deprecated this field in favor of `RunStrategy` (e.g., `RunStrategyAlways`, `RunStrategyManual`). While the deprecated field still works, it may be removed in a future KubeVirt release, and the project already depends on KubeVirt v1.7.0.

Steps to Reproduce

  1. Inspect `internal/vm/vm.go:112`
  2. Observe that it sets `Running: &running` instead of `RunStrategy`
  3. This field is deprecated in KubeVirt 1.0+ and may be removed in a future release

Expected Behavior

Replace with `RunStrategy: kubevirtv1.RunStrategyAlways`.

Actual Behavior

Uses deprecated `Running: &running` field which may break in future KubeVirt versions.

Command

virtwork run

Go Version

Not provided

Deployment Mode

Not provided

Additional Context

Severity: Low

Files:

  • `internal/vm/vm.go:112` — `Running: &running`

Metadata

Metadata

Assignees

No one assigned

    Labels

    good first issueDenotes an issue ready for a new contributor, according to the "help wanted" guidelines.kind/bugCategorizes issue or PR as related to a bug.size/XSDenotes a PR that changes 0-9 lines, ignoring generated files.

    Type

    No type

    Fields

    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions