-
Notifications
You must be signed in to change notification settings - Fork 37
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
Adding configuration resource #57
base: main
Are you sure you want to change the base?
Conversation
@poddm thanks for the PR, do we need to use the latest go-sdk release (https://github.com/apache/cloudstack-go/releases) ? LGTM havn't tested it though. (closed/reopen PR to rekick Github actions build job) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM didn't test it though
e6e7db2
to
a0c7bb6
Compare
a0c7bb6
to
555472a
Compare
@vishesh92, These were cascading changes from updating to the latest client sdk apache/cloudstack-go#47. They were not meant to be committed. I'll rebase when #101 is merged. |
Update - I'd still like to get these merged once the cloudstack SDK is updated. Related SDK changes |
Closing/re-opening to kick tests. |
@poddm there are conflicts on the PR - can you please resolve? |
555472a
to
afb0d11
Compare
@poddm Acceptance tests are failing. Can you check and fix those. |
@poddm can you review/rebase the PR, advise on outstanding review comments & look at failing tests. Thanks. |
115e7d8
to
1f3678d
Compare
@rohityadavcloud @vishesh92 , rebased and updated to the latest cloudstack client. |
Adding cloudstack configuration resource.
This requires the latest client. See -
https://github.com/apache/cloudstack-go/releases/tag/v2.14.0
apache/cloudstack-go#47