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

Add missing API calls #91

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Add missing API calls #91

wants to merge 3 commits into from

Conversation

phsm
Copy link

@phsm phsm commented Nov 5, 2024

Fixes #22

I've noticed that a lot of API calls are missing from the cloudstack/ directory.

Thus, I took listApis output from the latest Cloudstack version at the time of writing this, 4.19.1.2, and added all those missing calls to the layout.go.

There are 3 commits in the PR:

  • One updates listApis to the newer Cloudstack version.
  • The second one adds the missing API calls. I've also moved some calls to the Service they actually belong, e.g. createStoragePool is not a part of the PoolService, but of the StoragePoolService.
  • The last commit contains all the files that make all generated

There were no errors/warnings during the make:

~/g/cloudstack-go main ↑2 ↓2 $ make all
go run generate/generate.go generate/layout.go generate/requiredParams.go --api=generate/listApis.json
~/g/cloudstack-go main* ↑2 ↓2 28.8s $

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing API support
1 participant