-
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
Added correct docker and terraform configuration command #53
Added correct docker and terraform configuration command #53
Conversation
bf2e77c
to
c7fdb45
Compare
Below is an example configuration to initialize provider and create a Virtual Machine instance | ||
|
||
```sh | ||
$ cat provider.tf | ||
terraform { | ||
required_providers { | ||
cloudstack = { | ||
source = "cloudstack/cloudstack" | ||
source = "localdomain/provider/cloudstack" |
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.
The previous one worked when I was testing locally @kiranchavala , may you want to add what localdomain/provider means ?
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.
after building the terraform provider locally using "make build", the generated go binary will contain the new code changes
To test the changes we have to use
source = "localdomain/provider/cloudstack
If we use " source "cloudstack/cloudstack" ,
when "terraform init" is called terraform will download the binary from terraform registry https://registry.terraform.io/providers/cloudstack/cloudstack/latest
and place it /.terraform.d/plugins/registry.terraform.io/cloudstack/cloudstack/0.4.0/linux_amd64/terraform-provider-cloudstack_v0.4.0/
If replace the binary with the binary we had built in the same path and do a "terraform apply" , you will encounter a checksum error as terraform.lock.hcl will be different
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.
thanks @kiranchavala I used to delete the lock file to test my changes, so that way it worked.
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
@rohityadavcloud @davidjumani @shwstppr Could you please review this |
Co-authored-by: David Jumani <[email protected]>
Co-authored-by: David Jumani <[email protected]>
If there are any new changes @kiranchavala pl submit as separate PR cc @harikrishna-patnala |
* Added correct docker and terraform configuration commands * Update README.md Co-authored-by: David Jumani <[email protected]> * Update README.md Co-authored-by: David Jumani <[email protected]> --------- Co-authored-by: Rohit Yadav <[email protected]> Co-authored-by: David Jumani <[email protected]>
No description provided.