{"action":"created","issue":{"url":"https://api.github.com/repos/curationexperts/ansible-hydra/issues/148","repository_url":"https://api.github.com/repos/curationexperts/ansible-hydra","labels_url":"https://api.github.com/repos/curationexperts/ansible-hydra/issues/148/labels{/name}","comments_url":"https://api.github.com/repos/curationexperts/ansible-hydra/issues/148/comments","events_url":"https://api.github.com/repos/curationexperts/ansible-hydra/issues/148/events","html_url":"https://github.com/curationexperts/ansible-hydra/issues/148","id":166151325,"number":148,"title":"Redis error on `centos` branch when installing on Ubuntu","user":{"login":"mark-dce","id":3064318,"avatar_url":"https://avatars.githubusercontent.com/u/3064318?v=3","gravatar_id":"","url":"https://api.github.com/users/mark-dce","html_url":"https://github.com/mark-dce","followers_url":"https://api.github.com/users/mark-dce/followers","following_url":"https://api.github.com/users/mark-dce/following{/other_user}","gists_url":"https://api.github.com/users/mark-dce/gists{/gist_id}","starred_url":"https://api.github.com/users/mark-dce/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mark-dce/subscriptions","organizations_url":"https://api.github.com/users/mark-dce/orgs","repos_url":"https://api.github.com/users/mark-dce/repos","events_url":"https://api.github.com/users/mark-dce/events{/privacy}","received_events_url":"https://api.github.com/users/mark-dce/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2016-07-18T17:46:37Z","updated_at":"2016-07-18T21:51:38Z","closed_at":null,"body":"I'm running a local Vagrant build [camper](https://github.com/curationexperts/camper) that should build a development system to run the hydra tutorials.\r\n\r\nWhen I merge in the `centos` branch changes, I get the following ansible errors:\r\n```\r\nTASK [sufia_dependencies/install : redis and resque] ***************************\r\nincluded: /Users/mark/Documents/workspace/_no_backup/deployinghydra/camper/ansible-hydra/roles/sufia_dependencies/install/tasks/redis_resque.yml for default\r\n\r\nTASK [sufia_dependencies/install : enable epel repo for redis on CentOS] *******\r\nskipping: [default]\r\n\r\nTASK [sufia_dependencies/install : install redis for CentOS] *******************\r\n[DEPRECATION WARNING]: Skipping task due to undefined Error, in the future this\r\n will be a fatal error.: 'redis' is undefined.\r\nThis feature will be removed in \r\na future release. Deprecation warnings can be disabled by setting \r\ndeprecation_warnings=False in ansible.cfg.\r\n\r\nTASK [sufia_dependencies/install : run redis] **********************************\r\nfatal: [default]: FAILED! => {\"failed\": true, \"msg\": \"'redis' is undefined\"}\r\n\tto retry, use: --limit @build_camp_box.retry\r\n\r\nPLAY RECAP *********************************************************************\r\ndefault                    : ok=28   changed=21   unreachable=0    failed=1   \r\n\r\nAnsible failed to complete successfully. Any error output should be\r\nvisible above. Please fix these errors and try again.\r\n```"},"comment":{"url":"https://api.github.com/repos/curationexperts/ansible-hydra/issues/comments/233470168","html_url":"https://github.com/curationexperts/ansible-hydra/issues/148#issuecomment-233470168","issue_url":"https://api.github.com/repos/curationexperts/ansible-hydra/issues/148","id":233470168,"user":{"login":"mark-dce","id":3064318,"avatar_url":"https://avatars.githubusercontent.com/u/3064318?v=3","gravatar_id":"","url":"https://api.github.com/users/mark-dce","html_url":"https://github.com/mark-dce","followers_url":"https://api.github.com/users/mark-dce/followers","following_url":"https://api.github.com/users/mark-dce/following{/other_user}","gists_url":"https://api.github.com/users/mark-dce/gists{/gist_id}","starred_url":"https://api.github.com/users/mark-dce/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mark-dce/subscriptions","organizations_url":"https://api.github.com/users/mark-dce/orgs","repos_url":"https://api.github.com/users/mark-dce/repos","events_url":"https://api.github.com/users/mark-dce/events{/privacy}","received_events_url":"https://api.github.com/users/mark-dce/received_events","type":"User","site_admin":false},"created_at":"2016-07-18T21:51:38Z","updated_at":"2016-07-18T21:51:38Z","body":"It's definitely a balance between elegance once you know what's going on vs. the transparence of seeing exactly what's happening all in one place (with the redundancy and non-executing code that implies).\r\n\r\nAs it's set up now, It did take me about 10 minutes to do a few searches and walk through everything to convince myself that there was a variable missing and not something set up incorrectly on my end.  \r\n\r\nI'd ask yourself this: would your future self prefer the refined version of a single task with variables appropriate to each OS (potentially having to take extra steps to figure out the actual value of those variables).  Or would your future self prefer to have two largely similar hunks of code with explicit settings right in front of you (knowing that you have to ignore every other one because it won't be in effect for whichever OS you choose). I's say go with whichever you think will be easiest to get your head wrapped back around when you haven't looked at it for 6-12 months."}}
{"actor":{"display_login":"mark-dce"}}