Skip to content

feat(vm): add uptime printable column#2279

Merged
danilrwx merged 11 commits into
mainfrom
feat/vm/add-uptime
Apr 29, 2026
Merged

feat(vm): add uptime printable column#2279
danilrwx merged 11 commits into
mainfrom
feat/vm/add-uptime

Conversation

@danilrwx
Copy link
Copy Markdown
Contributor

@danilrwx danilrwx commented Apr 27, 2026

Description

Add the Uptime printable column for VirtualMachine.

The column is derived from the Running condition transition time using reason=Running. The PhaseAge printable column was removed.

Why do we need it, and what problem does it solve?

Users need to see how long a virtual machine has been running directly in the VM list.

During migration the VM can temporarily receive another condition reason while still running. The controller now keeps the Running reason for a ready VM with an empty VMI ready reason, so the uptime column does not restart after migration.

What is the expected result?

  1. Start a virtual machine.
  2. Check the Uptime column in kubectl get vm.
  3. Migrate the VM.
  4. Verify the Uptime column does not start counting from zero again.
  5. Verify the PhaseAge column is no longer shown.

Checklist

  • The code is covered by unit tests.
  • e2e tests passed.
  • Documentation updated according to the changes.
  • Changes were tested in the Kubernetes cluster manually.

Changelog entries

section: core
type: feature
summary: Add the Uptime printable column for VirtualMachine resources.

Signed-off-by: Daniil Antoshin <daniil.antoshin@flant.com>
@danilrwx danilrwx force-pushed the feat/vm/add-uptime branch from c769c18 to 5cc0079 Compare April 27, 2026 09:43
Signed-off-by: Daniil Antoshin <daniil.antoshin@flant.com>
@danilrwx danilrwx marked this pull request as ready for review April 28, 2026 16:18
@danilrwx danilrwx added this to the v1.9.0 milestone Apr 28, 2026
Signed-off-by: Daniil Antoshin <daniil.antoshin@flant.com>
Signed-off-by: Daniil Antoshin <daniil.antoshin@flant.com>
Signed-off-by: Daniil Antoshin <daniil.antoshin@flant.com>
Signed-off-by: Daniil Antoshin <daniil.antoshin@flant.com>
Signed-off-by: Daniil Antoshin <daniil.antoshin@flant.com>
Signed-off-by: Daniil Antoshin <daniil.antoshin@flant.com>
Signed-off-by: Daniil Antoshin <daniil.antoshin@flant.com>
Signed-off-by: Daniil Antoshin <daniil.antoshin@flant.com>
Signed-off-by: Daniil Antoshin <daniil.antoshin@flant.com>
@danilrwx danilrwx force-pushed the feat/vm/add-uptime branch from e60836a to 38322d6 Compare April 29, 2026 13:51
@danilrwx danilrwx merged commit b79a975 into main Apr 29, 2026
59 of 60 checks passed
@danilrwx danilrwx deleted the feat/vm/add-uptime branch April 29, 2026 13:56
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.

2 participants