[T-5361] Rename resource_group_id to source_group_id (#36)

Author: mikebaldry

Makefile

@@ -8,7 +8,7 @@ GOLANGCI_LINT := golangci-lint run --disable-all \
 	-E staticcheck \
 	-E typecheck \
 	-E unused
-VERSION := 0.5
+VERSION := 0.5.1
 .PHONY: test build
 
 help:

docs/data-sources/source.md

@@ -79,12 +79,12 @@ This Data Source allows you to look up existing Sources using their table name.
     - `ubuntu`
     - `vector`
     - `vercel_integration`
-- `resource_group_id` (Number) The ID of the resource group this source belongs to.
 - `scrape_frequency_secs` (Number) For scrape platform types, how often to scrape the URLs.
 - `scrape_request_basic_auth_password` (String, Sensitive) Basic auth password for scraping.
 - `scrape_request_basic_auth_user` (String) Basic auth username for scraping.
 - `scrape_request_headers` (List of Map of String) An array of request headers, each containing `name` and `value` fields.
 - `scrape_urls` (List of String) For scrape platform types, the set of urls to scrape.
+- `source_group_id` (Number) The ID of the source group this source belongs to.
 - `team_name` (String) Used to specify the team the resource should be created in when using global tokens.
 - `token` (String) The token of this source. This token is used to identify and route the data you will send to Better Stack.
 - `updated_at` (String) The time when this monitor group was updated.

docs/resources/source.md

@@ -75,12 +75,12 @@ This resource allows you to create, modify, and delete your Sources. For more in
 - `live_tail_pattern` (String) Freeform text template for formatting Live tail output with columns wrapped in {column} brackets. Example: "PID: {message_json.pid} {level} {message}"
 - `logs_retention` (Number) Data retention for logs in days. There might be additional charges for longer retention.
 - `metrics_retention` (Number) Data retention for metrics in days. There might be additional charges for longer retention.
-- `resource_group_id` (Number) The ID of the resource group this source belongs to.
 - `scrape_frequency_secs` (Number) For scrape platform types, how often to scrape the URLs.
 - `scrape_request_basic_auth_password` (String, Sensitive) Basic auth password for scraping.
 - `scrape_request_basic_auth_user` (String) Basic auth username for scraping.
 - `scrape_request_headers` (List of Map of String) An array of request headers, each containing `name` and `value` fields.
 - `scrape_urls` (List of String) For scrape platform types, the set of urls to scrape.
+- `source_group_id` (Number) The ID of the source group this source belongs to.
 - `team_name` (String) Used to specify the team the resource should be created in when using global tokens.
 
 ### Read-Only

internal/provider/resource_source.go

@@ -203,8 +203,8 @@ var sourceSchema = map[string]*schema.Schema{
 		Optional:    true,
 		Computed:    true,
 	},
-	"resource_group_id": {
-		Description: "The ID of the resource group this source belongs to.",
+	"source_group_id": {
+		Description: "The ID of the source group this source belongs to.",
 		Type:        schema.TypeInt,
 		Optional:    true,
 	},
@@ -244,7 +244,7 @@ type source struct {
 	ScrapeRequestBasicAuthUser     *string                   `json:"scrape_request_basic_auth_user,omitempty"`
 	ScrapeRequestBasicAuthPassword *string                   `json:"scrape_request_basic_auth_password,omitempty"`
 	DataRegion                     *string                   `json:"data_region,omitempty"`
-	ResourceGroupID                *int                      `json:"resource_group_id,omitempty"`
+	SourceGroupID                  *int                      `json:"source_group_id,omitempty"`
 }
 
 type sourceHTTPResponse struct {
@@ -279,7 +279,7 @@ func sourceRef(in *source) []struct {
 		{k: "scrape_request_basic_auth_user", v: &in.ScrapeRequestBasicAuthUser},
 		{k: "scrape_request_basic_auth_password", v: &in.ScrapeRequestBasicAuthPassword},
 		{k: "data_region", v: &in.DataRegion},
-		{k: "resource_group_id", v: &in.ResourceGroupID},
+		{k: "source_group_id", v: &in.SourceGroupID},
 	}
 }
 

internal/provider/resource_source_test.go

@@ -93,7 +93,7 @@ func TestResourceSource(t *testing.T) {
 					name             = "%s"
 					platform         = "%s"
 					data_region      = "eu-hel-1-legacy"
-					resource_group_id = 123
+					source_group_id = 123
 				}
 				`, name, platform),
 				Check: resource.ComposeTestCheckFunc(
@@ -103,7 +103,7 @@ func TestResourceSource(t *testing.T) {
 					resource.TestCheckResourceAttr("logtail_source.this", "token", "generated_by_logtail"),
 					resource.TestCheckResourceAttr("logtail_source.this", "ingesting_host", "in.logs.betterstack.com"),
 					resource.TestCheckResourceAttr("logtail_source.this", "data_region", "eu-hel-1-legacy"),
-					resource.TestCheckResourceAttr("logtail_source.this", "resource_group_id", "123"),
+					resource.TestCheckResourceAttr("logtail_source.this", "source_group_id", "123"),
 				),
 			},
 			// Step 2 - update.
@@ -120,7 +120,7 @@ func TestResourceSource(t *testing.T) {
 					metrics_retention = 60
    					live_tail_pattern = "{level} {message}"
 					ingesting_paused  = true
-					resource_group_id = 456
+					source_group_id = 456
 				}
 				`, name, platform),
 				Check: resource.ComposeTestCheckFunc(
@@ -131,7 +131,7 @@ func TestResourceSource(t *testing.T) {
 					resource.TestCheckResourceAttr("logtail_source.this", "token", "generated_by_logtail"),
 					resource.TestCheckResourceAttr("logtail_source.this", "ingesting_host", "in.logs.betterstack.com"),
 					resource.TestCheckResourceAttr("logtail_source.this", "data_region", "eu-hel-1-legacy"),
-					resource.TestCheckResourceAttr("logtail_source.this", "resource_group_id", "456"),
+					resource.TestCheckResourceAttr("logtail_source.this", "source_group_id", "456"),
 				),
 			},
 			// Step 3 - make no changes, check plan is empty (omitted attributes are not controlled)
@@ -144,7 +144,7 @@ func TestResourceSource(t *testing.T) {
 				resource "logtail_source" "this" {
 					name             = "%s"
 					platform         = "%s"
-					resource_group_id = 456
+					source_group_id = 456
 				}
 				`, name, platform),
 				PlanOnly: true,