Skip to content

Change hypercore_vm group parameter to list of tags #53

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

Merged
merged 1 commit into from
May 13, 2025

Conversation

anazobec
Copy link
Collaborator

@anazobec anazobec commented May 12, 2025

Closes #49

This MR changes the VM's group parameter to list of tags.

  • before: group = "my-group-tag"
  • now: tags = ["tag1", "tag2", ...]

@anazobec anazobec requested a review from justinc1 May 12, 2025 07:54
@anazobec anazobec self-assigned this May 12, 2025
Copy link
Collaborator

@justinc1 justinc1 left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you @anazobec

@anazobec anazobec force-pushed the 49-allow-manipulating-list-of-vm-tags branch from 3a40761 to d2848da Compare May 12, 2025 13:04
@anazobec
Copy link
Collaborator Author

@justinc1 Upon a quick self-review before merge, I found I forgot to also modify the integration tests - replace group with tags

@anazobec anazobec force-pushed the 49-allow-manipulating-list-of-vm-tags branch from d2848da to 2f939a3 Compare May 13, 2025 05:51
Changed the VM's group parameter to list of tags.
- before: `group = "my-group-tag"`
- now: `tags = ["tag1", "tag2", ...]`

Updated integration tests - `group` -> `tags`
@anazobec anazobec force-pushed the 49-allow-manipulating-list-of-vm-tags branch from 2f939a3 to 02fcdc9 Compare May 13, 2025 05:53
Copy link
Collaborator

@domendobnikar domendobnikar left a comment

Choose a reason for hiding this comment

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

Looks good, thanks Ana 👍

@anazobec anazobec merged commit af8c302 into main May 13, 2025
3 of 4 checks passed
@anazobec anazobec deleted the 49-allow-manipulating-list-of-vm-tags branch May 13, 2025 13:03
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 12, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

hypercore_vm should allow manipulating list of vm tags (not just first tag which = group)
3 participants