Re: [PR] update listApis.json, Quota API, and fixed double url value handling [cloudstack-go]

2024-12-13 Thread via GitHub
DaanHoogland commented on PR #90: URL: https://github.com/apache/cloudstack-go/pull/90#issuecomment-2541600390 @tonymmm1 can you deal with the conflict, please? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

Re: [PR] Add missing API calls [cloudstack-go]

2024-12-13 Thread via GitHub
phsm commented on PR #91: URL: https://github.com/apache/cloudstack-go/pull/91#issuecomment-2541358557 I see you've merged the previous commits implementing the slices of API calls that were emerged at different CS versions. This may as well render my PR obsolete. Later I will

Re: [PR] Add missing API calls [cloudstack-go]

2024-12-13 Thread via GitHub
DaanHoogland commented on PR #91: URL: https://github.com/apache/cloudstack-go/pull/91#issuecomment-254119 @phsm I merged some PRs and now you are stuck with a lot of conflicts. So for I can see these are not complex, just a lot. Can you have a look, please? -- This is an automated me

Re: [PR] Update SDK with 4.20 APIs [cloudstack-go]

2024-12-13 Thread via GitHub
DaanHoogland commented on PR #93: URL: https://github.com/apache/cloudstack-go/pull/93#issuecomment-2541149461 > code loks good and I can run tests locally without errors, but I see a failing github action. I can run the setup tools step locally without these errors, so probably a runner en

Re: [PR] Update SDK with 4.20 APIs [cloudstack-go]

2024-12-13 Thread via GitHub
DaanHoogland merged PR #93: URL: https://github.com/apache/cloudstack-go/pull/93 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@cloudst

Re: [PR] Update SDK with 4.20 APIs [cloudstack-go]

2024-12-13 Thread via GitHub
DaanHoogland closed pull request #93: Update SDK with 4.20 APIs URL: https://github.com/apache/cloudstack-go/pull/93 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscr

Re: [PR] Update go version [cloudstack-go]

2024-12-13 Thread via GitHub
DaanHoogland merged PR #94: URL: https://github.com/apache/cloudstack-go/pull/94 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@cloudst

Re: [PR] update listApis.json, Quota API, and fixed double url value handling [cloudstack-go]

2024-12-13 Thread via GitHub
DaanHoogland closed pull request #90: update listApis.json, Quota API, and fixed double url value handling URL: https://github.com/apache/cloudstack-go/pull/90 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

Re: [PR] add +30 more apis [cloudstack-go]

2024-12-13 Thread via GitHub
DaanHoogland merged PR #89: URL: https://github.com/apache/cloudstack-go/pull/89 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@cloudst

Re: [I] createUser/updateUser APIs need to be POST [cloudstack-go]

2024-12-13 Thread via GitHub
DaanHoogland commented on issue #85: URL: https://github.com/apache/cloudstack-go/issues/85#issuecomment-2541114637 fixed by #86 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comme

Re: [I] createUser/updateUser APIs need to be POST [cloudstack-go]

2024-12-13 Thread via GitHub
DaanHoogland closed issue #85: createUser/updateUser APIs need to be POST URL: https://github.com/apache/cloudstack-go/issues/85 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment.

Re: [PR] update listApis.json, Quota API, and fixed double url value handling [cloudstack-go]

2024-12-13 Thread via GitHub
DaanHoogland commented on code in PR #90: URL: https://github.com/apache/cloudstack-go/pull/90#discussion_r1883710327 ## generate/generate.go: ## @@ -1865,6 +1865,20 @@ func (s *service) generateResponseType(a *API) { tn := capitalize(strings.TrimPrefix(a.Name, "configur

Re: [PR] Update SDK with 4.20 APIs [cloudstack-go]

2024-12-13 Thread via GitHub
DaanHoogland commented on PR #93: URL: https://github.com/apache/cloudstack-go/pull/93#issuecomment-2541094765 code loks good and I can run tests locally without errors, but I see a failing github action. I can run the setup tools step locally without these errors, so probably a runner env