Libvirt/include sasl auth #809

Closed
wants to merge 3 commits into
from

Conversation

Projects
None yet
3 participants
@Katana-Steel
Contributor

Katana-Steel commented Jun 9, 2016

include sasl auth to libvirt_driver

Description

Adding SASL authentication to QEMU+tcp remote connections

Because this really annoyed me. We have about +100 KVM servers and I'd
just to iterate over them and get all the available nodes. Apply a few filters
and then return a list of nodes where the end user don't need to worry about
which KVM host his VM is running on and just be able to reboot it should he
need to.

Status

  • work in progress

Checklist (tick everything that applies)

  • Code linting (required, can be done after the PR checks)
  • Documentation
  • Tests
  • ICLA (required for bigger changes)
@Kami

This comment has been minimized.

Show comment
Hide comment
@Kami

Kami Jun 9, 2016

Member

Awesome, thanks.

I will look into it ASAP.

Member

Kami commented Jun 9, 2016

Awesome, thanks.

I will look into it ASAP.

Rene Kjellerup and others added some commits Jun 9, 2016

Libvirt include sasl authentication setup
inlude sasl auth for uri='qemu+tcp://<host>/system' to provide
access to remote libvirtd servers providing user and pass
authentication.
added documentation
updated the constructor documentation and added description for
the callback function.
+ except libvirt.libvirtError:
+ if key is None or secret is None:
+ raise RuntimeError('The remote Libvirt instance requires ' +
+ 'authenication, please set \'key\' and ' +

This comment has been minimized.

@tonybaloney

tonybaloney Jun 14, 2016

Contributor

little typo

@tonybaloney

tonybaloney Jun 14, 2016

Contributor

little typo

This comment has been minimized.

@Katana-Steel

Katana-Steel Jun 15, 2016

Contributor

Thanks, fixed

@Katana-Steel

Katana-Steel Jun 15, 2016

Contributor

Thanks, fixed

@tonybaloney

This comment has been minimized.

Show comment
Hide comment
@tonybaloney

tonybaloney Jun 24, 2016

Contributor

👍

Contributor

tonybaloney commented Jun 24, 2016

👍

@asfgit asfgit closed this in 00b70ef Jun 24, 2016

asfgit pushed a commit that referenced this pull request Jun 24, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment