Skip to content

[CLOUDSTACK-9923] Add support for vSphere 6.5 - #2111

Closed
FabriceBrazier wants to merge 3 commits into
apache:masterfrom
apalia:CLOUDSTACK-9923
Closed

[CLOUDSTACK-9923] Add support for vSphere 6.5#2111
FabriceBrazier wants to merge 3 commits into
apache:masterfrom
apalia:CLOUDSTACK-9923

Conversation

@FabriceBrazier

Copy link
Copy Markdown
Contributor

vSphere 6.5 is not registered in the ACS DB. Therefore, new VMs are always created with os type "other". This is a blocking point, especially when using Windows 2012 OS (boot process fails).
This fix updates the DB accordingly.

@resmo

resmo commented May 17, 2017

Copy link
Copy Markdown
Member

Since vSphere 6.5 is officially supported in LTS 4.9, this should probably even go into 4.9.3 /cc @rhtyd Any thoughts?

I'll have to check again but this was probably the cause of issues we had of putting an updgraded ESX from 5.5 to 6.5 into cloudstack . AFAIR we had to remove and add the host to the cluster in cloudstack. Let me verify again.

@resmo
resmo self-requested a review May 17, 2017 22:01
@yadvr

yadvr commented May 18, 2017

Copy link
Copy Markdown
Member

@resmo yes, good idea
@FabriceBrazier yes, if possible can you target the PR for 4.9 branch, and move the sql changes into the 4.9.2->4.9.3 upgrade path. There is an outstanding PR #1951 which @karuturi has asked me to merge, I can fwd-merge this to master and change master's upgrade path from 4.9.2.0 to 4.9.3.0->4.10.0.0. For master, the upgrade path looks as: 4.9.2.0->4.9.3.0->4.10.0.0. After the fwd-merge, we'll freeze any changes to the 4.9.2.0->4.9.3.0 upgrade path.

@yadvr

yadvr commented May 18, 2017

Copy link
Copy Markdown
Member

LGTM, otherwise.

@FabriceBrazier

Copy link
Copy Markdown
Contributor Author

Modif is done. Could you please approve?

@yadvr

yadvr commented May 18, 2017

Copy link
Copy Markdown
Member

Thanks @FabriceBrazier but that's not what I meant, we wanted to get this into 4.9 branch. Look, I've picked your change as a separate commit in: #1951 see if that's okay?

@FabriceBrazier

Copy link
Copy Markdown
Contributor Author

Thanks, ok for me. I will close this ticket.

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.

4 participants