{"action":"created","issue":{"url":"https://api.github.com/repos/tobegit3hub/seagull/issues/62","repository_url":"https://api.github.com/repos/tobegit3hub/seagull","labels_url":"https://api.github.com/repos/tobegit3hub/seagull/issues/62/labels{/name}","comments_url":"https://api.github.com/repos/tobegit3hub/seagull/issues/62/comments","events_url":"https://api.github.com/repos/tobegit3hub/seagull/issues/62/events","html_url":"https://github.com/tobegit3hub/seagull/pull/62","id":133578829,"number":62,"title":"Smaller container (fixes #56)","user":{"login":"dlsniper","id":607868,"avatar_url":"https://avatars.githubusercontent.com/u/607868?v=3","gravatar_id":"","url":"https://api.github.com/users/dlsniper","html_url":"https://github.com/dlsniper","followers_url":"https://api.github.com/users/dlsniper/followers","following_url":"https://api.github.com/users/dlsniper/following{/other_user}","gists_url":"https://api.github.com/users/dlsniper/gists{/gist_id}","starred_url":"https://api.github.com/users/dlsniper/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dlsniper/subscriptions","organizations_url":"https://api.github.com/users/dlsniper/orgs","repos_url":"https://api.github.com/users/dlsniper/repos","events_url":"https://api.github.com/users/dlsniper/events{/privacy}","received_events_url":"https://api.github.com/users/dlsniper/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/tobegit3hub/seagull/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/tobegit3hub/seagull/milestones/9","html_url":"https://github.com/tobegit3hub/seagull/milestones/1.7.0","labels_url":"https://api.github.com/repos/tobegit3hub/seagull/milestones/9/labels","id":1585901,"number":9,"title":"1.7.0","description":null,"creator":{"login":"tobegit3hub","id":2715000,"avatar_url":"https://avatars.githubusercontent.com/u/2715000?v=3","gravatar_id":"","url":"https://api.github.com/users/tobegit3hub","html_url":"https://github.com/tobegit3hub","followers_url":"https://api.github.com/users/tobegit3hub/followers","following_url":"https://api.github.com/users/tobegit3hub/following{/other_user}","gists_url":"https://api.github.com/users/tobegit3hub/gists{/gist_id}","starred_url":"https://api.github.com/users/tobegit3hub/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tobegit3hub/subscriptions","organizations_url":"https://api.github.com/users/tobegit3hub/orgs","repos_url":"https://api.github.com/users/tobegit3hub/repos","events_url":"https://api.github.com/users/tobegit3hub/events{/privacy}","received_events_url":"https://api.github.com/users/tobegit3hub/received_events","type":"User","site_admin":false},"open_issues":1,"closed_issues":1,"state":"open","created_at":"2016-02-16T15:07:04Z","updated_at":"2016-02-16T15:54:04Z","due_on":null,"closed_at":null},"comments":8,"created_at":"2016-02-14T20:54:28Z","updated_at":"2016-02-17T00:26:07Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/tobegit3hub/seagull/pulls/62","html_url":"https://github.com/tobegit3hub/seagull/pull/62","diff_url":"https://github.com/tobegit3hub/seagull/pull/62.diff","patch_url":"https://github.com/tobegit3hub/seagull/pull/62.patch"},"body":"Hi,\r\n\r\nThank you for the great project.\r\n\r\nThis PR does the following two things:\r\n- updates the dependencies and adds the missing ones (needed for step 2)\r\n- improves the testing and release process by making the containers as small as possible\r\n\r\n*Please note*: This disables CGO and forces the application to use the Go dns resolver not the system one (which should be ok). Also, the debug information is stripped away from the binary so that the size is reduced by ~33%.\r\n\r\nWith the changes I've made, the new container drops from ~560mb to ~18mb, see the screenshot below: \r\n![snapshot221](https://cloud.githubusercontent.com/assets/607868/13036327/b530b644-d364-11e5-962e-5715f2b57531.png)\r\n\r\nThe new workflow would be like this:\r\n- run tests ` /container-test.sh `\r\n- build the container ` ./container-make.sh <container-name> <container-tag> ` (for example: ` ./container-make.sh dlsniper/seagull 13 `\r\n- release the container ` ./container-release <container-name> <container-tag> ` (for example ` ./container-make.sh dlsniper/seagull 13`\r\n\r\n*Please note*: the release process is not finished as I don't know what the docker tag and push commands you are currently using / maybe there are some other env variables / docker login action you have to do first. Please change this after merging.\r\n\r\nLooking forward for comments / getting this merged.\r\n\r\nKind regards.\r\n\r\n(fixes #56)"},"comment":{"url":"https://api.github.com/repos/tobegit3hub/seagull/issues/comments/184939528","html_url":"https://github.com/tobegit3hub/seagull/pull/62#issuecomment-184939528","issue_url":"https://api.github.com/repos/tobegit3hub/seagull/issues/62","id":184939528,"user":{"login":"SydOps","id":8954908,"avatar_url":"https://avatars.githubusercontent.com/u/8954908?v=3","gravatar_id":"","url":"https://api.github.com/users/SydOps","html_url":"https://github.com/SydOps","followers_url":"https://api.github.com/users/SydOps/followers","following_url":"https://api.github.com/users/SydOps/following{/other_user}","gists_url":"https://api.github.com/users/SydOps/gists{/gist_id}","starred_url":"https://api.github.com/users/SydOps/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SydOps/subscriptions","organizations_url":"https://api.github.com/users/SydOps/orgs","repos_url":"https://api.github.com/users/SydOps/repos","events_url":"https://api.github.com/users/SydOps/events{/privacy}","received_events_url":"https://api.github.com/users/SydOps/received_events","type":"User","site_admin":false},"created_at":"2016-02-17T00:26:07Z","updated_at":"2016-02-17T00:26:07Z","body":"@tobegit3hub \r\n\r\nSame as I worked on Alpine image half year before, seems you have same attitude on @dlsniper 's idea again. Not easily accept new idea. \r\n\r\nNow docker.io itself wants to move the base images to alpine, you accept that alpine can be the base image for Seagull as well. But why not go further again?\r\n\r\nI vote up for @dlsniper's idea, some users like mini seagull image. But maybe not do that in master branch directly. So what can we do?\r\n\r\n1) I raise a pull request to build seagull on Alpine to master branch, it will replace currently base image `Dockerfile` with alpine base image. Of course you need test the image that no functions are lost.\r\n\r\n2) @dlsniper can raise a pull request and merge to another branch, maybe it can be called `mini` or something else. @tobegit3hub you need create that branch on your repository first.\r\n\r\n3) In hub.docker.com automation build, we can build image on different branches directly, so finally you should have several images in one account.\r\n\r\n ```\r\ntobegit3hub/seagull:latest (same as alpine-3.3)\r\ntobegit3hub/seagull:alpine-3.3 (size is about 170MB)\r\ntobegit3hub/seagull:mini  (size is about 20MB)\r\n...\r\n```\r\n\r\nLet you me if you are fine with this or not. "}}