{"action":"created","issue":{"url":"https://api.github.com/repos/apache/zeppelin/issues/1538","repository_url":"https://api.github.com/repos/apache/zeppelin","labels_url":"https://api.github.com/repos/apache/zeppelin/issues/1538/labels{/name}","comments_url":"https://api.github.com/repos/apache/zeppelin/issues/1538/comments","events_url":"https://api.github.com/repos/apache/zeppelin/issues/1538/events","html_url":"https://github.com/apache/zeppelin/pull/1538","id":183684229,"number":1538,"title":"[ZEPPELIN-1386] Docker images for running Apache Zeppelin releases","user":{"login":"mfelgamal","id":10679887,"avatar_url":"https://avatars.githubusercontent.com/u/10679887?v=3","gravatar_id":"","url":"https://api.github.com/users/mfelgamal","html_url":"https://github.com/mfelgamal","followers_url":"https://api.github.com/users/mfelgamal/followers","following_url":"https://api.github.com/users/mfelgamal/following{/other_user}","gists_url":"https://api.github.com/users/mfelgamal/gists{/gist_id}","starred_url":"https://api.github.com/users/mfelgamal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mfelgamal/subscriptions","organizations_url":"https://api.github.com/users/mfelgamal/orgs","repos_url":"https://api.github.com/users/mfelgamal/repos","events_url":"https://api.github.com/users/mfelgamal/events{/privacy}","received_events_url":"https://api.github.com/users/mfelgamal/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":36,"created_at":"2016-10-18T13:20:32Z","updated_at":"2016-12-07T00:41:48Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/zeppelin/pulls/1538","html_url":"https://github.com/apache/zeppelin/pull/1538","diff_url":"https://github.com/apache/zeppelin/pull/1538.diff","patch_url":"https://github.com/apache/zeppelin/pull/1538.patch"},"body":"### What is this PR for?\n\nThis PR is for making docker images for zeppelin releases. It contains a script for building image for each release. Another script is used for publishing images to zeppelin Dockerhub account.\n\nThis repo, https://github.com/mfelgamal/zeppelin-dockers, is a demonstration of this PR. It contains zeppelin-base image and an image for each zeppelin release.\n### What type of PR is it?\n\n[Feature]\n### Todos\n- Review Comments\n- Documentation\n### What is the Jira issue?\n\nhttps://issues.apache.org/jira/browse/ZEPPELIN-1386\n### How should this be tested?\n- run create_release script or publish_release script.\n### Screenshots (if appropriate)\n### Questions:\n- Does the licenses files need update? no\n- Is there breaking changes for older versions? no\n- Does this needs documentation? yes\n"},"comment":{"url":"https://api.github.com/repos/apache/zeppelin/issues/comments/265320253","html_url":"https://github.com/apache/zeppelin/pull/1538#issuecomment-265320253","issue_url":"https://api.github.com/repos/apache/zeppelin/issues/1538","id":265320253,"user":{"login":"bzz","id":5582506,"avatar_url":"https://avatars.githubusercontent.com/u/5582506?v=3","gravatar_id":"","url":"https://api.github.com/users/bzz","html_url":"https://github.com/bzz","followers_url":"https://api.github.com/users/bzz/followers","following_url":"https://api.github.com/users/bzz/following{/other_user}","gists_url":"https://api.github.com/users/bzz/gists{/gist_id}","starred_url":"https://api.github.com/users/bzz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bzz/subscriptions","organizations_url":"https://api.github.com/users/bzz/orgs","repos_url":"https://api.github.com/users/bzz/repos","events_url":"https://api.github.com/users/bzz/events{/privacy}","received_events_url":"https://api.github.com/users/bzz/received_events","type":"User","site_admin":false},"created_at":"2016-12-07T00:41:48Z","updated_at":"2016-12-07T00:41:48Z","body":"@mfelgamal could you please explain, why do you think one more script is needed - `start-zeppelin.sh` and why `CMD[\"/bin/zeppelin.sh\"]` can not be used as entrypoint for the image?\r\n\r\n@1ambda has the point, and we should try to reduce the number of shell scripts that need to be supported later on, as well as possible issues with setting up classpath, etc.\r\n"}}
{"actor":{"display_login":"bzz"}}