-
Notifications
You must be signed in to change notification settings - Fork 66
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
Enable live-migration NBD TLS #626
Enable live-migration NBD TLS #626
Conversation
MUST set live_migration_uri as it's already set to qemu+ssh://... in the 01 nova conf file Disable post-copy when NBD TLS is enabled until https://issues.redhat.com/browse/LIBVIRT-1113 is resolved. Fix qemu cert path and permissions. Qemu is expecting the ca cert to be in a single dir and root:qemu 0640. https://docs.openstack.org/nova/latest/admin/secure-live-migration-with-qemu-native-tls.html#other-tls-environment-related-checks-on-compute-nodes Related: OSPRH-6173
Live-migration with NBD TLS is working with these changes, first migration failed with what looks like a nova bug:
Raised https://bugs.launchpad.net/nova/+bug/2061701 for the nova issue |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's make sure we capture this with a QE job as well. Particularly while we have that FIXME libvirt issue open.
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bshephar, olliewalsh, stuggi 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:
Approvers can indicate their approval by writing |
9a6b6a7
into
openstack-k8s-operators:main
{% if edpm_nova_live_migration_tls|bool %} | ||
live_migration_scheme = tls | ||
live_migration_uri = qemu+tls://%s/system |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 we had talks about replaceing this with the URI during the ptg last week so I'm glad to see this included here
MUST set live_migration_uri as it's already set to qemu+ssh://... in the 01 nova conf file
Disable post-copy when NBD TLS is enabled until
https://issues.redhat.com/browse/LIBVIRT-1113 is resolved.
Fix qemu cert path and permissions. Qemu is expecting the ca cert to be in a single dir and root:qemu 0640.
https://docs.openstack.org/nova/latest/admin/secure-live-migration-with-qemu-native-tls.html#other-tls-environment-related-checks-on-compute-nodes
Related: OSPRH-6173
Depends-On: openstack-k8s-operators/install_yamls#804
Depends-On: openstack-k8s-operators/dataplane-operator#827