fix: support add alisa and publish function version

brycewwang · brycewwang · commit c1d7068d9f7d · 2024-11-18T22:42:46.000+08:00
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "tencent-component-toolkit",
-  "version": "2.27.0",
+  "version": "2.27.1",
   "description": "Tencent component toolkit",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",
diff --git a/src/modules/scf/config.ts b/src/modules/scf/config.ts
@@ -6,6 +6,7 @@ const CONFIGS = {
   defaultInitTimeout: 3,
   waitStatus: ['Creating', 'Updating', 'Publishing', 'Deleting'],
   failStatus: ['CreateFailed	', 'UpdateFailed', 'PublishFailed', 'DeleteFailed'],
+  defaultDiskSize: 512,
 };
 
 export default CONFIGS;
diff --git a/src/modules/scf/constants.ts b/src/modules/scf/constants.ts
@@ -1 +1,12 @@
 export const WebServerImageDefaultPort = 9000;
+
+export enum VersionOperation {
+  Publish = 'publish',
+  Delete = 'delete',
+}
+
+export enum AliasOperation {
+  Create = 'create',
+  Update = 'update',
+  Delete = 'delete',
+}
diff --git a/src/modules/scf/entities/alias.ts b/src/modules/scf/entities/alias.ts
@@ -20,14 +20,17 @@ export default class AliasEntity extends BaseEntity {
       Name: inputs.aliasName,
       Namespace: inputs.namespace || 'default',
       Description: inputs.description || 'Published by Serverless Component',
+      RoutingConfig: {
+        AdditionalVersionWeights: inputs.additionalVersions
+          ? inputs.additionalVersions?.map((v) => {
+              return {
+                Version: v.version,
+                Weight: v.weight,
+              };
+            })
+          : [],
+      },
     };
-    if (inputs.lastVersion && inputs.traffic) {
-      publishInputs.RoutingConfig = {
-        AdditionalVersionWeights: [
-          { Version: inputs.lastVersion, Weight: strip(1 - inputs.traffic) },
-        ],
-      };
-    }
     const Response = await this.request(publishInputs);
     return Response;
   }
@@ -43,12 +46,14 @@ export default class AliasEntity extends BaseEntity {
       Name: inputs.aliasName || '$DEFAULT',
       Namespace: inputs.namespace || 'default',
       RoutingConfig: {
-        AdditionalVersionWeights: inputs.additionalVersions?.map((v) => {
-          return {
-            Version: v.version,
-            Weight: v.weight,
-          };
-        }),
+        AdditionalVersionWeights: inputs.additionalVersions
+          ? inputs.additionalVersions?.map((v) => {
+              return {
+                Version: v.version,
+                Weight: v.weight,
+              };
+            })
+          : [],
       },
       Description: inputs.description || 'Configured by Serverless Component',
     };
diff --git a/src/modules/scf/index.ts b/src/modules/scf/index.ts
@@ -280,6 +280,28 @@ export default class Scf {
     const functionName = inputs.name;
     const { ignoreTriggers = false } = inputs;
 
+    if (inputs?.aliasName) {
+      if (!inputs?.additionalVersionWeights) {
+        throw new ApiTypeError(
+          'PARAMETER_SCF',
+          'additionalVersionWeights is required when aliasName is setted',
+        );
+      }
+      if (!inputs?.aliasFunctionVersion) {
+        throw new ApiTypeError(
+          'PARAMETER_SCF',
+          'aliasFunctionVersion is required when aliasName is setted',
+        );
+      }
+    } else {
+      if (inputs?.additionalVersionWeights || inputs?.aliasFunctionVersion) {
+        throw new ApiTypeError(
+          'PARAMETER_SCF',
+          'aliasName is required when additionalVersionWeights or aliasFunctionVersion is setted',
+        );
+      }
+    }
+
     // 在部署前，检查函数初始状态，如果初始为 CreateFailed，尝试先删除，再重新创建
     let funcInfo = await this.scf.getInitialStatus({ namespace, functionName });
 
@@ -321,13 +343,10 @@ export default class Scf {
       });
     }
 
