-
Notifications
You must be signed in to change notification settings - Fork 35
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
cloudstack/provider.go
Outdated
"cloudstack_disk": resourceCloudStackDisk(), | ||
"cloudstack_egress_firewall": resourceCloudStackEgressFirewall(), | ||
"cloudstack_firewall": resourceCloudStackFirewall(), | ||
"cloudstack_instance": resourceCloudStackInstance(), | ||
"cloudstack_ipaddress": resourceCloudStackIPAddress(), | ||
"cloudstack_kubernetes_cluster": resourceCloudStackKubernetesCluster(), |
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.
why is it required to remove this?
@@ -570,7 +570,7 @@ func resourceCloudStackInstanceUpdate(d *schema.ResourceData, meta interface{}) | |||
if d.HasChange("keypair") { | |||
log.Printf("[DEBUG] SSH keypair changed for %s, starting update", name) | |||
|
|||
p := cs.SSH.NewResetSSHKeyForVirtualMachineParams(d.Id(), d.Get("keypair").(string)) | |||
p := cs.SSH.NewResetSSHKeyForVirtualMachineParams(d.Id()) |
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.
you should set the keypair
param here as well.
@@ -100,7 +100,6 @@ func resourceCloudStackPrivateGatewayCreate(d *schema.ResourceData, meta interfa | |||
d.Get("gateway").(string), | |||
ipaddress, | |||
d.Get("netmask").(string), | |||
d.Get("vlan").(string), | |||
d.Get("vpc_id").(string), | |||
) |
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.
If vlan is available, update the params.
d.Set("name", v.Name) | ||
d.Set("value", v.Value) | ||
d.Set("scope", v.Scope) | ||
found = true |
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.
found = true | |
found = true | |
break |
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. |
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