fix(appconfig): deployment recreated on every cdk deployment

packages/@aws-cdk/aws-appconfig-alpha/lib/configuration.ts

@@ -203,8 +203,6 @@ abstract class ConfigurationBase extends Construct implements IConfiguration, IE
     });
   }
 
-  protected abstract getDeploymentHash(environment: IEnvironment): string;
-
   /**
    * Adds an extension defined by the action point and event destination
    * and also creates an extension association to the configuration profile.
@@ -312,8 +310,7 @@ abstract class ConfigurationBase extends Construct implements IConfiguration, IE
    * please take a look into https://docs.aws.amazon.com/cdk/api/v2/docs/aws-cdk-lib.aws_appconfig.CfnDeployment.html.
    */
   public deploy(environment: IEnvironment) {
-    const logicalId = `Deployment${this.getDeploymentHash(environment)}`;
-    new CfnDeployment(this, logicalId, {
+    new CfnDeployment(this, `Deployment${getHash(environment.name!)}`, {
       applicationId: this.application.applicationId,
       configurationProfileId: this.configurationProfileId,
       deploymentStrategyId: this.deploymentStrategy!.deploymentStrategyId,
@@ -485,16 +482,6 @@ export class HostedConfiguration extends ConfigurationBase {
     this.addExistingEnvironmentsToApplication();
     this.deployConfigToEnvironments();
   }
-
-  protected getDeploymentHash(environment: IEnvironment): string {
-    const combinedString = `
-      ${this.application!.name!}
-      ${this.name!}
-      ${environment.name!}
-      ${this.content}
-    `;
-    return getHash(combinedString);
-  }
 }
 
 /**
@@ -626,17 +613,6 @@ export class SourcedConfiguration extends ConfigurationBase {
     this.deployConfigToEnvironments();
   }
 
-  protected getDeploymentHash(environment: IEnvironment): string {
-    const combinedString = `
-      ${this.application!.name!}
-      ${this.name!}
-      ${environment.name!}
-      ${this.versionNumber}
-      ${this.location.type}
-    `;
-    return getHash(combinedString);
-  }
-
   private getPolicyForRole(): iam.PolicyDocument {
     const policy = new iam.PolicyStatement({
       effect: iam.Effect.ALLOW,

packages/@aws-cdk/aws-appconfig-alpha/test/integ.configuration.js.snapshot/aws-appconfig-configuration.template.json

@@ -168,7 +168,7 @@
    "UpdateReplacePolicy": "Retain",
    "DeletionPolicy": "Retain"
   },
-  "MyHostedConfigDeploymentE4742E8E64DF4": {
+  "MyHostedConfigDeployment8E5DB1FCC9183": {
    "Type": "AWS::AppConfig::Deployment",
    "Properties": {
     "ApplicationId": {
@@ -213,7 +213,7 @@
    "UpdateReplacePolicy": "Retain",
    "DeletionPolicy": "Retain"
   },
-  "MyHostedConfigFromJsonDeploymentCD82E3049E374": {
+  "MyHostedConfigFromJsonDeployment3AC6ED5A2933B": {
    "Type": "AWS::AppConfig::Deployment",
    "Properties": {
     "ApplicationId": {
@@ -233,7 +233,7 @@
     }
    }
   },
-  "MyHostedConfigFromJsonDeploymentEFECDBC4087F1": {
+  "MyHostedConfigFromJsonDeploymentFA00BC528601D": {
    "Type": "AWS::AppConfig::Deployment",
    "Properties": {
     "ApplicationId": {
@@ -278,7 +278,7 @@
    "UpdateReplacePolicy": "Retain",
    "DeletionPolicy": "Retain"
   },
-  "MyHostedConfigFromYamlDeploymentFE9624CBE5FC6": {
+  "MyHostedConfigFromYamlDeploymentD93A5BDCCAAB2": {
    "Type": "AWS::AppConfig::Deployment",
    "Properties": {
     "ApplicationId": {
@@ -472,7 +472,7 @@
     ]
    }
   },
-  "MyConfigFromParameterDeployment0C84B4B3BAD6B": {
+  "MyConfigFromParameterDeployment28051DE6185BF": {
    "Type": "AWS::AppConfig::Deployment",
    "Properties": {
     "ApplicationId": {
@@ -593,7 +593,7 @@
     }
    }
   },
-  "MyConfigFromDocumentDeployment1520EE7C916D3": {
+  "MyConfigFromDocumentDeployment7A5BA9D7D69C3": {
    "Type": "AWS::AppConfig::Deployment",
    "Properties": {
     "ApplicationId": {