exclude tags and reservedConcurrency from version hash

lib/plugins/aws/package/compile/functions/index.js

@@ -494,7 +494,13 @@ class AwsCompileFunctions {
             });
         }).then(() => {
           // Include function configuration in version id hash (without the Code part)
-          const properties = _.omit(_.get(functionResource, 'Properties', {}), 'Code');
+          const properties = _.omit(
+            _.get(functionResource, 'Properties', {}),
+            'Code',
+            // Properties applied to function globally (not specific to version or alias)
+            'ReservedConcurrentExecutions',
+            'Tags'
+          );
           _.forOwn(properties, value => {
             if (_.isObject(value)) {
               versionHash.write(JSON.stringify(value));

lib/plugins/aws/package/compile/functions/index.test.js

@@ -10,6 +10,7 @@ const AwsProvider = require('../../../provider/awsProvider');
 const AwsCompileFunctions = require('./index');
 const Serverless = require('../../../../../Serverless');
 const runServerless = require('../../../../../../test/utils/run-serverless');
+const fixtures = require('../../../../../../test/fixtures');
 
 const { getTmpDirPath, createTmpFile } = require('../../../../../../test/utils/fs');
 
@@ -2831,4 +2832,40 @@ describe('AwsCompileFunctions #2', () => {
       });
     });
   });
+
+  describe('function config', () => {
+    it('should not create a new version object if only function-wide configuration changed', () => {
+      return fixtures.setup('function').then(({ servicePath, updateConfig }) =>
+        runServerless({
+          cwd: servicePath,
+          cliArgs: ['package'],
+        }).then(({ cfTemplate: originalTemplate }) => {
+          const originalVersionArn =
+            originalTemplate.Outputs.FooLambdaFunctionQualifiedArn.Value.Ref;
+          return updateConfig({
+            functions: {
+              foo: {
+                tags: {
+                  foo: 'bar',
+                },
+                reservedConcurrency: 1,
+              },
+            },
+          }).then(() =>
+            runServerless({
+              cwd: servicePath,
+              cliArgs: ['package'],
+            }).then(({ cfTemplate: updatedTemplate }) => {
+              expect(
+                updatedTemplate.Resources.FooLambdaFunction.Properties.ReservedConcurrentExecutions
+              ).to.equal(1);
+              const updatedVersionArn =
+                updatedTemplate.Outputs.FooLambdaFunctionQualifiedArn.Value.Ref;
+              expect(originalVersionArn).to.equal(updatedVersionArn);
+            })
+          );
+        })
+      );
+    });
+  });
 });