PLT-1326 added context to read cloud account

docs/data-sources/cloudaccount_aws.md

@@ -33,6 +33,7 @@ data "spectrocloud_cloudaccount_aws" "aws_account" {
 
 ### Optional
 
+- `context` (String) The context of the cluster. Allowed values are `project` or `tenant` or ``.
 - `depends` (String)
 - `id` (String) ID of the AWS cloud account registered in Palette.
 - `name` (String) Name of the AWS cloud account registered in Palette.

docs/data-sources/cloudaccount_azure.md

@@ -33,6 +33,7 @@ data "spectrocloud_cloudaccount_azure" "azure_account" {
 
 ### Optional
 
+- `context` (String) The context of the cluster. Allowed values are `project` or `tenant` or ``.
 - `id` (String) ID of the Azure cloud account registered in Palette.
 - `name` (String) Name of the Azure cloud account registered in Palette.
 

docs/data-sources/cloudaccount_custom.md

@@ -21,5 +21,6 @@ description: |-
 
 ### Optional
 
+- `context` (String) The context of the cluster. Allowed values are `project` or `tenant` or ``.
 - `id` (String) The unique identifier of the cloud account. Either `id` or `name` must be provided.
 - `name` (String) The name of the cloud account. Either `id` or `name` must be provided.

docs/data-sources/cloudaccount_gcp.md

@@ -33,5 +33,6 @@ data "spectrocloud_cloudaccount_gcp" "gcp_account" {
 
 ### Optional
 
+- `context` (String) The context of the cluster. Allowed values are `project` or `tenant` or ``.
 - `id` (String) ID of the GCP cloud account registered in Palette.
 - `name` (String) Name of the GCP cloud account registered in Palette.

docs/data-sources/cloudaccount_maas.md

@@ -17,6 +17,7 @@ description: |-
 
 ### Optional
 
+- `context` (String) The context of the cluster. Allowed values are `project` or `tenant` or ``.
 - `id` (String) The unique ID of the cloud account. Either `id` or `name` must be provided, but not both.
 - `name` (String) The name of the cloud account. This can be used instead of `id` to retrieve the account details. Only one of `id` or `name` can be specified.
 

docs/data-sources/cloudaccount_openstack.md

@@ -17,5 +17,6 @@ description: |-
 
 ### Optional
 
+- `context` (String) The context of the cluster. Allowed values are `project` or `tenant` or ``.
 - `id` (String) The unique ID of the OpenStack cloud account. Either `id` or `name` must be provided, but not both.
 - `name` (String) The name of the OpenStack cloud account. Either `id` or `name` must be provided, but not both.

docs/data-sources/cloudaccount_tencent.md

@@ -17,6 +17,7 @@ description: |-
 
 ### Optional
 
+- `context` (String) The context of the cluster. Allowed values are `project` or `tenant` or ``.
 - `id` (String) The unique ID of the Tencent cloud account. Either `id` or `name` must be provided, but not both.
 - `name` (String) The name of the Tencent cloud account. Either `id` or `name` must be provided, but not both.
 

docs/data-sources/cloudaccount_vsphere.md

@@ -17,5 +17,6 @@ description: |-
 
 ### Optional
 
+- `context` (String) The context of the cluster. Allowed values are `project` or `tenant` or ``.
 - `id` (String) The unique ID of the vSphere cloud account. Either `id` or `name` must be provided, but not both.
 - `name` (String) The name of the vSphere cloud account. Either `id` or `name` must be provided, but not both.

examples/data-sources/spectrocloud_cloud_account/data-source.tf

@@ -0,0 +1,9 @@
+data "spectrocloud_cloudaccount_aws" "aws_account" {
+  # id = <uid>
+  name = "ran-tf"
+  # context = "tenant"
+}
+
+output "aws_account" {
+  value = data.spectrocloud_cloudaccount_aws.aws_account
+}

examples/data-sources/spectrocloud_cloud_account/providers.tf

@@ -0,0 +1,18 @@
+terraform {
+  required_providers {
+    spectrocloud = {
+      version = ">= 0.13.2"
+      source  = "spectrocloud/spectrocloud"
+    }
+  }
+}
+
+variable "sc_host" {}
+variable "sc_api_key" {}
+variable "sc_project_name" {}
+
+provider "spectrocloud" {
+  host         = var.sc_host
+  api_key      = var.sc_api_key
+  project_name = var.sc_project_name
+}

examples/data-sources/spectrocloud_cloud_account/terraform.template.tfvars

@@ -0,0 +1,4 @@
+# Spectro Cloud credentials
+sc_host         = "{Enter Spectro Cloud API Host}" #e.g: api.spectrocloud.com (for SaaS)
+sc_api_key      = "{Enter Spectro Cloud API Key}"
+sc_project_name = "{Enter Spectro Cloud Project Name}" #e.g: Default

spectrocloud/data_source_cloud_account_aws.go

@@ -2,9 +2,10 @@ package spectrocloud
 
 import (
 	"context"
-
+	"fmt"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/diag"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
+	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/validation"
 	"github.com/spectrocloud/palette-sdk-go/api/models"
 )
 
@@ -28,6 +29,13 @@ func dataSourceCloudAccountAws() *schema.Resource {
 				Computed:     true,
 				ExactlyOneOf: []string{"id", "name"},
 			},
+			"context": {
+				Type:         schema.TypeString,
+				Optional:     true,
+				Default:      "",
+				ValidateFunc: validation.StringInSlice([]string{"", "project", "tenant"}, false),
+				Description:  "The context of the cluster. Allowed values are `project` or `tenant` or ``. ",
+			},
 			"depends": {
 				Type:         schema.TypeString,
 				Optional:     true,
@@ -48,20 +56,38 @@ func dataSourceCloudAccountAwsRead(_ context.Context, d *schema.ResourceData, m
 	if err != nil {
 		return diag.FromErr(err)
 	}
-
-	var account *models.V1AwsAccount
-	for _, a := range accounts {
-
-		if v, ok := d.GetOk("id"); ok && v.(string) == a.Metadata.UID {
-			account = a
-			break
-		} else if v, ok := d.GetOk("name"); ok && v.(string) == a.Metadata.Name {
-			account = a
+	var fAccount *models.V1AwsAccount
+	filteredAccounts := make([]*models.V1AwsAccount, 0)
+	for _, acc := range accounts {
+		if v, ok := d.GetOk("id"); ok && v.(string) == acc.Metadata.UID {
+			fAccount = acc
 			break
 		}
+		if v, ok := d.GetOk("name"); ok && v.(string) == acc.Metadata.Name {
+			filteredAccounts = append(filteredAccounts, acc)
+		}
+	}
+	if len(filteredAccounts) > 1 {
+		if accContext, ok := d.GetOk("context"); ok && accContext != "" {
+			for _, ac := range filteredAccounts {
+				if ac.Metadata.Annotations["scope"] == accContext {
+					fAccount = ac
+					break
+				}
+			}
+		} else {
+			diags = append(diags, diag.Diagnostic{
+				Severity: diag.Error,
+				Summary:  "Found more than 1 AWS account",
+				Detail:   fmt.Sprintf("more than 1 aws account found for name - '%s'. Kindly re-try with `context` set, Allowed value `project` or `tenant`", d.Get("name").(string)),
+			})
+			return diags
+		}
+	} else if len(filteredAccounts) == 1 {
+		fAccount = filteredAccounts[0]
 	}
 
-	if account == nil {
+	if fAccount == nil {
 		diags = append(diags, diag.Diagnostic{
 			Severity: diag.Error,
 			Summary:  "Unable to find aws cloud account",
@@ -70,8 +96,8 @@ func dataSourceCloudAccountAwsRead(_ context.Context, d *schema.ResourceData, m
 		return diags
 	}
 
-	d.SetId(account.Metadata.UID)
-	err = d.Set("name", account.Metadata.Name)
+	d.SetId(fAccount.Metadata.UID)
+	err = d.Set("name", fAccount.Metadata.Name)
 	if err != nil {
 		return diag.FromErr(err)
 	}

spectrocloud/data_source_cloud_account_aws_test.go

@@ -0,0 +1,35 @@
+package spectrocloud
+
+import (
+	"context"
+	"github.com/hashicorp/terraform-plugin-sdk/v2/diag"
+	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
+	"github.com/stretchr/testify/assert"
+	"testing"
+)
+
+func prepareDataSourceCloudAccountAwsSchema() *schema.ResourceData {
+	d := dataSourceCloudAccountAws().TestResourceData()
+	return d
+}
+
+func TestDataSourceCloudAccountAwsRead(t *testing.T) {
+	d := prepareDataSourceCloudAccountAwsSchema()
+	var diags diag.Diagnostics
+
+	var ctx context.Context
+	_ = d.Set("name", "test-aws-account-1")
+	_ = d.Set("context", "project")
+	diags = dataSourceCloudAccountAwsRead(ctx, d, unitTestMockAPIClient)
+	assert.Equal(t, 0, len(diags))
+}
+
+func TestDataSourceCloudAccountAwsReadNegative(t *testing.T) {
+	d := prepareDataSourceCloudAccountAwsSchema()
+	var diags diag.Diagnostics
+
+	var ctx context.Context
+	_ = d.Set("name", "test-aws-account-1")
+	diags = dataSourceCloudAccountAwsRead(ctx, d, unitTestMockAPINegativeClient)
+	assertFirstDiagMessage(t, diags, "Unable to find aws cloud account")
+}

spectrocloud/data_source_cloud_account_azure.go

@@ -2,6 +2,8 @@ package spectrocloud
 
 import (
 	"context"
+	"fmt"
+	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/validation"
 
 	"github.com/hashicorp/terraform-plugin-sdk/v2/diag"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
@@ -48,6 +50,13 @@ func dataSourceCloudAccountAzure() *schema.Resource {
 				Description: "The status of the disable properties option.",
 				Computed:    true,
 			},
+			"context": {
+				Type:         schema.TypeString,
+				Optional:     true,
+				Default:      "",
+				ValidateFunc: validation.StringInSlice([]string{"", "project", "tenant"}, false),
+				Description:  "The context of the cluster. Allowed values are `project` or `tenant` or ``. ",
+			},
 		},
 	}
 }
@@ -64,15 +73,35 @@ func dataSourceCloudAccountAzureRead(_ context.Context, d *schema.ResourceData,
 	}
 
 	var account *models.V1AzureAccount
+	filteredAccounts := make([]*models.V1AzureAccount, 0)
 	for _, a := range accounts {
 
 		if v, ok := d.GetOk("id"); ok && v.(string) == a.Metadata.UID {
 			account = a
 			break
 		} else if v, ok := d.GetOk("name"); ok && v.(string) == a.Metadata.Name {
-			account = a
-			break
+			filteredAccounts = append(filteredAccounts, a)
+		}
+	}
+
+	if len(filteredAccounts) > 1 {
+		if accContext, ok := d.GetOk("context"); ok && accContext != "" {
+			for _, ac := range filteredAccounts {
+				if ac.Metadata.Annotations["scope"] == accContext {
+					account = ac
+					break
+				}
+			}
+		} else {
+			diags = append(diags, diag.Diagnostic{
+				Severity: diag.Error,
+				Summary:  "Found multiple cloud accounts",
+				Detail:   fmt.Sprintf("more than 1 account found for name - '%s'. Kindly re-try with `context` set, Allowed value `project` or `tenant`", d.Get("name").(string)),
+			})
+			return diags
 		}
+	} else if len(filteredAccounts) == 1 {
+		account = filteredAccounts[0]
 	}
 
 	if account == nil {

spectrocloud/data_source_cloud_account_custom.go

@@ -2,8 +2,10 @@ package spectrocloud
 
 import (
 	"context"
+	"fmt"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/diag"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
+	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/validation"
 	"github.com/spectrocloud/palette-sdk-go/api/models"
 )
 
@@ -31,6 +33,13 @@ func dataSourceCloudAccountCustom() *schema.Resource {
 				Required:    true,
 				Description: "The custom cloud provider name (e.g., `nutanix`).",
 			},
+			"context": {
+				Type:         schema.TypeString,
+				Optional:     true,
+				Default:      "",
+				ValidateFunc: validation.StringInSlice([]string{"", "project", "tenant"}, false),
+				Description:  "The context of the cluster. Allowed values are `project` or `tenant` or ``. ",
+			},
 		},
 	}
 }
@@ -46,14 +55,34 @@ func dataSourceCloudAccountCustomRead(_ context.Context, d *schema.ResourceData,
 		return diag.FromErr(err)
 	}
 	var account *models.V1CustomAccount
+	filteredAccounts := make([]*models.V1CustomAccount, 0)
 	for _, a := range accounts {
 		if v, ok := d.GetOk("id"); ok && v.(string) == a.Metadata.UID {
 			account = a
 			break
 		} else if v, ok := d.GetOk("name"); ok && v.(string) == a.Metadata.Name {
-			account = a
-			break
+			filteredAccounts = append(filteredAccounts, a)
+		}
+	}
+
+	if len(filteredAccounts) > 1 {
+		if accContext, ok := d.GetOk("context"); ok && accContext != "" {
+			for _, ac := range filteredAccounts {
+				if ac.Metadata.Annotations["scope"] == accContext {
+					account = ac
+					break
+				}
+			}
+		} else {
+			diags = append(diags, diag.Diagnostic{
+				Severity: diag.Error,
+				Summary:  "Found multiple cloud accounts",
+				Detail:   fmt.Sprintf("more than 1 account found for name - '%s'. Kindly re-try with `context` set, Allowed value `project` or `tenant`", d.Get("name").(string)),
+			})
+			return diags
 		}
+	} else if len(filteredAccounts) == 1 {
+		account = filteredAccounts[0]
 	}
 
 	if account == nil {