{"action":"created","issue":{"url":"https://api.github.com/repos/open-craft/opencraft/issues/1","labels_url":"https://api.github.com/repos/open-craft/opencraft/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/open-craft/opencraft/issues/1/comments","events_url":"https://api.github.com/repos/open-craft/opencraft/issues/1/events","html_url":"https://github.com/open-craft/opencraft/pull/1","id":99922395,"number":1,"title":"OpenCraft MVP - Fake PR for review","user":{"login":"antoviaque","id":514483,"avatar_url":"https://avatars.githubusercontent.com/u/514483?v=3","gravatar_id":"","url":"https://api.github.com/users/antoviaque","html_url":"https://github.com/antoviaque","followers_url":"https://api.github.com/users/antoviaque/followers","following_url":"https://api.github.com/users/antoviaque/following{/other_user}","gists_url":"https://api.github.com/users/antoviaque/gists{/gist_id}","starred_url":"https://api.github.com/users/antoviaque/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/antoviaque/subscriptions","organizations_url":"https://api.github.com/users/antoviaque/orgs","repos_url":"https://api.github.com/users/antoviaque/repos","events_url":"https://api.github.com/users/antoviaque/events{/privacy}","received_events_url":"https://api.github.com/users/antoviaque/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":9,"created_at":"2015-08-09T19:44:02Z","updated_at":"2015-08-20T07:04:55Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/open-craft/opencraft/pulls/1","html_url":"https://github.com/open-craft/opencraft/pull/1","diff_url":"https://github.com/open-craft/opencraft/pull/1.diff","patch_url":"https://github.com/open-craft/opencraft/pull/1.patch"},"body":"See http://console.opencraft.com/ for a working example, and the README for installation instructions.\r\n\r\n### Being discussed\r\n\r\nReviewers input needed:\r\n\r\n- [ ] [Determine list of XBlocks to load by default](https://github.com/open-craft/opencraft/pull/1#discussion_r36607383)\r\n- [ ] Pylint configuration\r\n  - [ ] [Which warnings to ignore](https://github.com/open-craft/opencraft/pull/1#discussion-diff-36737650)\r\n  - [ ] [Max locals](https://github.com/open-craft/opencraft/pull/1#discussion-diff-36738319) & [Max branches](https://github.com/open-craft/opencraft/pull/1#discussion-diff-36738464)\r\n- [ ] [Better status transition management - state machine?](https://github.com/open-craft/opencraft/pull/1#discussion_r36733250)\r\n- [ ] [Better approach for reboot detection](https://github.com/open-craft/opencraft/pull/1#discussion_r36733404)\r\n\r\n### Todo\r\n\r\n- [X] Small nits\r\n- [X] [Clearing temporary files & directories](https://github.com/open-craft/opencraft/pull/1#discussion-diff-36727282)\r\n- [X] Use constants for instance & server model status\r\n- [X] Web UI loading indicator\r\n- [ ] Integration tests\r\n  - [X] Add an integration test that fully instantiates a sandbox\r\n  - [X] Setup CircleCI to run integration tests\r\n  - [ ] Make CircleCI builds reliable (they currently timeout)\r\n  - [ ] [Get the build to fail when the ansible run fails (might also require error states for VMs & retries)](https://github.com/open-craft/opencraft/pull/1#discussion_r36624887)\r\n- [ ] [Switch to OpenStack CLI instead of class](https://github.com/open-craft/opencraft/pull/1#discussion_r36623788)\r\n  - [ ] [Use OS_* variables for OpenStack configuration](https://github.com/open-craft/opencraft/pull/1#discussion-diff-36625380)\r\n  - [ ] [Use vanilla Ubuntu image from upstream](https://github.com/open-craft/opencraft/pull/1#discussion-diff-36625198)\r\n  - [ ] [Implementation-agnostic public IP retreival](https://github.com/open-craft/opencraft/pull/1#discussion_r36624339)\r\n- [ ] [Improve logging management](https://github.com/open-craft/opencraft/pull/1#discussion_r36728597)\r\n  - [ ] [Use custom constants for logging levels](https://github.com/open-craft/opencraft/pull/1#discussion-diff-36729765)\r\n- [ ] Better PR handling - dedicated model?\r\n  - [ ] [Shorter & updated sandbox names](https://github.com/open-craft/opencraft/pull/1#discussion_r36607416)\r\n  - [ ] [Links to PRs in Web UI](https://github.com/open-craft/opencraft/pull/1#discussion_r36713342)\r\n- [ ] OpenCraft logo & \"Powered by\"/licensing footer\r\n- [ ] [Relax regex syntax for configuration variables in PRs](https://github.com/open-craft/opencraft/pull/1#discussion_r37372753)"},"comment":{"url":"https://api.github.com/repos/open-craft/opencraft/issues/comments/132913571","html_url":"https://github.com/open-craft/opencraft/pull/1#issuecomment-132913571","issue_url":"https://api.github.com/repos/open-craft/opencraft/issues/1","id":132913571,"user":{"login":"antoviaque","id":514483,"avatar_url":"https://avatars.githubusercontent.com/u/514483?v=3","gravatar_id":"","url":"https://api.github.com/users/antoviaque","html_url":"https://github.com/antoviaque","followers_url":"https://api.github.com/users/antoviaque/followers","following_url":"https://api.github.com/users/antoviaque/following{/other_user}","gists_url":"https://api.github.com/users/antoviaque/gists{/gist_id}","starred_url":"https://api.github.com/users/antoviaque/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/antoviaque/subscriptions","organizations_url":"https://api.github.com/users/antoviaque/orgs","repos_url":"https://api.github.com/users/antoviaque/repos","events_url":"https://api.github.com/users/antoviaque/events{/privacy}","received_events_url":"https://api.github.com/users/antoviaque/received_events","type":"User","site_admin":false},"created_at":"2015-08-20T07:04:55Z","updated_at":"2015-08-20T07:04:55Z","body":"@bradenmacdonald Thanks for the code review - started addressing your comments above - will keep going through the rest tomorrow."}}