-
Notifications
You must be signed in to change notification settings - Fork 7
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
Can't enable HA. #8
Comments
Hi, I'm successful added the SR before using GUI or CLI without patch, but HA was not working: Enabling HA on pool 'XEN-SRV-00' Thanks for the driver! |
Hi Tom, Did you patch it with the file inside of the RBD-HA.patch.zip or with copy-paste from the comment? Like I've said, the comment doesn't have proper indent(the github formatting thing) so it will throw an error if you've used comment version. You can reverse the patch and then apply one from the zip or you can open LVMoISCSISR with vim or nano and insert 8 spaces before statement on line 586 and probably 587,8,9. If you having problems with that, email me the LVMoISCSISR and I'll have a look |
Sorry being so late but everything is working fine, I had to apply the patch which added: With this patch only I could move VMs and recover them after node failure. I have put steps on my blog: how to create the enviorment, Thank you for the great work and providing us awesome driver! Regards, |
Hi Tom, That's great news. I just have few suggestions about your instructions if I may:
Other than that, excellent stuff. I appreciate your interest in the plugin! :) One of those days I will merge the localIQN changes into the RBDSR.py. I just need to test and recreate patches, because that one line would shift everything around. |
iSCSI SR uses dedicated class to enable HA(LVHDoISCSIVDI) and tests if iscsid daemon is running and initiator name is set with "localIQN" parameter.
This key is not applicable to RBD connection, since we only need rbd driver to be loaded into the kernel to be able to establish connection to CEPH cluster.
Following patch(no spaces in the patch preview, RBD-HA.patch.zip attached) would workaround the problem, but need to test if there are any other uses of localIQN in HA.
RBD-HA.patch.zip
--- /opt/xensource/sm/LVHDoISCSISR.py 2016-05-09 14:48:38.000000000 +1000
+++ ./LVHDoISCSISR.py 2016-05-09 14:48:19.000000000 +1000
@@ -552,7 +552,10 @@
if not lvutil._checkLV(self.path):
raise xs_errors.XenError('VDIUnavailable')
dict = {}
- self.sr.dconf['localIQN'] = self.sr.iscsi.localIQN
+ if self.sr.dconf['port'] in '6789':
+ self.sr.dconf['localIQN'] = 'rbd-client'
+ else:
+ self.sr.dconf['localIQN'] = self.sr.iscsi.localIQN
self.sr.dconf['multipathing'] = self.sr.mpath
self.sr.dconf['multipathhandle'] = self.sr.mpathhandle
dict['device_config'] = self.sr.dconf
The text was updated successfully, but these errors were encountered: