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

Ref #36499 - libvirt configuration needs server and username #168

Merged
merged 1 commit into from Jul 31, 2023

Conversation

lfu
Copy link
Contributor

@lfu lfu commented Jul 20, 2023

server=qemu:///system is the configuration for kvm hypervisor where virt-who is installed locally on the hypervisor
and the configuration script runs on the kvm hypervisor as well.

The virt-who support for rhevm is to run the virt-who configuration script on the satellite server.
The configuration needs the information about the hypervisor server and username.
Satellite server's SSH key should be added to rhevm hypervisor's authorized_keys.

Followup of #166

Steps to test

  1. copy satellite server's key to rhevm hypervisor
    # ssh-keygen -t rsa -P '' -f /root/.ssh/id_rsa ; ssh-copy-id -i /root/.ssh/id_rsa.pub hypervisor.redhat.com
  2. Register guest to satellite
  3. Create virt-who config with hypervisor type libvirt
  4. Deploy virt-who config on satellite server
    # hammer virt-who-config deploy --id 1 --organization-id 1
  5. check mapping info in /var/log/rhsm/rhsm.log

Copy link
Member

@chris1984 chris1984 left a comment

Choose a reason for hiding this comment

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

@lfu

I am not seeing the username or server field added back:

Screenshot from 2023-07-25 16-22-09

Copy link
Member

@chris1984 chris1984 left a comment

Choose a reason for hiding this comment

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

Tried a different browser and machine and i'm seeing the fields correctly now.

ACK

@lfu
Copy link
Contributor Author

lfu commented Jul 31, 2023

image

@lfu lfu merged commit 09262b7 into theforeman:master Jul 31, 2023
1 check passed
@lfu lfu deleted the libivrt_config_36499 branch November 3, 2023 20:46
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