-    const aliasAddionalVersion = inputs.aliasAddionalVersion || inputs.lastVersion;
-    const needSetTraffic =
-      inputs.traffic != null && aliasAddionalVersion && aliasAddionalVersion !== '$LATEST';
-    const needSetAlias = (inputs.aliasName && inputs.aliasName !== '$DEFAULT') || needSetTraffic;
-    if (needSetAlias) {
-      let needCreateAlias = false;
-      if (inputs.aliasName && inputs.aliasName !== '$DEFAULT') {
+    // 检测配置的别名是否存在，不存在就创建，存在的话就设置流量
+    let needCreateAlias = false;
+    if (inputs.aliasName) {
+      if (inputs.aliasName !== '$DEFAULT') {
         try {
           const aliasInfo = await this.alias.get({
             namespace,
@@ -347,32 +366,33 @@ export default class Scf {
           }
         }
       }
-      if (needCreateAlias) {
-        await this.alias.create({
-          namespace,
-          functionName,
-          functionVersion: inputs.aliasFunctionVersion || funcInfo?.Qualifier,
-          aliasName: inputs.aliasName!,
-          lastVersion: aliasAddionalVersion!,
-          traffic: inputs.traffic!,
-          description: inputs.aliasDescription,
-        });
-      } else {
-        await this.alias.update({
-          namespace,
-          functionName,
-          functionVersion: inputs.aliasFunctionVersion || funcInfo?.Qualifier,
-          additionalVersions: needSetTraffic
-            ? [{ weight: strip(1 - inputs.traffic!), version: aliasAddionalVersion! }]
-            : [],
-          region: this.region,
-          aliasName: inputs.aliasName,
-          description: inputs.aliasDescription,
-        });
+      try {
+        // 创建别名
+        if (needCreateAlias) {
+          await this.alias.create({
+            namespace,
+            functionName,
+            functionVersion: inputs.aliasFunctionVersion || funcInfo?.Qualifier,
+            aliasName: inputs.aliasName!,
+            description: inputs.aliasDescription,
+            additionalVersions: inputs.additionalVersionWeights,
+          });
+        } else {
+          // 更新别名
+          await this.alias.update({
+            namespace,
+            functionName,
+            functionVersion: inputs.aliasFunctionVersion || funcInfo?.Qualifier,
+            additionalVersions: inputs.additionalVersionWeights,
+            region: this.region,
+            aliasName: inputs.aliasName,
+            description: inputs.aliasDescription,
+          });
+        }
+      } catch (error) {
+        const errorType = needCreateAlias ? 'CREATE_ALIAS_SCF' : 'UPDATE_ALIAS_SCF';
+        throw new ApiTypeError(errorType, error.message);
       }
-
-      outputs.Traffic = inputs.traffic;
-      outputs.ConfigTrafficVersion = inputs.lastVersion;
     }
 
     // get default alias
diff --git a/src/modules/scf/interface.ts b/src/modules/scf/interface.ts
@@ -35,6 +35,7 @@ export interface BaseFunctionConfig {
   Timeout?: number;
   InitTimeout?: number;
   MemorySize?: number;
+  DiskSize?: number;
   Type?: 'HTTP' | 'Event';
   DeployMode?: 'code' | 'image';
   PublicNetConfig?: {
@@ -69,6 +70,7 @@ export interface BaseFunctionConfig {
   ProtocolParams?: ProtocolParams;
   NodeType?: string;
   NodeSpec?: string;
+  InstanceConcurrencyConfig?: { DynamicEnabled: 'TRUE' | 'FALSE'; MaxConcurrency?: number };
 }
 
 export interface TriggerType {
@@ -145,9 +147,10 @@ export interface ScfCreateAlias {
   functionVersion?: string;
   aliasName: string;
   namespace?: string;
-  lastVersion: string;
-  traffic: number;
+  lastVersion?: string;
+  traffic?: number;
   description?: string;
+  additionalVersions?: { version: string; weight: number }[];
 }
 
 export interface ScfCreateFunctionInputs {
@@ -167,6 +170,7 @@ export interface ScfCreateFunctionInputs {
   timeout?: number;
   initTimeout?: number;
   memorySize?: number;
+  diskSize?: number;
   publicAccess?: boolean;
   eip?: boolean;
   l5Enable?: boolean;
@@ -245,6 +249,13 @@ export interface ScfCreateFunctionInputs {
 
   protocolType?: string;
   protocolParams?: ProtocolParams;
+
+  // 请求多并发配置
+  instanceConcurrencyConfig?: {
+    enable: boolean; // 是否开启多并发
+    dynamicEnabled: boolean; // 是否开启动态配置
+    maxConcurrency: number; // 最大并发数
+  };
 }
 
 export interface ScfUpdateAliasTrafficInputs {
@@ -270,17 +281,18 @@ export interface ScfDeployInputs extends ScfCreateFunctionInputs {
   enableRoleAuth?: boolean;
   region?: string;
 
+  // 版本相关配置
   lastVersion?: string;
   publish?: boolean;
   publishDescription?: string;
-
   needSetTraffic?: boolean;
   traffic?: number;
 
+  // 别名相关配置
   aliasName?: string;
   aliasDescription?: string;
   aliasFunctionVersion?: string;
-  aliasAddionalVersion?: string;
+  additionalVersionWeights?: { version: string; weight: number }[];
 
   tags?: Record<string, string>;
 
diff --git a/src/modules/scf/utils.ts b/src/modules/scf/utils.ts
@@ -21,6 +21,7 @@ export const formatInputs = (inputs: ScfCreateFunctionInputs) => {
     },
     L5Enable: inputs.l5Enable === true ? 'TRUE' : 'FALSE',
     InstallDependency: inputs.installDependency === true ? 'TRUE' : 'FALSE',
+    DiskSize: +(inputs.diskSize || CONFIGS.defaultDiskSize),
   };
 
   if (inputs.nodeType) {
@@ -96,6 +97,23 @@ export const formatInputs = (inputs: ScfCreateFunctionInputs) => {
         functionInputs.ProtocolParams = protocolParams;
       }
     }
+    // 仅web函数支持单实例请求多并发,instanceConcurrencyConfig.enable:true,启用多并发；instanceConcurrencyConfig.enable:false,关闭多并发
+    if (inputs.instanceConcurrencyConfig) {
+      if (inputs.instanceConcurrencyConfig.enable) {
+        functionInputs.InstanceConcurrencyConfig = {
+          DynamicEnabled: inputs.instanceConcurrencyConfig.dynamicEnabled ? 'TRUE' : 'FALSE',
+          MaxConcurrency: inputs.instanceConcurrencyConfig.maxConcurrency || 2,
+        };
+      } else {
+        functionInputs.InstanceConcurrencyConfig = {
+          DynamicEnabled: '' as any,
+        };
+      }
+      console.log(
+        'functionInputs.instanceConcurrencyConfig:',
+        functionInputs.InstanceConcurrencyConfig,
+      );
+    }
   }
 
   if (inputs.role) {
diff --git a/src/utils/index.ts b/src/utils/index.ts
@@ -31,6 +31,22 @@ export function isArray<T>(obj: T[] | T): obj is T[] {
   return Object.prototype.toString.call(obj) == '[object Array]';
 }
 
+/**
+ * is positive integer(正整数)
+ * @param obj object
+ */
+export function isPositiveInteger(value: string | number) {
+  return +value > 0 && Number.isInteger(+value);
+}
+
+/**
+ * is number（数字)
+ * @param obj object
+ */
+export function isNumber(value: string | number) {
+  return !Number.isNaN(+value);
+}
+
 /**
  * is object
  * @param obj object

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "tencent-component-toolkit",`
`3`		`- "version": "2.27.0",`
	`3`	`+ "version": "2.27.1",`
`4`	`4`	`"description": "Tencent component toolkit",`
`5`	`5`	`"main": "lib/index.js",`
`6`	`6`	`"types": "lib/index.d.ts",`