kiranchavala commented on code in PR #53: URL: https://github.com/apache/cloudstack-terraform-provider/pull/53#discussion_r1116867776
########## README.md: ########## @@ -91,16 +105,16 @@ In order for all the tests to pass, you will need to create a new (empty) projec $ make testacc ``` -Sample Terraform configuration ------------------------------- +Sample Terraform configuration when testing locally +------------------------------------------------------------ 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" Review Comment: @harikrishna-patnala 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 we will hit if we try to replace -- 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...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org