search

ronivay / XenOrchestraInstallerUpdater

Xen Orchestra install/update script
GNU General Public License v3.0
1.18k stars 189 forks source link

Make it compatible with XCP-NG 8.2.0 #87

Closed flamme-demon closed 3 years ago

flamme-demon commented 3 years ago

Hello,

Thank you for your work. Would it be possible for you to make your script compatible with the XCP-NG 8.2.0 iso available on the official website?

image

ronivay commented 3 years ago

Hi,

Thank you. Could you clarify a bit what you mean by making it compatible? If there’s an issue, i need a little more information to work with.

flamme-demon commented 3 years ago

I'm sorry, here are some elements

Here is the error when I run the script image

If I execute the different elements to check OSNAME in the CheckOS function returns an error. image

I hope I was more precise.

ronivay commented 3 years ago

Ah okay, makes sense. One of my recent changes moved the xcp-ng/xenserver host check further so you hit the OS check before that. I’ll fix that. You should see an error that you shouldn’t install xen orchestra directly to your host. Please use xo-appliance.sh script instead to import a VM (more info in readme) or install a fresh VM with one of the supported OS’s an continue to use xo-install.sh on that VM.

E: order or checks is now fixed in https://github.com/ronivay/XenOrchestraInstallerUpdater/commit/48e24b021f0df40f856c9f0ddf5516c06e15ef52

flamme-demon commented 3 years ago

I had modified your script to see the next error. And I understood my initial error.

image

Thank you for your work.

ronivay commented 3 years ago

Cheers. Let me know if you bump into any other issues, i’ll close this for now.