Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Labels tab to Virtual Machines resource page #1045

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

votdev
Copy link
Member

@votdev votdev commented Jun 5, 2024

Summary

Allow the user to manage the labels of a Virtual Machine resource via UI. Right now it is only possible to manage instance labels that are synced from the VM to the VMI resource.

Fixes #harvester/harvester#5974

Screenshot/Video

grafik

grafik

@votdev votdev added the Enhancement minor or greater UI enhancement label Jun 5, 2024
@votdev votdev requested a review from torchiaf June 5, 2024 14:06
@votdev votdev self-assigned this Jun 5, 2024
@votdev votdev marked this pull request as draft June 5, 2024 14:06
@votdev votdev force-pushed the issue_5974_display_labels_on_vm branch from abecb3b to 041123c Compare June 5, 2024 15:08
@votdev votdev changed the title Add Labels tab to Virtual Machines resource page Add Labels tab to VirtualMachines resource page Jun 6, 2024
@votdev votdev changed the title Add Labels tab to VirtualMachines resource page Add Labels tab to Virtual Machines resource page Jun 6, 2024
@votdev votdev mentioned this pull request Jul 1, 2024
3 tasks
@votdev votdev force-pushed the issue_5974_display_labels_on_vm branch 2 times, most recently from 6458103 to b809e94 Compare July 5, 2024 14:31
@votdev votdev force-pushed the issue_5974_display_labels_on_vm branch 2 times, most recently from 8284df3 to b4a05af Compare August 12, 2024 12:07
@votdev votdev marked this pull request as ready for review August 19, 2024 13:52
Copy link
Collaborator

@torchiaf torchiaf left a comment

Choose a reason for hiding this comment

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

LGTM, just a typo to remove.

pkg/harvester/l10n/en-us.yaml Outdated Show resolved Hide resolved
Allow the user to manage the labels of a Virtual Machine resource via UI. Right now it is only possible to manage instance labels that are synced from the VM to the VMI resource.

Related to: harvester/harvester#5974

Signed-off-by: Volker Theile <[email protected]>
@votdev votdev force-pushed the issue_5974_display_labels_on_vm branch from b4a05af to d499b45 Compare October 7, 2024 14:43
Copy link
Collaborator

@torchiaf torchiaf left a comment

Choose a reason for hiding this comment

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

LGTM

@votdev votdev force-pushed the issue_5974_display_labels_on_vm branch from d499b45 to 42793a7 Compare November 12, 2024 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement minor or greater UI enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants