STAC-22844: Drop support for setting up a scope on a subject

Author: aacevedoosorio

cmd/rbac/flags.go

@@ -4,20 +4,14 @@ const (
 	Subject    = "subject"
 	Permission = "permission"
 	Resource   = "resource"
-	Scope      = "scope"
 
 	SubjectUsage            = "The handle of the subject of the permissions. If you're using LDAP, please use the usernameKey configured in StackState"
 	PermissionRevokeUsage   = "The permission to revoke"
 	PermissionGrantUsage    = "The permission to grant"
 	PermissionDescribeUsage = "Filter the permissions by permission name"
-	ResourceDescribeUsage   = "Filter the permissions by a resource identifier (e.g. system or a view name)"
-	ResourceGrantUsage      = "The resource to grant the permission to (e.g. \"system\" or a view name)"
-	ResourceRevokeUsage     = "The resource to revoke the permission to (e.g. \"system\" or a view name)"
-	ScopeUsage              = "The query in STQL that will be prepended to every topology element retrieved in StackState. " +
-		"For example, if your scope is \"label = 'A'\", then all STQL executed in StackState" +
-		" (e.g. Retrieving topology) will only return elements that have the label A"
+	ResourceDescribeUsage   = "Filter the permissions by a resource identifier (e.g. system, a resource name or tag)"
+	ResourceGrantUsage      = "The resource to grant the permission to (e.g. \"system\" or a resource name or tag)"
+	ResourceRevokeUsage     = "The resource to revoke the permission to (e.g. \"system\" or a resource name or tag)"
 
-	DefaultResource    = "system"
-	DefaultScope       = "id = '-1'"
-	DefaultSTQLVersion = "0.0.1"
+	DefaultResource = "system"
 )

cmd/rbac/rbac_create_subject.go

