chore: remove id for data source (#23)

Author: ecmadao

.gitignore

@@ -1,3 +1,39 @@
 vendor
 examples/.terraform*
 examples/terraform.*
+
+# Local .terraform directories
+**/.terraform/*
+**/.terraform.lock.hcl
+
+# .tfstate files
+*.tfstate
+*.tfstate.*
+
+# Crash log files
+crash.log
+crash.*.log
+
+# Exclude all .tfvars files, which are likely to contain sensitive data, such as
+# password, private keys, and other secrets. These should not be part of version 
+# control as they are data points which are potentially sensitive and subject 
+# to change depending on the environment.
+*.tfvars
+*.tfvars.json
+
+# Ignore override files as they are usually used to override resources locally and so
+# are not checked in
+override.tf
+override.tf.json
+*_override.tf
+*_override.tf.json
+
+# Include override files you do wish to add to version control using negated pattern
+# !example_override.tf
+
+# Include tfplan files to ignore the plan output of command: terraform plan -out=tfplan
+# example: *tfplan*
+
+# Ignore CLI configuration files
+.terraformrc
+terraform.rc

VERSION

@@ -1 +1 @@
-0.0.5-alpha
+0.0.5-beta

docs/data-sources/instance.md

@@ -55,7 +55,6 @@ You can check [examples](https://github.com/bytebase/terraform-provider-bytebase
 
 #### Read-Only
 
-- `id` (Number) The data source id.
 - `name` (String) The unique data source name in this instance.
 - `type` (String) The data source type. Should be one of:
   - `ADMIN`: The ADMIN type of data source.

docs/resources/instance.md

@@ -80,7 +80,3 @@ You can check [examples](https://github.com/bytebase/terraform-provider-bytebase
 - `ssl_key` (String) The client key. Optional, you can set this if the engine type is `CLICKHOUSE`.
 - `host_override` (String) The Read-replica Host. Only works for RO type data source.
 - `port_override` (String) The Read-replica Port. Only works for RO type data source.
-
-#### Read-Only
-
-- `id` (String) The data source unique id.

provider/data_source_instance.go

@@ -68,11 +68,6 @@ func dataSourceInstance() *schema.Resource {
 				Computed: true,
 				Elem: &schema.Resource{
 					Schema: map[string]*schema.Schema{
-						"id": {
-							Type:        schema.TypeInt,
-							Computed:    true,
-							Description: "The data source unique id",
-						},
 						"name": {
 							Type:        schema.TypeString,
 							Computed:    true,

provider/data_source_instance_list.go

@@ -71,11 +71,6 @@ func dataSourceInstanceList() *schema.Resource {
 							Computed: true,
 							Elem: &schema.Resource{
 								Schema: map[string]*schema.Schema{
-									"id": {
-										Type:        schema.TypeInt,
-										Computed:    true,
-										Description: "The data source unique id",
-									},
 									"name": {
 										Type:        schema.TypeString,
 										Computed:    true,

provider/resource_instance.go

@@ -84,11 +84,6 @@ func resourceInstance() *schema.Resource {
 				Description: "The connection for the instance. You can configure read-only or admin connection account here.",
 				Elem: &schema.Resource{
 					Schema: map[string]*schema.Schema{
-						"id": {
-							Type:        schema.TypeInt,
-							Computed:    true,
-							Description: "The data source unique id.",
-						},
 						"name": {
 							Type:        schema.TypeString,
 							Required:    true,
@@ -304,7 +299,6 @@ func flattenDataSourceList(dataSourceList []*api.DataSource) []interface{} {
 	res := []interface{}{}
 	for _, dataSource := range dataSourceList {
 		raw := map[string]interface{}{}
-		raw["id"] = dataSource.ID
 		raw["name"] = dataSource.Name
 		raw["type"] = dataSource.Type
 		raw["username"] = dataSource.Username