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

Reply via email to