Changed url link in readme, made changes to resource_rest to allow parallel execution

Author: anvitha-jain

README.md

@@ -25,8 +25,9 @@ $ make build
 ```
 
 Using The Provider
+<!-- https://www.terraform.io/docs/plugins/basics.html#installing-a-plugin -->
 ------------------
-If you are building the provider, follow the instructions to [install it as a plugin.](https://www.terraform.io/docs/plugins/basics.html#installing-a-plugin) After placing it into your plugins directory, run `terraform init` to initialize it.
+If you are building the provider, follow the instructions to [install it as a plugin.](https://www.terraform.io/docs/cli/plugins/index.html) After placing it into your plugins directory, run `terraform init` to initialize it.
 
 ex.
 ```hcl

aci/resource_rest.go

@@ -15,8 +15,6 @@ import (
 const Created = "created"
 const Deleted = "deleted"
 
-var class string
-
 const ErrDistinguishedNameNotFound = "The Dn is not present in the content"
 
 func resourceAciRest() *schema.Resource {
@@ -64,7 +62,7 @@ func resourceAciRestCreate(d *schema.ResourceData, m interface{}) error {
 	className := classNameIntf.(string)
 	dn := models.StripQuotes(models.StripSquareBrackets(cont.Search(className, "attributes", "dn").String()))
 	if dn == "{}" {
-		d.SetId(GetDN(d, m))
+		d.SetId(GetDN(d, m, className))
 	} else {
 		d.SetId(dn)
 	}
@@ -81,7 +79,7 @@ func resourceAciRestUpdate(d *schema.ResourceData, m interface{}) error {
 	className := classNameIntf.(string)
 	dn := models.StripQuotes(models.StripSquareBrackets(cont.Search(className, "attributes", "dn").String()))
 	if dn == "{}" {
-		d.SetId(GetDN(d, m))
+		d.SetId(GetDN(d, m, className))
 	} else {
 		d.SetId(dn)
 	}
@@ -106,11 +104,11 @@ func resourceAciRestDelete(d *schema.ResourceData, m interface{}) error {
 	return nil
 }
 
-func GetDN(d *schema.ResourceData, m interface{}) string {
+func GetDN(d *schema.ResourceData, m interface{}, className string) string {
 	aciClient := m.(*client.Client)
 	path := d.Get("path").(string)
 	cont, _ := aciClient.GetViaURL(path)
-	dn := models.StripQuotes(models.StripSquareBrackets(cont.Search("imdata", class, "attributes", "dn").String()))
+	dn := models.StripQuotes(models.StripSquareBrackets(cont.Search("imdata", className, "attributes", "dn").String()))
 	return fmt.Sprintf("%s", dn)
 }
 
@@ -169,6 +167,7 @@ func PostAndSetStatus(d *schema.ResourceData, m interface{}, status string) (*co
 	if err_output != nil {
 		return nil, err_output
 	}
+	var class string
 	for key, _ := range output {
 		class = key
 	}

examples/epg/contract.tf

@@ -10,7 +10,7 @@ resource "aci_contract" "rs_prov_contract" {
   scope                    = "context"
   target_dscp              = "VA"
   prio                     = "unspecified"
-  relation_vz_rs_graph_att = "test3" # Relation to vnsAbsGraph class. Cardinality - N_TO_ONE
+  relation_vz_rs_graph_att = aci_rest.rest_abs_graph.id # Relation to vnsAbsGraph class. Cardinality - N_TO_ONE
 }
 resource "aci_contract" "rs_cons_contract" {
   tenant_dn                = aci_tenant.tenant_for_contract.id
@@ -19,7 +19,7 @@ resource "aci_contract" "rs_cons_contract" {
   scope                    = "context"
   target_dscp              = "VA"
   prio                     = "unspecified"
-  relation_vz_rs_graph_att = "test3" # Relation to vnsAbsGraph class. Cardinality - N_TO_ONE
+  relation_vz_rs_graph_att = aci_rest.rest_abs_graph.id # Relation to vnsAbsGraph class. Cardinality - N_TO_ONE
 }
 resource "aci_contract" "intra_epg_contract" {
   tenant_dn                = aci_tenant.tenant_for_contract.id
@@ -28,5 +28,5 @@ resource "aci_contract" "intra_epg_contract" {
   scope                    = "context"
   target_dscp              = "VA"
   prio                     = "unspecified"
-  relation_vz_rs_graph_att = "test3" # Relation to vnsAbsGraph class. Cardinality - N_TO_ONE
+  relation_vz_rs_graph_att = aci_rest.rest_abs_graph.id # Relation to vnsAbsGraph class. Cardinality - N_TO_ONE
 }

examples/epg/epg.tf

@@ -22,23 +22,26 @@ resource "aci_application_epg" "demoepg" {
   description                  = "This epg is created by terraform ACI providers"
   flood_on_encap               = "disabled"
   fwd_ctrl                     = "none"
-  is_attr_based_epg           = "no"
+  is_attr_based_epg            = "no"
   match_t                      = "None"
   pc_enf_pref                  = "unenforced"
   pref_gr_memb                 = "exclude"
   prio                         = "unspecified"
   relation_fv_rs_bd            = aci_bridge_domain.bd_for_rel.id      # Relation to fvBD class. Cardinality - N_TO_ONE.
   relation_fv_rs_cust_qos_pol  = aci_rest.rest_qos_custom_pol.id      # Relation to qosCustomPol class. Cardinality - N_TO_ONE.
-  relation_fv_rs_dom_att       = [aci_rest.rest_infra_domp.id]        # Relation to infraDomP class. Cardinality - N_TO_M.
-  relation_fv_rs_fc_path_att   = ["testfabric"]                            # Relation to fabricPathEp class. Cardinality - N_TO_M.
+
+  // remove// relation_fv_rs_dom_att       = [aci_rest.rest_infra_domp.id]        # Relation to infraDomP class. Cardinality - N_TO_M.
+  // relation_fv_rs_fc_path_att   = ["testfabric"]                            # Relation to fabricPathEp class. Cardinality - N_TO_M.
+
   relation_fv_rs_prov          = [aci_contract.rs_prov_contract.id]   # Relation to vzBrCP class. Cardinality - N_TO_M.
-  relation_fv_rs_cons_if       = [aci_rest.rest_vz_cons_if.id]        # Relation to vzCPIf class. Cardinality - N_TO_M.
+
+  // relation_fv_rs_cons_if       = [aci_rest.rest_vz_cons_if.id]        # Relation to vzCPIf class. Cardinality - N_TO_M.
   relation_fv_rs_sec_inherited = [aci_application_epg.inherit_epg.id] # Relation to fvEPg class. Cardinality - N_TO_M.
-  relation_fv_rs_node_att      = ["testnodeatt"]                           # Relation to fabricNode class. Cardinality - N_TO_M.
-  relation_fv_rs_dpp_pol       = aci_rest.rest_qos_dpp_pol.id         # Relation to qosDppPol class. Cardinality - N_TO_ONE.
+  // relation_fv_rs_node_att      = ["testnodeatt"]                           # Relation to fabricNode class. Cardinality - N_TO_M.
+  // relation_fv_rs_dpp_pol       = aci_rest.rest_qos_dpp_pol.id         # Relation to qosDppPol class. Cardinality - N_TO_ONE.ye
   relation_fv_rs_cons          = [aci_contract.rs_cons_contract.id]   # Relation to vzBrCP class. Cardinality - N_TO_M.
-  relation_fv_rs_trust_ctrl    = aci_rest.rest_trust_ctrl_pol.id      # Relation to fhsTrustCtrlPol class. Cardinality - N_TO_ONE.
-  relation_fv_rs_prot_by       = [aci_rest.rest_taboo_con.id]         # Relation to vzTaboo class. Cardinality - N_TO_M.
-  relation_fv_rs_aepg_mon_pol = aci_rest.rest_mon_epg_pol.id         # Relation to monEPGPol class. Cardinality - N_TO_ONE.
+  // relation_fv_rs_trust_ctrl    = aci_rest.rest_trust_ctrl_pol.id      # Relation to fhsTrustCtrlPol class. Cardinality - N_TO_ONE.
+  // relation_fv_rs_prot_by       = [aci_rest.rest_taboo_con.id]         # Relation to vzTaboo class. Cardinality - N_TO_M.
+  // relation_fv_rs_aepg_mon_pol = aci_rest.rest_mon_epg_pol.id         # Relation to monEPGPol class. Cardinality - N_TO_ONE.
   relation_fv_rs_intra_epg     = [aci_contract.intra_epg_contract.id] # Relation to vzBrCP class. Cardinality - N_TO_M.
 }

examples/epg/rest.tf

@@ -7,7 +7,6 @@ resource "aci_rest" "rest_qos_custom_pol" {
   }
 }
 
-
 resource "aci_rest" "rest_infra_domp" {
   path       = "api/node/mo/uni/fc-test.json"
   class_name = "fcDomP"
@@ -57,3 +56,12 @@ resource "aci_rest" "rest_taboo_con" {
     "name" = "testcon"
   }
 }
+
+resource "aci_rest" "rest_abs_graph" {
+  path       = "api/node/mo/${aci_tenant.tenant_for_contract.id}/AbsGraph-testgraph.json"
+  class_name = "vnsAbsGraph"
+
+  content = {
+    "name" = "testgraph"
+  }
+}