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

use Virtual resp. Virtualization Host consistently #2668

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

keichwa
Copy link
Contributor

@keichwa keichwa commented Dec 5, 2023

Do not use Virtualized Host instead
https://bugzilla.suse.com/show_bug.cgi?id=1034064
https://github.com/SUSE/spacewalk/issues/22644

Before you submit this PR make sure you have added an entry to .changelog file.

Description

Short summary of why you created this PR (if you added documentation, please add any relevant diagram).

Target branches

  • Which documentation version does this PR apply to?
  • Does this content need to be backported? If this is the case, create an issue for tracking it and add the link to this PR.
  • Whenever possible, cross-reference each backport PR here, so that all backports can be easily accessed from the description.
  • master
  • manager-4.3

Links

  • This PR tracks issue #<insert spacewalk issue, if any>
  • Related development PR #<insert PR link, if any>

@Bischoff
Copy link
Contributor

Bischoff commented Dec 6, 2023

Hi Karl. I am not sure this really helps. When it is about virtualization (libvirt and kvm, essentially), i don't think there is anything wrong with terms like "virtual host", "virtualized system", "virtualization host", etc.

The problem arises from the "Virtual Hosts Manager", which has nothing to do with "virtual" systems, despite the name. This thing lists "foreign" systems and would be better named "Foreign Hosts Manager" or "Foreign Systems Manager"... but that's probably something for developers.

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.

None yet

2 participants