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

New version of vSphere REST API #31

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

kiruka4
Copy link

@kiruka4 kiruka4 commented Jun 4, 2021

Without my changes this script didn't work with vCenter v 7.0.2. Additional information about REST API changes here - https://core.vmware.com/blog/vsphere-7-update-2-rest-api-modernization . Also script tried to register new host in any folder under specified datacenter. As for me it's inconvinient trying to find new host in a random folder. So I changed this behavior. Now new host will be added directly in the datacenter. Without any folders

Without my changes this script didn't work with vCenter v 7.0.2. Additional information about REST API changes here - https://core.vmware.com/blog/vsphere-7-update-2-rest-api-modernization . Also script tried to register new host in any folder under specified datacenter. As for me it's inconvinient trying to find new host in a random folder. So I changed this behavior. Now new host will be added directly in the datacenter. Without any folders
@ltrager
Copy link
Contributor

ltrager commented Jun 4, 2021

Thanks for the patch! Do you know if these changes are compatible with vCenter 6? If not we need to modify this so it works with vCenter 6 and 7.

@ITJamie
Copy link

ITJamie commented Apr 25, 2022

its not backwards compatible.

@SK1Y101
Copy link
Member

SK1Y101 commented Aug 18, 2023

Hey @kiruka4,
Necoring an old PR here: Are you still willing to work on this?
If so, could you add a small modification to switch between /rest and /api depending on the version of VCentre in use?
We'd rather not introduce breaking changes where possible.
Thanks!

@SK1Y101 SK1Y101 added bug Something isn't working ESXi triaged Triaged to be addressed in a given cycle labels Aug 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ESXi triaged Triaged to be addressed in a given cycle
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants