Skip container function with uri defined

lib/utils.js

@@ -114,9 +114,15 @@ function isNodeRuntime(runtime) {
 
 function getAllNodeFunctions() {
   const functions = this.serverless.service.getAllFunctions();
+
   return _.filter(functions, funcName => {
     const func = this.serverless.service.getFunction(funcName);
 
+    // if `uri` is provided, it means the image isn't built by Serverless so we shouldn't take care of it
+    if (func.image && func.image.uri) {
+      return false;
+    }
+
     return isNodeRuntime(func.runtime || this.serverless.service.provider.runtime || 'nodejs');
   });
 }

tests/validate.test.js

@@ -507,7 +507,7 @@ describe('validate', () => {
         },
         dockerfunc: {
           image: {
-            name: 'some-image-uri',
+            name: 'some-docker-image',
             command: ['com.serverless.Handler']
           },
           events: [
@@ -518,6 +518,21 @@ describe('validate', () => {
               }
             }
           ]
+        },
+        dockerfuncuri: {
+          image: {
+            name: 'some-image-with-uri',
+            uri: 'http://hub.dock.er/image',
+            command: ['method.lambda']
+          },
+          events: [
+            {
+              http: {
+                method: 'POST',
+                path: 'mydockerfuncpath'
+              }
+            }
+          ]
         }
       };
 
@@ -693,6 +708,51 @@ describe('validate', () => {
         });
       });
 
+      it('should skip image defined with URI', () => {
+        const testOutPath = 'test';
+        const testFunctionsConfig = {
+          dockerfuncwithuri: {
+            image: {
+              name: 'some-image-with-uri',
+              uri: 'http://hub.dock.er/image',
+              command: ['method.lambda']
+            },
+            events: [
+              {
+                http: {
+                  method: 'POST',
+                  path: 'mydockerfuncpath'
+                }
+              }
+            ]
+          }
+        };
+
+        const testConfig = {
+          entry: 'test',
+          context: 'testcontext',
+          output: {
+            path: testOutPath
+          },
+          getFunction: func => {
+            return testFunctionsConfig[func];
+          }
+        };
+
+        _.set(module.serverless.service, 'custom.webpack.config', testConfig);
+        module.serverless.service.functions = testFunctionsConfig;
+        globSyncStub.callsFake(filename => [_.replace(filename, '*', 'js')]);
+        return expect(module.validate()).to.be.fulfilled.then(() => {
+          const lib = require('../lib/index');
+          const expectedLibEntries = {};
+
+          expect(lib.entries).to.deep.equal(expectedLibEntries);
+          expect(globSyncStub).to.have.callCount(0);
+          expect(serverless.cli.log).to.not.have.been.called;
+          return null;
+        });
+      });
+
       it('should throw error if container image is not well defined', () => {
         const testOutPath = 'test';
         const testFunctionsConfig = {