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

Libvirt SASL support #628

Merged

Conversation

olliewalsh
Copy link
Contributor

@olliewalsh olliewalsh commented Apr 16, 2024

Configure libvirt SASL authentication.
Requires a secret with LibvirtPassword key set.

Depends-On: openstack-k8s-operators/dataplane-operator#831

Related: OSPRH-6172

roles/edpm_libvirt/defaults/main.yml Show resolved Hide resolved
roles/edpm_libvirt/defaults/main.yml Outdated Show resolved Hide resolved
roles/edpm_libvirt/tasks/configure.yml Show resolved Hide resolved
roles/edpm_libvirt/tasks/configure.yml Outdated Show resolved Hide resolved
roles/edpm_libvirt/tasks/configure.yml Show resolved Hide resolved
roles/edpm_libvirt/templates/auth.conf Outdated Show resolved Hide resolved
roles/edpm_libvirt/templates/virtproxyd.conf Outdated Show resolved Hide resolved
@olliewalsh olliewalsh force-pushed the libvirt_sasl branch 3 times, most recently from 4344206 to fc2defb Compare April 16, 2024 22:49
@olliewalsh olliewalsh force-pushed the libvirt_sasl branch 5 times, most recently from 54abd3f to ad4ff99 Compare April 17, 2024 00:17
Copy link

This change depends on a change that failed to merge.

Change openstack-k8s-operators/dataplane-operator#831 is needed.

Copy link

This change depends on a change that failed to merge.

Change openstack-k8s-operators/dataplane-operator#831 is needed.

@olliewalsh
Copy link
Contributor Author

recheck

@olliewalsh
Copy link
Contributor Author

Passing tempest live migration tests in openstack-k8s-operators/openstack-operator#736

Configure libvirt SASL authentication.
Requires a secret with LibvirtPassword key set.

Depends-On: openstack-k8s-operators/dataplane-operator#831

Related: OSPRH-6172
@vakwetu
Copy link
Contributor

vakwetu commented Apr 18, 2024

+2

@openshift-ci openshift-ci bot added the lgtm label Apr 18, 2024
Copy link
Contributor

openshift-ci bot commented Apr 18, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: olliewalsh, SeanMooney

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:
  • OWNERS [SeanMooney,olliewalsh]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot openshift-merge-bot bot merged commit 319d295 into openstack-k8s-operators:main Apr 18, 2024
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants