backport of commit c5018c1

.github/workflows/nightly-test-1.17.x.yaml → .github/workflows/nightly-test-1.13.x.yaml

@@ -1,16 +1,16 @@
 # Copyright (c) HashiCorp, Inc.
 # SPDX-License-Identifier: MPL-2.0
 
-name: Nightly Frontend Test 1.17.x
+name: Nightly Frontend Test 1.13.x
 on:
   schedule:
     - cron: '0 4 * * *'
   workflow_dispatch: {}
 
 env:
   EMBER_PARTITION_TOTAL: 4        # Has to be changed in tandem with the matrix.partition
-  BRANCH: "release/1.17.x"
-  BRANCH_NAME: "release-1.17.x"   # Used for naming artifacts
+  BRANCH: "release/1.13.x"
+  BRANCH_NAME: "release-1.13.x"   # Used for naming artifacts
   GOPRIVATE: github.com/hashicorp # Required for enterprise deps
 
 jobs:

agent/grpc-external/services/resource/delete.go

@@ -7,7 +7,6 @@ import (
 	"context"
 	"errors"
 	"fmt"
-	"strings"
 	"time"
 
 	"github.com/oklog/ulid/v2"
@@ -176,5 +175,5 @@ func (s *Server) validateDeleteRequest(req *pbresource.DeleteRequest) (*resource
 // name by embedding the resources's Uid in the name.
 func tombstoneName(deleteId *pbresource.ID) string {
 	// deleteId.Name is just included for easier identification
-	return fmt.Sprintf("tombstone-%v-%v", deleteId.Name, strings.ToLower(deleteId.Uid))
+	return fmt.Sprintf("tombstone-%v-%v", deleteId.Name, deleteId.Uid)
 }

agent/grpc-external/services/resource/delete_test.go

@@ -5,7 +5,6 @@ package resource
 
 import (
 	"context"
-	"strings"
 	"testing"
 
 	"github.com/stretchr/testify/mock"
@@ -23,98 +22,39 @@ import (
 func TestDelete_InputValidation(t *testing.T) {
 	server := testServer(t)
 	client := testClient(t, server)
-	demo.RegisterTypes(server.Registry)
 
-	type testCase struct {
-		modFn       func(artistId, recordLabelId *pbresource.ID) *pbresource.ID
-		errContains string
-	}
+	demo.RegisterTypes(server.Registry)
 
-	testCases := map[string]testCase{
-		"no id": {
-			modFn: func(_, _ *pbresource.ID) *pbresource.ID {
-				return nil
-			},
-			errContains: "id is required",
-		},
-		"no type": {
-			modFn: func(artistId, _ *pbresource.ID) *pbresource.ID {
-				artistId.Type = nil
-				return artistId
-			},
-			errContains: "id.type is required",
-		},
-		"no name": {
-			modFn: func(artistId, _ *pbresource.ID) *pbresource.ID {
-				artistId.Name = ""
-				return artistId
-			},
-			errContains: "id.name invalid",
-		},
-		"mixed case name": {
-			modFn: func(artistId, _ *pbresource.ID) *pbresource.ID {
-				artistId.Name = "DepecheMode"
-				return artistId
-			},
-			errContains: "id.name invalid",
-		},
-		"name too long": {
-			modFn: func(artistId, _ *pbresource.ID) *pbresource.ID {
-				artistId.Name = strings.Repeat("n", resource.MaxNameLength+1)
-				return artistId
-			},
-			errContains: "id.name invalid",
-		},
-		"partition mixed case": {
-			modFn: func(artistId, _ *pbresource.ID) *pbresource.ID {
-				artistId.Tenancy.Partition = "Default"
-				return artistId
-			},
-			errContains: "id.tenancy.partition invalid",
+	testCases := map[string]func(artistId, recordLabelId *pbresource.ID) *pbresource.ID{
+		"no id": func(artistId, recordLabelId *pbresource.ID) *pbresource.ID {
+			return nil
 		},
-		"partition name too long": {
-			modFn: func(artistId, _ *pbresource.ID) *pbresource.ID {
-				artistId.Tenancy.Partition = strings.Repeat("p", resource.MaxNameLength+1)
-				return artistId
-			},
-			errContains: "id.tenancy.partition invalid",
+		"no type": func(artistId, _ *pbresource.ID) *pbresource.ID {
+			artistId.Type = nil
+			return artistId
 		},
-		"namespace mixed case": {
-			modFn: func(artistId, _ *pbresource.ID) *pbresource.ID {
-				artistId.Tenancy.Namespace = "Default"
-				return artistId
-			},
-			errContains: "id.tenancy.namespace invalid",
+		"no name": func(artistId, _ *pbresource.ID) *pbresource.ID {
+			artistId.Name = ""
+			return artistId
 		},
-		"namespace name too long": {
-			modFn: func(artistId, _ *pbresource.ID) *pbresource.ID {
-				artistId.Tenancy.Namespace = strings.Repeat("n", resource.MaxNameLength+1)
-				return artistId
-			},
-			errContains: "id.tenancy.namespace invalid",
-		},
-		"partition scoped resource with namespace": {
-			modFn: func(_, recordLabelId *pbresource.ID) *pbresource.ID {
-				recordLabelId.Tenancy.Namespace = "ishouldnothaveanamespace"
-				return recordLabelId
-			},
-			errContains: "cannot have a namespace",
+		"partition scoped resource with namespace": func(_, recordLabelId *pbresource.ID) *pbresource.ID {
+			recordLabelId.Tenancy.Namespace = "ishouldnothaveanamespace"
+			return recordLabelId
 		},
 	}
-	for desc, tc := range testCases {
+	for desc, modFn := range testCases {
 		t.Run(desc, func(t *testing.T) {
-			recordLabel, err := demo.GenerateV1RecordLabel("looney-tunes")
+			recordLabel, err := demo.GenerateV1RecordLabel("LoonyTunes")
 			require.NoError(t, err)
 
 			artist, err := demo.GenerateV2Artist()
 			require.NoError(t, err)
 
-			req := &pbresource.DeleteRequest{Id: tc.modFn(artist.Id, recordLabel.Id), Version: ""}
+			req := &pbresource.DeleteRequest{Id: modFn(artist.Id, recordLabel.Id), Version: ""}
 
 			_, err = client.Delete(testContext(t), req)
 			require.Error(t, err)
 			require.Equal(t, codes.InvalidArgument.String(), status.Code(err).String())
-			require.ErrorContains(t, err, tc.errContains)
 		})
 	}
 }
@@ -189,7 +129,7 @@ func TestDelete_Success(t *testing.T) {
 					server, client, ctx := testDeps(t)
 					demo.RegisterTypes(server.Registry)
 
-					recordLabel, err := demo.GenerateV1RecordLabel("looney-tunes")
+					recordLabel, err := demo.GenerateV1RecordLabel("LoonyTunes")
 					require.NoError(t, err)
 					writeRsp, err := client.Write(ctx, &pbresource.WriteRequest{Resource: recordLabel})
 					require.NoError(t, err)

agent/grpc-external/services/resource/list.go

@@ -100,9 +100,8 @@ func (s *Server) validateListRequest(req *pbresource.ListRequest) (*resource.Reg
 		return nil, err
 	}
 
-	if err := validateWildcardTenancy(req.Tenancy, req.NamePrefix); err != nil {
-		return nil, err
-	}
+	// Lowercase
+	resource.Normalize(req.Tenancy)
 
 	// Error when partition scoped and namespace not empty.
 	if reg.Scope == resource.ScopePartition && req.Tenancy.Namespace != "" {

agent/grpc-external/services/resource/list_by_owner.go

@@ -105,6 +105,9 @@ func (s *Server) validateListByOwnerRequest(req *pbresource.ListByOwnerRequest)
 		return nil, err
 	}
 
+	// Lowercase
+	resource.Normalize(req.Owner.Tenancy)
+
 	// Error when partition scoped and namespace not empty.
 	if reg.Scope == resource.ScopePartition && req.Owner.Tenancy.Namespace != "" {
 		return nil, status.Errorf(