@@ -9,7 +9,6 @@ import (
 
 type CreateSubjectArgs struct {
 	Subject string
-	Scope   string
 }
 
 func CreateSubjectCommand(deps *di.Deps) *cobra.Command {
@@ -24,8 +23,6 @@ func CreateSubjectCommand(deps *di.Deps) *cobra.Command {
 	cmd.Flags().StringVar(&args.Subject, Subject, "", SubjectUsage)
 	cmd.MarkFlagRequired(Subject) //nolint:errcheck
 
-	cmd.Flags().StringVar(&args.Scope, Scope, DefaultScope, ScopeUsage)
-
 	return cmd
 }
 
@@ -36,9 +33,7 @@ func RunCreateSubjectCommand(args *CreateSubjectArgs) di.CmdWithApiFn {
 		api *stackstate_api.APIClient,
 		serverInfo *stackstate_api.ServerInfo,
 	) common.CLIError {
-		subject := stackstate_api.NewCreateSubject(args.Scope, DefaultSTQLVersion)
 		resp, err := api.SubjectApi.CreateSubject(cli.Context, args.Subject).
-			CreateSubject(*subject).
 			Execute()
 
 		if err != nil {

cmd/rbac/rbac_create_subject_test.go

@@ -4,7 +4,6 @@ import (
 	"fmt"
 	"testing"
 
-	"github.com/stackvista/stackstate-cli/generated/stackstate_api"
 	"github.com/stackvista/stackstate-cli/internal/di"
 	"github.com/stretchr/testify/assert"
 )
@@ -24,10 +23,6 @@ func TestCreateSubjectJson(t *testing.T) {
 	assert.Len(t, calls, 1)
 	assert.Equal(t, SomeSubject, calls[0].Psubject)
 
-	expectedSubject := stackstate_api.NewCreateSubject(DefaultScope, DefaultSTQLVersion)
-
-	assert.Equal(t, expectedSubject, calls[0].PcreateSubject)
-
 	expectedJson := []map[string]interface{}{
 		{
 			"created-subject": SomeSubject,
@@ -41,16 +36,12 @@ func TestCreateSubject(t *testing.T) {
 	cli := di.NewMockDeps(t)
 	cmd := CreateSubjectCommand(&cli.Deps)
 
-	di.ExecuteCommandWithContextUnsafe(&cli.Deps, cmd, "--subject", SomeOtherSubject, "--scope", SomeScope)
+	di.ExecuteCommandWithContextUnsafe(&cli.Deps, cmd, "--subject", SomeOtherSubject)
 
 	calls := *cli.MockClient.ApiMocks.SubjectApi.CreateSubjectCalls
 	assert.Len(t, calls, 1)
 	assert.Equal(t, SomeOtherSubject, calls[0].Psubject)
 
-	otherExpectedSubject := stackstate_api.NewCreateSubject(SomeScope, DefaultSTQLVersion)
-
-	assert.Equal(t, otherExpectedSubject, calls[0].PcreateSubject)
-
 	expectedStrings := []string{
 		fmt.Sprintf("Created subject '%s'", SomeOtherSubject),
 	}

cmd/rbac/rbac_describe_subjects.go

@@ -42,16 +42,14 @@ func RunDescribeSubjectsCommand(args *DescribeSubjectsArgs) di.CmdWithApiFn {
 
 			if cli.IsJson() {
 				cli.Printer.PrintJson(map[string]interface{}{
-					"handle":     subject.Handle,
-					"scopeQuery": safeDeref(subject.ScopeQuery),
+					"handle": subject.Handle,
 				})
 			} else {
 				cli.Printer.Table(printer.TableData{
-					Header: []string{"Subject", "Scope Query"},
+					Header: []string{"Subject"},
 					Data: [][]interface{}{
 						{
 							subject.Handle,
-							safeDeref(subject.ScopeQuery),
 						},
 					},
 					MissingTableDataMsg: printer.NotFoundMsg{Types: "matching subjects"},
@@ -72,11 +70,11 @@ func RunDescribeSubjectsCommand(args *DescribeSubjectsArgs) di.CmdWithApiFn {
 				data := make([][]interface{}, 0)
 
 				for _, subject := range subjects {
-					data = append(data, []interface{}{subject.Handle, safeDeref(subject.ScopeQuery)})
+					data = append(data, []interface{}{subject.Handle})
 				}
 
 				cli.Printer.Table(printer.TableData{
-					Header:              []string{"Subject", "Scope Query"},
+					Header:              []string{"Subject"},
 					Data:                data,
 					MissingTableDataMsg: printer.NotFoundMsg{Types: "subjects"},
 				})

cmd/rbac/rbac_describe_subjects_test.go

@@ -12,21 +12,17 @@ import (
 var (
 	SomeScopeVar   = SomeScope
 	SubjectConfig1 = stackstate_api.SubjectConfig{
-		Handle:     SomeSubject,
-		ScopeQuery: &SomeScopeVar,
+		Handle: SomeSubject,
 	}
 
 	SomeOtherSubject = "handle"
-	SomeOtherScope   = "meaningOfLife = 23"
 
 	SubjectConfig2 = stackstate_api.SubjectConfig{
-		Handle:     SomeOtherSubject,
-		ScopeQuery: &SomeOtherScope,
+		Handle: SomeOtherSubject,
 	}
 
 	SubjectConfig3 = stackstate_api.SubjectConfig{
-		Handle:     SubjectHandle,
-		ScopeQuery: nil,
+		Handle: SubjectHandle,
 	}
 )
 
@@ -47,11 +43,11 @@ func TestDescribeSubjectsTable(t *testing.T) {
 
 	expected := []printer.TableData{
 		{
-			Header: []string{"Subject", "Scope Query"},
+			Header: []string{"Subject"},
 			Data: [][]interface{}{
-				{SubjectConfig1.Handle, *SubjectConfig1.ScopeQuery},
-				{SubjectConfig2.Handle, *SubjectConfig2.ScopeQuery},
-				{SubjectConfig3.Handle, ""},
+				{SubjectConfig1.Handle},
+				{SubjectConfig2.Handle},
+				{SubjectConfig3.Handle},
 			},
 			MissingTableDataMsg: printer.NotFoundMsg{Types: "subjects"},
 		},
@@ -100,9 +96,9 @@ func TestDescribeSubjectsTableWithFilter(t *testing.T) {
 
 	expected := []printer.TableData{
 		{
-			Header: []string{"Subject", "Scope Query"},
+			Header: []string{"Subject"},
 			Data: [][]interface{}{
-				{SubjectConfig1.Handle, *SubjectConfig1.ScopeQuery},
+				{SubjectConfig1.Handle},
 			},
 			MissingTableDataMsg: printer.NotFoundMsg{Types: "matching subjects"},
 		},
@@ -130,12 +126,10 @@ func TestDescribeSubjectsJsonWithFilter(t *testing.T) {
 
 	expectedJson := []map[string]interface{}{
 		{
-			"handle":     SubjectConfig1.Handle,
-			"scopeQuery": *SubjectConfig1.ScopeQuery,
+			"handle": SubjectConfig1.Handle,
 		},
 		{
-			"handle":     SubjectConfig3.Handle,
-			"scopeQuery": "",
+			"handle": SubjectConfig3.Handle,
 		},
 	}
 

generated/stackstate_api/.openapi-generator/FILES

@@ -8,6 +8,7 @@ api_agent_registrations.go
 api_api_token.go
 api_authorize_ingestion_api_key.go
 api_component.go
+api_dashboards.go
 api_dummy.go
 api_event.go
 api_export.go
@@ -84,7 +85,23 @@ docs/ComponentHighlightMetricSectionAllOf.md
 docs/ComponentNotFoundError.md
 docs/ComponentQuery.md
 docs/ComponentViewArguments.md
-docs/CreateSubject.md
+docs/DashboardAuthorizationError.md
+docs/DashboardClientErrors.md
+docs/DashboardCloneSchema.md
+docs/DashboardInvalidIdentifierError.md
+docs/DashboardList.md
+docs/DashboardNotFoundError.md
+docs/DashboardPatchSchema.md
+docs/DashboardReadBaseSchema.md
+docs/DashboardReadFullSchema.md
+docs/DashboardReadFullSchemaAllOf.md
+docs/DashboardReadMetadataSchema.md
+docs/DashboardReadMetadataSchemaAllOf.md
+docs/DashboardReadSchema.md
+docs/DashboardScope.md
+docs/DashboardValidationError.md
+docs/DashboardWriteSchema.md
+docs/DashboardsApi.md
 docs/DataUnavailable.md
 docs/DependencyDirection.md
 docs/DummyApi.md
@@ -278,10 +295,42 @@ docs/OpsgenieChannelRefId.md
 docs/OpsgenieChannelWriteSchema.md
 docs/OpsgenieNotificationChannel.md
 docs/OpsgenieNotificationChannelAllOf.md
+docs/OpsgeniePriority.md
+docs/OpsgenieRegion.md
 docs/OpsgenieResponder.md
+docs/OpsgenieResponderType.md
 docs/PermissionDescription.md
 docs/Permissions.md
 docs/PermissionsApi.md
+docs/PersesDashboard.md
+docs/PersesDashboardDisplaySpec.md
+docs/PersesDashboardSpec.md
+docs/PersesDatasourceSpec.md
+docs/PersesGridItem.md
+docs/PersesGridLayoutCollapse.md
+docs/PersesGridLayoutDisplay.md
+docs/PersesJSONRef.md
+docs/PersesLayout.md
+docs/PersesLayoutKind.md
+docs/PersesLayoutSpec.md
+docs/PersesLink.md
+docs/PersesListVariable.md
+docs/PersesListVariableDefaultSingleValue.md
+docs/PersesListVariableDefaultSliceValues.md
+docs/PersesListVariableDefaultValue.md
+docs/PersesListVariableSpec.md
+docs/PersesPanel.md
+docs/PersesPanelDisplay.md
+docs/PersesPanelSpec.md
+docs/PersesPlugin.md
+docs/PersesProjectMetadata.md
+docs/PersesQuery.md
+docs/PersesQuerySpec.md
+docs/PersesTextVariable.md
+docs/PersesTextVariableSpec.md
+docs/PersesVariableDisplaySpec.md
+docs/PersesVariableSort.md
+docs/PersesVariableTypes.md
 docs/ProblemApi.md
 docs/ProblemNotFound.md
 docs/PromData.md
@@ -332,6 +381,7 @@ docs/SpanFilter.md
 docs/SpanKind.md
 docs/SpanLink.md
 docs/SpanParentType.md
+docs/SpanResponse.md
 docs/SpanSortDirection.md
 docs/SpanSortField.md
 docs/SpanSortOption.md
@@ -369,6 +419,7 @@ docs/TimelineSummary.md
 docs/TimelineSummaryError.md
 docs/TimelineSummaryEventBucket.md
 docs/TimelineSummaryHealthChange.md
+docs/TimelineSummaryOverflow.md
 docs/TimelineSummaryRequest.md
 docs/TimelineSummaryRequestArguments.md
 docs/TooManyActiveQueries.md
@@ -397,9 +448,10 @@ docs/TraceApiSpanNotFound.md
 docs/TraceApiSpansBadRequest.md
 docs/TraceApiTraceNotFound.md
 docs/TraceFilter.md
-docs/TraceIdentifier.md
 docs/TraceQuery.md
-docs/Traces.md
+docs/TraceQueryMatch.md
+docs/TraceQueryMatchAllOf.md
+docs/TraceQueryResponse.md
 docs/TracesApi.md
 docs/UnlicensedSubscription.md
 docs/UnmatchedCheckState.md
@@ -458,7 +510,22 @@ model_component_highlight_metric_section_all_of.go
 model_component_not_found_error.go
 model_component_query.go
 model_component_view_arguments.go
-model_create_subject.go
+model_dashboard_authorization_error.go
+model_dashboard_client_errors.go
+model_dashboard_clone_schema.go
+model_dashboard_invalid_identifier_error.go
+model_dashboard_list.go
+model_dashboard_not_found_error.go
+model_dashboard_patch_schema.go
+model_dashboard_read_base_schema.go
+model_dashboard_read_full_schema.go
+model_dashboard_read_full_schema_all_of.go
+model_dashboard_read_metadata_schema.go
+model_dashboard_read_metadata_schema_all_of.go
+model_dashboard_read_schema.go
+model_dashboard_scope.go
+model_dashboard_validation_error.go
+model_dashboard_write_schema.go
 model_data_unavailable.go
 model_dependency_direction.go
 model_duration_histogram.go
@@ -638,9 +705,41 @@ model_opsgenie_channel_ref_id.go
 model_opsgenie_channel_write_schema.go
 model_opsgenie_notification_channel.go
 model_opsgenie_notification_channel_all_of.go
+model_opsgenie_priority.go
+model_opsgenie_region.go
 model_opsgenie_responder.go
+model_opsgenie_responder_type.go
 model_permission_description.go
 model_permissions.go
+model_perses_dashboard.go
+model_perses_dashboard_display_spec.go
+model_perses_dashboard_spec.go
+model_perses_datasource_spec.go
+model_perses_grid_item.go
+model_perses_grid_layout_collapse.go
+model_perses_grid_layout_display.go
+model_perses_json_ref.go
+model_perses_layout.go
+model_perses_layout_kind.go
+model_perses_layout_spec.go
+model_perses_link.go
+model_perses_list_variable.go
+model_perses_list_variable_default_single_value.go
+model_perses_list_variable_default_slice_values.go
+model_perses_list_variable_default_value.go
+model_perses_list_variable_spec.go
+model_perses_panel.go
+model_perses_panel_display.go
+model_perses_panel_spec.go
+model_perses_plugin.go
+model_perses_project_metadata.go
+model_perses_query.go
+model_perses_query_spec.go
+model_perses_text_variable.go
+model_perses_text_variable_spec.go
+model_perses_variable_display_spec.go
+model_perses_variable_sort.go
+model_perses_variable_types.go
 model_problem_not_found.go
 model_prom_data.go
 model_prom_data_result.go
@@ -687,6 +786,7 @@ model_span_filter.go
 model_span_kind.go
 model_span_link.go
 model_span_parent_type.go
+model_span_response.go
 model_span_sort_direction.go
 model_span_sort_field.go
 model_span_sort_option.go
@@ -719,6 +819,7 @@ model_timeline_summary.go
 model_timeline_summary_error.go
 model_timeline_summary_event_bucket.go
 model_timeline_summary_health_change.go
+model_timeline_summary_overflow.go
 model_timeline_summary_request.go
 model_timeline_summary_request_arguments.go
 model_too_many_active_queries.go
@@ -745,9 +846,10 @@ model_trace_api_span_not_found.go
 model_trace_api_spans_bad_request.go
 model_trace_api_trace_not_found.go
 model_trace_filter.go
-model_trace_identifier.go
 model_trace_query.go
-model_traces.go
+model_trace_query_match.go
+model_trace_query_match_all_of.go
+model_trace_query_response.go
 model_unlicensed_subscription.go
 model_unmatched_check_state.go
 model_user_name_mismatch_error.go