fix(controller): do not set entrypoint when no procfile

Author: jianxiaoguo

rootfs/api/models/app.py

@@ -153,8 +153,11 @@ def _get_entrypoint(self, container_type):
         """
         Return the kubernetes "container command" to be sent off to the scheduler.
         """
-        entrypoint = ['/bin/sh', '-c']
+        entrypoint = []
         release = self.release_set.filter(failed=False).latest()
+        if release is not None and release.build is not None:
+            if release.build.procfile and container_type in release.build.procfile:
+                entrypoint = ['/bin/sh', '-c']
         if self._get_stack(release) == "buildpack":
             if container_type in release.build.procfile:
                 entrypoint = [container_type]

rootfs/api/tests/test_pods.py

@@ -589,7 +589,7 @@ def test_run_command_good(self, mock_requests):
         response = self.client.post(url, body)
         self.assertEqual(response.status_code, 200, response.data)
         app = App.objects.get(id=app_id)
-        self.assertEqual(app._get_entrypoint('cmd'), ['/bin/sh', '-c'])
+        self.assertEqual(app._get_entrypoint('cmd'), [])
 
         # procfile workflow
         build.sha = 'somereallylongsha'