ref(controller): use /bin/sh for better portability

This allows us to run admin commands on micro-images such as busybox.

Author: Matthew Fisher

controller/api/models.py

@@ -476,7 +476,7 @@ def run(self, command):
                                    'to run this command')
         image = self.release.image
         job_id = self._job_id
-        entrypoint = '/bin/bash'
+        entrypoint = '/bin/sh'
         # if this is a procfile-based app, switch the entrypoint to slugrunner's default
         # FIXME: remove slugrunner's hardcoded entrypoint
         if self.release.build.procfile and \

controller/api/tests/test_container.py

@@ -499,12 +499,12 @@ def test_run_command_good(self):
                                      num=1)
         rc, output = c.run('echo hi')
         self.assertEqual(rc, 0)
-        self.assertEqual(json.loads(output)['entrypoint'], '/bin/bash')
+        self.assertEqual(json.loads(output)['entrypoint'], '/bin/sh')
         # docker image workflow
         build.dockerfile = None
         build.sha = None
         rc, output = c.run('echo hi')
-        self.assertEqual(json.loads(output)['entrypoint'], '/bin/bash')
+        self.assertEqual(json.loads(output)['entrypoint'], '/bin/sh')
         # procfile workflow
         build.sha = 'somereallylongsha'
         rc, output = c.run('echo hi')