Update storage_credential docs to reference name not id

[karayount]

Changes

Updates docs/resources/storage_credential.md examples of creating databricks_grants resources that reference a storage_credential. Previously it had indicated referencing the credential by id. In my case, the name and id are not identical, and referencing credential by id caused errors, where referencing by name did not.

It appears that this package may be written with the assumption that name and id have the same value.
On line 117 of this same storage_credential.md it says
- 'id' - ID of this storage credential - same as the 'name'.
And there is a test catalog/external_location_test.go that indicates storage_credential name and id are interchangeable

func externalLocationTemplateWithOwner(comment string, owner string) string {
	return fmt.Sprintf(`
		resource "databricks_external_location" "some" {
			name            = "external-{var.STICKY_RANDOM}"
			url             = "s3://{env.TEST_BUCKET}/some{var.STICKY_RANDOM}"
			credential_name = databricks_storage_credential.external.id
			isolation_mode  = "ISOLATION_MODE_ISOLATED"
			comment         = "%s"
			owner = "%s"
		}
	`, comment, owner)
}

Tests

• make test run locally
• relevant change in docs/ folder
• covered with integration tests in internal/acceptance
• using Go SDK
• using TF Plugin Framework

[github-actions]

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/terraform

Inputs:

• PR number: 4684
• Commit SHA: 405e8656489d6a3db7b10ffe66ec830fd0642ae0

Checks will be approved automatically on success.

[alexott]

Show the error that you get - Storage credentials and External locations have .id the same as .name: https://github.com/databricks/terraform-provider-databricks/blob/main/catalog/resource_storage_credential.go#L90

We're using .id in the example to make sure that object is created...

[karayount]

Show the error that you get - Storage credentials and External locations have .id the same as .name: https://github.com/databricks/terraform-provider-databricks/blob/main/catalog/resource_storage_credential.go#L90

We're using .id in the example to make sure that object is created...

The terraform apply failed with this error, when I used storage_credential's .id. The screenshot shows the name it's searching for is a uuid which is our id, rather than the name ("databricks-csat-data-access")

The apply succeeded when I changed .id to .name in this code