{"action":"created","issue":{"url":"https://api.github.com/repos/openshift/origin/issues/7496","repository_url":"https://api.github.com/repos/openshift/origin","labels_url":"https://api.github.com/repos/openshift/origin/issues/7496/labels{/name}","comments_url":"https://api.github.com/repos/openshift/origin/issues/7496/comments","events_url":"https://api.github.com/repos/openshift/origin/issues/7496/events","html_url":"https://github.com/openshift/origin/issues/7496","id":135111973,"number":7496,"title":"All CLI commands should launch a standard shell ","user":{"login":"smarterclayton","id":1163175,"avatar_url":"https://avatars.githubusercontent.com/u/1163175?v=3","gravatar_id":"","url":"https://api.github.com/users/smarterclayton","html_url":"https://github.com/smarterclayton","followers_url":"https://api.github.com/users/smarterclayton/followers","following_url":"https://api.github.com/users/smarterclayton/following{/other_user}","gists_url":"https://api.github.com/users/smarterclayton/gists{/gist_id}","starred_url":"https://api.github.com/users/smarterclayton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/smarterclayton/subscriptions","organizations_url":"https://api.github.com/users/smarterclayton/orgs","repos_url":"https://api.github.com/users/smarterclayton/repos","events_url":"https://api.github.com/users/smarterclayton/events{/privacy}","received_events_url":"https://api.github.com/users/smarterclayton/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/openshift/origin/labels/component/build","name":"component/build","color":"0052cc"},{"url":"https://api.github.com/repos/openshift/origin/labels/component/cli","name":"component/cli","color":"0052cc"},{"url":"https://api.github.com/repos/openshift/origin/labels/priority/P1","name":"priority/P1","color":"eb6420"}],"state":"open","locked":false,"assignee":{"login":"rhcarvalho","id":88819,"avatar_url":"https://avatars.githubusercontent.com/u/88819?v=3","gravatar_id":"","url":"https://api.github.com/users/rhcarvalho","html_url":"https://github.com/rhcarvalho","followers_url":"https://api.github.com/users/rhcarvalho/followers","following_url":"https://api.github.com/users/rhcarvalho/following{/other_user}","gists_url":"https://api.github.com/users/rhcarvalho/gists{/gist_id}","starred_url":"https://api.github.com/users/rhcarvalho/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rhcarvalho/subscriptions","organizations_url":"https://api.github.com/users/rhcarvalho/orgs","repos_url":"https://api.github.com/users/rhcarvalho/repos","events_url":"https://api.github.com/users/rhcarvalho/events{/privacy}","received_events_url":"https://api.github.com/users/rhcarvalho/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/openshift/origin/milestones/11","html_url":"https://github.com/openshift/origin/milestones/1.2.0","labels_url":"https://api.github.com/repos/openshift/origin/milestones/11/labels","id":1502487,"number":11,"title":"1.2.0","description":"","creator":{"login":"smarterclayton","id":1163175,"avatar_url":"https://avatars.githubusercontent.com/u/1163175?v=3","gravatar_id":"","url":"https://api.github.com/users/smarterclayton","html_url":"https://github.com/smarterclayton","followers_url":"https://api.github.com/users/smarterclayton/followers","following_url":"https://api.github.com/users/smarterclayton/following{/other_user}","gists_url":"https://api.github.com/users/smarterclayton/gists{/gist_id}","starred_url":"https://api.github.com/users/smarterclayton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/smarterclayton/subscriptions","organizations_url":"https://api.github.com/users/smarterclayton/orgs","repos_url":"https://api.github.com/users/smarterclayton/repos","events_url":"https://api.github.com/users/smarterclayton/events{/privacy}","received_events_url":"https://api.github.com/users/smarterclayton/received_events","type":"User","site_admin":false},"open_issues":30,"closed_issues":29,"state":"open","created_at":"2016-01-12T16:06:32Z","updated_at":"2016-02-22T10:13:49Z","due_on":"2016-03-17T04:00:00Z","closed_at":null},"comments":8,"created_at":"2016-02-20T19:22:28Z","updated_at":"2016-02-22T14:07:14Z","closed_at":null,"body":"We recently identified an issue with the default shell being chosen for the new post-commit build hook.  It is using `/bin/sh`.  `oc rsh` currently uses `/bin/bash`, and the new `oc debug` command is using `/bin/sh`.\r\n\r\nWhen we invoke a default shell in an image, we should be using a consistent value.  Currently, RedHat and Ubuntu based images symlink `/bin/sh` to `/bin/bash` or to dash.  Alpine and Busybox images lack `/bin/bash` altogether.  Therefore, `/bin/bash` is not an appropriate default for `oc rsh` in the larger Docker ecosystem.\r\n\r\nHowever, the SCL images currently lack the ability to invoke `/bin/sh` and enter the correct environment for SCL.  Since the SCL images exist only to provide an environment for a language, it is a requirement that shell access to an SCL container be able to a) drop the user into an environment where the correct SCL environment is setup when `/bin/bash` is invoked, and b) do the same thing for `/bin/sh`.\r\n\r\nWe need to \r\n\r\n1. Identify any other compatibility issues with the default of `/bin/sh` for all containers\r\n2. Fix the SCL images to setup the correct environment when `/bin/sh` is invoked.\r\n3. Fix `oc rsh` to use `/bin/sh`."},"comment":{"url":"https://api.github.com/repos/openshift/origin/issues/comments/187188738","html_url":"https://github.com/openshift/origin/issues/7496#issuecomment-187188738","issue_url":"https://api.github.com/repos/openshift/origin/issues/7496","id":187188738,"user":{"login":"smarterclayton","id":1163175,"avatar_url":"https://avatars.githubusercontent.com/u/1163175?v=3","gravatar_id":"","url":"https://api.github.com/users/smarterclayton","html_url":"https://github.com/smarterclayton","followers_url":"https://api.github.com/users/smarterclayton/followers","following_url":"https://api.github.com/users/smarterclayton/following{/other_user}","gists_url":"https://api.github.com/users/smarterclayton/gists{/gist_id}","starred_url":"https://api.github.com/users/smarterclayton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/smarterclayton/subscriptions","organizations_url":"https://api.github.com/users/smarterclayton/orgs","repos_url":"https://api.github.com/users/smarterclayton/repos","events_url":"https://api.github.com/users/smarterclayton/events{/privacy}","received_events_url":"https://api.github.com/users/smarterclayton/received_events","type":"User","site_admin":false},"created_at":"2016-02-22T14:07:14Z","updated_at":"2016-02-22T14:07:14Z","body":"Could we do a small binary stub that did the correct setup (instead of\na shell script)?  Ie checked incoming env and exec either sh or bash\nbased on arguments, and only do something if we are in the /bin/sh\nposition?  This is something the SCL team could own.\n"}}