fix: fix datasource result blocks

docs/data-sources/environments.md

@@ -36,10 +36,10 @@ data "octopusdeploy_environments" "example" {
 
 ### Read-Only
 
-- `environments` (Block List) Provides information about existing environments. (see [below for nested schema](#nestedblock--environments))
+- `environments` (Attributes List) (see [below for nested schema](#nestedatt--environments))
 - `id` (String) The unique ID for this resource.
 
-<a id="nestedblock--environments"></a>
+<a id="nestedatt--environments"></a>
 ### Nested Schema for `environments`
 
 Read-Only:

docs/data-sources/feeds.md

@@ -36,10 +36,10 @@ data "octopusdeploy_feeds" "example" {
 
 ### Read-Only
 
-- `feeds` (Block List) (see [below for nested schema](#nestedblock--feeds))
+- `feeds` (Attributes List) (see [below for nested schema](#nestedatt--feeds))
 - `id` (String) The unique ID for this resource.
 
-<a id="nestedblock--feeds"></a>
+<a id="nestedatt--feeds"></a>
 ### Nested Schema for `feeds`
 
 Read-Only:

docs/data-sources/library_variable_sets.md

@@ -27,9 +27,9 @@ Provides information about existing library variable sets.
 ### Read-Only
 
 - `id` (String) The unique ID for this resource.
-- `library_variable_sets` (Block List) A list of library variable sets that match the filter(s). (see [below for nested schema](#nestedblock--library_variable_sets))
+- `library_variable_sets` (Attributes List) (see [below for nested schema](#nestedatt--library_variable_sets))
 
-<a id="nestedblock--library_variable_sets"></a>
+<a id="nestedatt--library_variable_sets"></a>
 ### Nested Schema for `library_variable_sets`
 
 Read-Only:

docs/data-sources/project_groups.md

@@ -35,9 +35,9 @@ data "octopusdeploy_project_groups" "example" {
 ### Read-Only
 
 - `id` (String) The unique ID for this resource.
-- `project_groups` (Block List) A list of project groups that match the filter(s). (see [below for nested schema](#nestedblock--project_groups))
+- `project_groups` (Attributes List) A list of project groups that match the filter(s). (see [below for nested schema](#nestedatt--project_groups))
 
-<a id="nestedblock--project_groups"></a>
+<a id="nestedatt--project_groups"></a>
 ### Nested Schema for `project_groups`
 
 Read-Only:

docs/data-sources/spaces.md

@@ -34,18 +34,21 @@ data "octopusdeploy_spaces" "spaces" {
 ### Read-Only
 
 - `id` (String) The unique ID for this resource.
-- `spaces` (Block List) Provides information about existing spaces. (see [below for nested schema](#nestedblock--spaces))
+- `spaces` (Attributes List) (see [below for nested schema](#nestedatt--spaces))
 
-<a id="nestedblock--spaces"></a>
+<a id="nestedatt--spaces"></a>
 ### Nested Schema for `spaces`
 
+Required:
+
+- `name` (String) The name of this resource, no more than 20 characters long
+
 Read-Only:
 
 - `description` (String) The description of this space.
 - `id` (String) The unique ID for this resource.
 - `is_default` (Boolean) Specifies if this space is the default space in Octopus.
 - `is_task_queue_stopped` (Boolean) Specifies the status of the task queue for this space.
-- `name` (String) The name of this resource, no more than 20 characters long
 - `slug` (String) The unique slug of this space
 - `space_managers_team_members` (Set of String) A list of user IDs designated to be managers of this space.
 - `space_managers_teams` (Set of String) A list of team IDs designated to be managers of this space.

docs/data-sources/tag_sets.md

@@ -26,17 +26,17 @@ Provides information about existing tag sets.
 ### Read-Only
 
 - `id` (String) The ID of this resource.
-- `tag_sets` (List of Object) A list of tag sets that match the filter(s). (see [below for nested schema](#nestedatt--tag_sets))
+- `tag_sets` (Attributes List) A list of tag sets that match the filter(s). (see [below for nested schema](#nestedatt--tag_sets))
 
 <a id="nestedatt--tag_sets"></a>
 ### Nested Schema for `tag_sets`
 
-Read-Only:
+Optional:
 
-- `description` (String)
-- `id` (String)
-- `name` (String)
-- `sort_order` (Number)
-- `space_id` (String)
+- `description` (String) The description of this tag set.
+- `id` (String) The unique ID for this resource.
+- `name` (String) The name of this resource.
+- `sort_order` (Number) The sort order associated with this resource.
+- `space_id` (String) The space ID associated with this resource.
 
 

docs/data-sources/tenant_projects.md

@@ -24,9 +24,9 @@ Provides information about existing tenants.
 
 ### Read-Only
 
-- `tenant_projects` (Block List) A list of related tenants, projects and environments that match the filter(s). (see [below for nested schema](#nestedblock--tenant_projects))
+- `tenant_projects` (Attributes List) A list of related tenants, projects and environments that match the filter(s). (see [below for nested schema](#nestedatt--tenant_projects))
 
-<a id="nestedblock--tenant_projects"></a>
+<a id="nestedatt--tenant_projects"></a>
 ### Nested Schema for `tenant_projects`
 
 Read-Only:

docs/data-sources/variables.md

@@ -27,7 +27,6 @@ data "octopusdeploy_variables" "example" {
 
 ### Optional
 
-- `scope` (Block List) As variable names can appear more than once under different scopes, a VariableScope must also be provided (see [below for nested schema](#nestedblock--scope))
 - `space_id` (String) A Space ID to filter by. Will revert what is specified on the provider if not set.
 
 ### Read-Only
@@ -37,24 +36,11 @@ data "octopusdeploy_variables" "example" {
 - `is_editable` (Boolean) Indicates whether or not this variable is considered editable.
 - `is_sensitive` (Boolean) Indicates whether or not this resource is considered sensitive and should be kept secret.
 - `prompt` (Attributes List) (see [below for nested schema](#nestedatt--prompt))
+- `scope` (Attributes List) As variable names can appear more than once under different scopes, a VariableScope must also be provided (see [below for nested schema](#nestedatt--scope))
 - `sensitive_value` (String, Sensitive)
 - `type` (String) The type of variable represented by this resource. Valid types are `AmazonWebServicesAccount`, `AzureAccount`, `GoogleCloudAccount`, `UsernamePasswordAccount`, `Certificate`, `Sensitive`, `String`, `WorkerPool`.
 - `value` (String)
 
-<a id="nestedblock--scope"></a>
-### Nested Schema for `scope`
-
-Optional:
-
-- `actions` (List of String) A list of actions that are scoped to this variable value.
-- `channels` (List of String) A list of channels that are scoped to this variable value.
-- `environments` (List of String) A list of environments that are scoped to this variable value.
-- `machines` (List of String) A list of machines that are scoped to this variable value.
-- `processes` (List of String) A list of processes that are scoped to this variable value.
-- `roles` (List of String) A list of roles that are scoped to this variable value.
-- `tenant_tags` (List of String) A list of tenant tags that are scoped to this variable value.
-
-
 <a id="nestedatt--prompt"></a>
 ### Nested Schema for `prompt`
 
@@ -82,3 +68,19 @@ Read-Only:
 - `value` (String) The select value
 
 
+
+
+<a id="nestedatt--scope"></a>
+### Nested Schema for `scope`
+
+Optional:
+
+- `actions` (List of String) A list of actions that are scoped to this variable value.
+- `channels` (List of String) A list of channels that are scoped to this variable value.
+- `environments` (List of String) A list of environments that are scoped to this variable value.
+- `machines` (List of String) A list of machines that are scoped to this variable value.
+- `processes` (List of String) A list of processes that are scoped to this variable value.
+- `roles` (List of String) A list of roles that are scoped to this variable value.
+- `tenant_tags` (List of String) A list of tenant tags that are scoped to this variable value.
+
+

octopusdeploy_framework/datasource_environments.go

@@ -52,11 +52,10 @@ func (*environmentDataSource) Schema(_ context.Context, req datasource.SchemaReq
 
 			//response
 			"id": schemas.GetIdDatasourceSchema(true),
-		},
-		Blocks: map[string]schema.Block{
-			"environments": schema.ListNestedBlock{
-				Description: "Provides information about existing environments.",
-				NestedObject: schema.NestedBlockObject{
+			"environments": schema.ListNestedAttribute{
+				Computed: true,
+				Optional: false,
+				NestedObject: schema.NestedAttributeObject{
 					Attributes: schemas.GetEnvironmentDatasourceSchema(),
 				},
 			},

octopusdeploy_framework/datasource_feeds.go

@@ -32,13 +32,6 @@ func (*feedsDataSource) Schema(_ context.Context, req datasource.SchemaRequest,
 	resp.Schema = datasourceSchema.Schema{
 		Description: "Provides information about existing feeds.",
 		Attributes:  schemas.GetFeedsDataSourceSchema(),
-		Blocks: map[string]datasourceSchema.Block{
-			"feeds": datasourceSchema.ListNestedBlock{
-				NestedObject: datasourceSchema.NestedBlockObject{
-					Attributes: schemas.GetFeedDataSourceSchema(),
-				},
-			},
-		},
 	}
 }
 

octopusdeploy_framework/datasource_project_groups.go

@@ -57,11 +57,10 @@ func (p *projectGroupsDataSource) Schema(_ context.Context, _ datasource.SchemaR
 
 			// response
 			"id": schemas.GetIdDatasourceSchema(true),
-		},
-		Blocks: map[string]schema.Block{
-			"project_groups": schema.ListNestedBlock{
+			"project_groups": schema.ListNestedAttribute{
+				Computed:    true,
 				Description: "A list of project groups that match the filter(s).",
-				NestedObject: schema.NestedBlockObject{
+				NestedObject: schema.NestedAttributeObject{
 					Attributes: schemas.GetProjectGroupDatasourceSchema(),
 				},
 			},

octopusdeploy_framework/datasource_spaces.go

@@ -43,12 +43,11 @@ func (*spacesDataSource) Schema(_ context.Context, _ datasource.SchemaRequest, r
 
 			// response
 			"id": schemas.GetIdDatasourceSchema(true),
-		},
-		Blocks: map[string]schema.Block{
-			"spaces": schema.ListNestedBlock{
-				Description: "Provides information about existing spaces.",
-				NestedObject: schema.NestedBlockObject{
-					Attributes: schemas.GetSpacesDatasourceSchema(),
+			"spaces": schema.ListNestedAttribute{
+				Computed: true,
+				Optional: false,
+				NestedObject: schema.NestedAttributeObject{
+					Attributes: schemas.GetSpaceDatasourceSchema(),
 				},
 			},
 		},

octopusdeploy_framework/datasource_tenant_projects.go

@@ -46,11 +46,11 @@ func (*tenantProjectsDataSource) Schema(_ context.Context, req datasource.Schema
 			"project_ids":     schemas.GetQueryIDsDatasourceSchema(),
 			"environment_ids": schemas.GetQueryIDsDatasourceSchema(),
 			"space_id":        schemas.GetSpaceIdDatasourceSchema("tenant projects", false),
-		},
-		Blocks: map[string]datasourceSchema.Block{
-			"tenant_projects": datasourceSchema.ListNestedBlock{
+			"tenant_projects": datasourceSchema.ListNestedAttribute{
+				Computed:    true,
+				Optional:    false,
 				Description: "A list of related tenants, projects and environments that match the filter(s).",
-				NestedObject: datasourceSchema.NestedBlockObject{
+				NestedObject: datasourceSchema.NestedAttributeObject{
 					Attributes: map[string]datasourceSchema.Attribute{
 						"id": schemas.GetIdDatasourceSchema(true),
 						"tenant_id": datasourceSchema.StringAttribute{

octopusdeploy_framework/schemas/feed.go

@@ -82,6 +82,13 @@ func GetFeedsDataSourceSchema() map[string]datasourceSchema.Attribute {
 
 		// response
 		"id": GetIdDatasourceSchema(true),
+		"feeds": datasourceSchema.ListNestedAttribute{
+			Computed: true,
+			Optional: false,
+			NestedObject: datasourceSchema.NestedAttributeObject{
+				Attributes: GetFeedDataSourceSchema(),
+			},
+		},
 	}
 }
 

octopusdeploy_framework/schemas/gitCredential.go

@@ -53,6 +53,7 @@ func GetGitCredentialDataSourceSchema() datasourceSchema.Schema {
 			"take":     util.DataSourceInt64().Optional().Description("The number of records to take.").Build(),
 			"git_credentials": datasourceSchema.ListNestedAttribute{
 				Computed:    true,
+				Optional:    false,
 				Description: "Provides information about existing GitCredentials.",
 				NestedObject: datasourceSchema.NestedAttributeObject{
 					Attributes: GetGitCredentialDatasourceAttributes(),

octopusdeploy_framework/schemas/library_variable_set.go

@@ -25,34 +25,29 @@ type LibraryVariableSetResourceModel struct {
 
 func GetLibraryVariableSetDataSourceSchema() datasourceSchema.Schema {
 	return datasourceSchema.Schema{
-		Attributes:  getLibraryVariableSetDataSchema(),
 		Description: "Provides information about existing library variable sets.",
-		Blocks: map[string]datasourceSchema.Block{
-			"library_variable_sets": datasourceSchema.ListNestedBlock{
-				Description: "A list of library variable sets that match the filter(s).",
-				NestedObject: datasourceSchema.NestedBlockObject{
+		Attributes: map[string]datasourceSchema.Attribute{
+			"content_type": datasourceSchema.StringAttribute{
+				Description: "A filter to search by content type.",
+				Optional:    true,
+			},
+			"id":           GetIdDatasourceSchema(true),
+			"space_id":     GetSpaceIdDatasourceSchema("library variable set", false),
+			"ids":          util.GetQueryIDsDatasourceSchema(),
+			"partial_name": util.GetQueryPartialNameDatasourceSchema(),
+			"skip":         util.GetQuerySkipDatasourceSchema(),
+			"take":         util.GetQueryTakeDatasourceSchema(),
+			"library_variable_sets": datasourceSchema.ListNestedAttribute{
+				Computed: true,
+				Optional: false,
+				NestedObject: datasourceSchema.NestedAttributeObject{
 					Attributes: GetLibraryVariableSetObjectDatasourceSchema(),
 				},
 			},
 		},
 	}
 }
 
-func getLibraryVariableSetDataSchema() map[string]datasourceSchema.Attribute {
-	return map[string]datasourceSchema.Attribute{
-		"content_type": datasourceSchema.StringAttribute{
-			Description: "A filter to search by content type.",
-			Optional:    true,
-		},
-		"id":           GetIdDatasourceSchema(true),
-		"space_id":     GetSpaceIdDatasourceSchema("library variable set", false),
-		"ids":          util.GetQueryIDsDatasourceSchema(),
-		"partial_name": util.GetQueryPartialNameDatasourceSchema(),
-		"skip":         util.GetQuerySkipDatasourceSchema(),
-		"take":         util.GetQueryTakeDatasourceSchema(),
-	}
-}
-
 func GetLibraryVariableSetObjectDatasourceSchema() map[string]datasourceSchema.Attribute {
 	return map[string]datasourceSchema.Attribute{
 		"description": GetReadonlyDescriptionDatasourceSchema("library variable set"),

octopusdeploy_framework/schemas/lifecycle.go

@@ -114,6 +114,7 @@ func getResourceRetentionPolicyBlockSchema() resourceSchema.ListNestedBlock {
 func getLifecyclesAttribute() datasourceSchema.ListNestedAttribute {
 	return datasourceSchema.ListNestedAttribute{
 		Computed: true,
+		Optional: false,
 		NestedObject: datasourceSchema.NestedAttributeObject{
 			Attributes: map[string]datasourceSchema.Attribute{
 				"id":                        util.DataSourceString().Computed().Description("The ID of the lifecycle.").Build(),

octopusdeploy_framework/schemas/tag_set.go

@@ -74,10 +74,39 @@ func GetTagSetDataSourceSchema() datasourceSchema.Schema {
 				Optional().
 				Description("A filter to specify the number of items to take (or return) in the response.").
 				Build(),
-			"tag_sets": datasourceSchema.ListAttribute{
+			"tag_sets": datasourceSchema.ListNestedAttribute{
 				Computed:    true,
-				ElementType: types.ObjectType{AttrTypes: GetTagSetAttrTypes()},
+				Optional:    false,
 				Description: "A list of tag sets that match the filter(s).",
+				NestedObject: datasourceSchema.NestedAttributeObject{
+					Attributes: map[string]datasourceSchema.Attribute{
+						"id": util.DataSourceString().
+							Optional().
+							Computed().
+							Description("The unique ID for this resource.").
+							Build(),
+						"name": util.DataSourceString().
+							Optional().
+							Computed().
+							Description("The name of this resource.").
+							Build(),
+						"description": util.DataSourceString().
+							Optional().
+							Computed().
+							Description("The description of this tag set.").
+							Build(),
+						"sort_order": util.DataSourceInt64().
+							Optional().
+							Computed().
+							Description("The sort order associated with this resource.").
+							Build(),
+						"space_id": util.DataSourceString().
+							Optional().
+							Computed().
+							Description("The space ID associated with this resource.").
+							Build(),
+					},
+				},
 			},
 		},
 	}

octopusdeploy_framework/schemas/variable.go

@@ -135,9 +135,7 @@ func GetVariableDatasourceSchema() datasourceSchema.Schema {
 				Computed: true,
 			},
 			VariableSchemaAttributeNames.Prompt: getVariablePromptDatasourceSchema(),
-		},
-		Blocks: map[string]datasourceSchema.Block{
-			VariableSchemaAttributeNames.Scope: getVariableScopeDatasourceSchema(),
+			VariableSchemaAttributeNames.Scope:  getVariableScopeDatasourceSchema(),
 		},
 	}
 }

octopusdeploy_framework/schemas/variable_scope.go

@@ -128,10 +128,11 @@ func getVariableScopeFieldResourceSchema(scopeDescription string) resourceSchema
 	}
 }
 
-func getVariableScopeDatasourceSchema() datasourceSchema.ListNestedBlock {
-	return datasourceSchema.ListNestedBlock{
+func getVariableScopeDatasourceSchema() datasourceSchema.ListNestedAttribute {
+	return datasourceSchema.ListNestedAttribute{
 		Description: "As variable names can appear more than once under different scopes, a VariableScope must also be provided",
-		NestedObject: datasourceSchema.NestedBlockObject{
+		Computed:    true,
+		NestedObject: datasourceSchema.NestedAttributeObject{
 			Attributes: map[string]datasourceSchema.Attribute{
 				variableScopeFieldNames.Actions:      getVariableScopeFieldDatasourceSchema(variableScopeFieldNames.Actions),
 				variableScopeFieldNames.Channels:     getVariableScopeFieldDatasourceSchema(variableScopeFieldNames.Channels),