fix(dynamoDB): make TableV2 taggable

packages/@aws-cdk-testing/framework-integ/test/aws-dynamodb/test/integ.dynamodb-v2.ondemand.js.snapshot/aws-cdk-global-table.template.json

@@ -124,7 +124,13 @@
        "MaxReadRequestUnits": 222
       },
       "Region": "eu-west-1",
-      "TableClass": "STANDARD_INFREQUENT_ACCESS"
+      "TableClass": "STANDARD_INFREQUENT_ACCESS",
+      "Tags": [
+       {
+        "Key": "primaryTableTagKey",
+        "Value": "primaryTableTagValue"
+       }
+      ]
      },
      {
       "ContributorInsightsSpecification": {

packages/@aws-cdk-testing/framework-integ/test/aws-dynamodb/test/integ.table-v2-global.js.snapshot/aws-cdk-global-table.template.json

@@ -148,6 +148,10 @@
       "Region": "us-east-2",
       "TableClass": "STANDARD_INFREQUENT_ACCESS",
       "Tags": [
+       {
+        "Key": "primaryTableTagKey",
+        "Value": "USE2Replica"
+       },
        {
         "Key": "USE2ReplicaTagKey",
         "Value": "USE2ReplicaTagValue"
@@ -184,6 +188,10 @@
       "Region": "us-west-2",
       "TableClass": "STANDARD",
       "Tags": [
+       {
+        "Key": "primaryTableTagKey",
+        "Value": "primaryTableTagValue"
+       },
        {
         "Key": "USW2ReplicaTagKey",
         "Value": "USW2ReplicaTagValue"

packages/@aws-cdk-testing/framework-integ/test/aws-dynamodb/test/integ.table-v2-global.ts

@@ -52,7 +52,13 @@ class TestStack extends Stack {
               contributorInsights: false,
             },
           },
-          tags: [{ key: 'USE2ReplicaTagKey', value: 'USE2ReplicaTagValue' }],
+          tags: [{
+            key: 'USE2ReplicaTagKey',
+            value: 'USE2ReplicaTagValue',
+          }, {
+            key: 'primaryTableTagKey',
+            value: 'USE2Replica',
+          }],
         },
         {
           region: 'us-west-2',

packages/aws-cdk-lib/aws-dynamodb/lib/table-v2.ts

@@ -4,15 +4,30 @@ import { Capacity } from './capacity';
 import { CfnGlobalTable } from './dynamodb.generated';
 import { TableEncryptionV2 } from './encryption';
 import {
+  Attribute,
+  BillingMode,
+  LocalSecondaryIndexProps,
+  ProjectionType,
+  SecondaryIndexProps,
   StreamViewType,
-  Attribute, TableClass, LocalSecondaryIndexProps,
-  SecondaryIndexProps, BillingMode, ProjectionType,
+  TableClass,
 } from './shared';
-import { TableBaseV2, ITableV2 } from './table-v2-base';
+import { ITableV2, TableBaseV2 } from './table-v2-base';
 import { PolicyDocument } from '../../aws-iam';
 import { IStream } from '../../aws-kinesis';
 import { IKey, Key } from '../../aws-kms';
-import { ArnFormat, CfnTag, FeatureFlags, Lazy, PhysicalName, RemovalPolicy, Stack, Token } from '../../core';
+import {
+  ArnFormat,
+  CfnTag,
+  FeatureFlags,
+  Lazy,
+  PhysicalName,
+  RemovalPolicy,
+  Stack,
+  TagManager,
+  TagType,
+  Token,
+} from '../../core';
 import * as cxapi from '../../cx-api';
 
 const HASH_KEY_TYPE = 'HASH';
@@ -483,6 +498,8 @@ export class TableV2 extends TableBaseV2 {
 
   protected readonly region: string;
 
+  protected readonly tags: TagManager;
+
   private readonly billingMode: string;
   private readonly partitionKey: Attribute;
   private readonly hasSortKey: boolean;
@@ -516,6 +533,7 @@ export class TableV2 extends TableBaseV2 {
     this.partitionKey = props.partitionKey;
     this.hasSortKey = props.sortKey !== undefined;
     this.region = this.stack.region;
+    this.tags = new TagManager(TagType.STANDARD, CfnGlobalTable.CFN_RESOURCE_TYPE_NAME, props.tags);
 
     this.encryption = props.encryption;
     this.encryptionKey = this.encryption?.tableKey;
@@ -665,6 +683,7 @@ export class TableV2 extends TableBaseV2 {
   private configureReplicaTable(props: ReplicaTableProps): CfnGlobalTable.ReplicaSpecificationProperty {
     const pointInTimeRecovery = props.pointInTimeRecovery ?? this.tableOptions.pointInTimeRecovery;
     const contributorInsights = props.contributorInsights ?? this.tableOptions.contributorInsights;
+
     /*
     * Feature flag set as the following may be a breaking change.
     * @see https://github.com/aws/aws-cdk/pull/31097
@@ -674,6 +693,15 @@ export class TableV2 extends TableBaseV2 {
       ? (props.region === this.region ? this.tableOptions.resourcePolicy : props.resourcePolicy) || undefined
       : props.resourcePolicy ?? this.tableOptions.resourcePolicy;
 
+    const propTags: Record<string, string> = (props.tags ?? []).reduce((p, item) =>
+      ({ ...p, [item.key]: item.value }), {},
+    );
+
+    const tags: CfnTag[] = Object.entries({
+      ...this.tags.tagValues(),
+      ...propTags,
+    }).map(([k, v]) => ({ key: k, value: v }));
+
     return {
       region: props.region,
       globalSecondaryIndexes: this.configureReplicaGlobalSecondaryIndexes(props.globalSecondaryIndexOptions),
@@ -692,7 +720,7 @@ export class TableV2 extends TableBaseV2 {
       readProvisionedThroughputSettings: props.readCapacity
         ? props.readCapacity._renderReadCapacity()
         : this.readProvisioning,
-      tags: props.tags,
+      tags: tags.length === 0 ? undefined : tags,
       readOnDemandThroughputSettings: props.maxReadRequestUnits
         ? { maxReadRequestUnits: props.maxReadRequestUnits }
         : this.maxReadRequestUnits