{"action":"created","issue":{"url":"https://api.github.com/repos/open-craft/ansible-dalite/issues/2","repository_url":"https://api.github.com/repos/open-craft/ansible-dalite","labels_url":"https://api.github.com/repos/open-craft/ansible-dalite/issues/2/labels{/name}","comments_url":"https://api.github.com/repos/open-craft/ansible-dalite/issues/2/comments","events_url":"https://api.github.com/repos/open-craft/ansible-dalite/issues/2/events","html_url":"https://github.com/open-craft/ansible-dalite/pull/2","id":165299184,"number":2,"title":"[WIP] Move log rotation and database dumps to separate volume (OC-1589)","user":{"login":"haikuginger","id":7773758,"avatar_url":"https://avatars.githubusercontent.com/u/7773758?v=3","gravatar_id":"","url":"https://api.github.com/users/haikuginger","html_url":"https://github.com/haikuginger","followers_url":"https://api.github.com/users/haikuginger/followers","following_url":"https://api.github.com/users/haikuginger/following{/other_user}","gists_url":"https://api.github.com/users/haikuginger/gists{/gist_id}","starred_url":"https://api.github.com/users/haikuginger/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/haikuginger/subscriptions","organizations_url":"https://api.github.com/users/haikuginger/orgs","repos_url":"https://api.github.com/users/haikuginger/repos","events_url":"https://api.github.com/users/haikuginger/events{/privacy}","received_events_url":"https://api.github.com/users/haikuginger/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2016-07-13T11:32:50Z","updated_at":"2016-07-13T14:51:24Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/open-craft/ansible-dalite/pulls/2","html_url":"https://github.com/open-craft/ansible-dalite/pull/2","diff_url":"https://github.com/open-craft/ansible-dalite/pull/2.diff","patch_url":"https://github.com/open-craft/ansible-dalite/pull/2.patch"},"body":"Adding plays for formatting and mounting a volume; adding compression to log rotation/database dump script.\r\n\r\n**JIRA tickets**: Implements OC-1589\r\n\r\n**Discussions**: Discussed in JIRA ticket\r\n\r\n**Dependencies**: None\r\n\r\n**Sandbox URL**: TBD - sandbox is being provisioned.\r\n\r\n**Partner information**: 3rd party-hosted open edX instance\r\n\r\n**Testing instructions**:\r\n\r\n1. Verify that currently, logs are being stored on the boot volume\r\n2. Create an OpenStack instance with an additional volume attached at `/dev/vdc`\r\n3. Provision that instance with this playbook\r\n4. Verify that logs and DB dumps are being stored on the additional volume\r\n\r\n**Author notes and concerns**:\r\n\r\n1. Ansible has modules to create and attach OpenStack volumes; I'd like to give those a try once I get access to see if they could be included here.\r\n2. If we don't go that route, I'm guessing there's a better way to pass the volume device ID than I did here; would we just skip defining the variable and have it passed in at runtime?\r\n3. We may want to mount the volume (and thus, the dump location) outside the `logs-download` user's home folder. If that's the case, then I'll add a couple plays to symlink those directories to the `logs-download` user's home folder to preserve continuity.\r\n\r\n**Reviewers**\r\n- [ ] @pomegranited "},"comment":{"url":"https://api.github.com/repos/open-craft/ansible-dalite/issues/comments/232379927","html_url":"https://github.com/open-craft/ansible-dalite/pull/2#issuecomment-232379927","issue_url":"https://api.github.com/repos/open-craft/ansible-dalite/issues/2","id":232379927,"user":{"login":"pomegranited","id":7556571,"avatar_url":"https://avatars.githubusercontent.com/u/7556571?v=3","gravatar_id":"","url":"https://api.github.com/users/pomegranited","html_url":"https://github.com/pomegranited","followers_url":"https://api.github.com/users/pomegranited/followers","following_url":"https://api.github.com/users/pomegranited/following{/other_user}","gists_url":"https://api.github.com/users/pomegranited/gists{/gist_id}","starred_url":"https://api.github.com/users/pomegranited/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pomegranited/subscriptions","organizations_url":"https://api.github.com/users/pomegranited/orgs","repos_url":"https://api.github.com/users/pomegranited/repos","events_url":"https://api.github.com/users/pomegranited/events{/privacy}","received_events_url":"https://api.github.com/users/pomegranited/received_events","type":"User","site_admin":false},"created_at":"2016-07-13T14:51:24Z","updated_at":"2016-07-13T14:51:24Z","body":"@haikuginger Your change is looking good, I've left a couple of comments inline.\r\n\r\nI've made some notes in the JIRA issue too, but could you please add a link in\r\nthe README here to the deployment repo:\r\n\r\nhttps://github.com/open-craft/deployment-deploy-services\r\n\r\nAlso, could you please update the `deployment-deploy-services` and\r\n`ansible-dalite` github repo descriptions to indicate what they do?\r\n"}}
{"actor":{"display_login":"pomegranited"}}