{"action":"created","issue":{"url":"https://api.github.com/repos/edx/configuration/issues/2774","repository_url":"https://api.github.com/repos/edx/configuration","labels_url":"https://api.github.com/repos/edx/configuration/issues/2774/labels{/name}","comments_url":"https://api.github.com/repos/edx/configuration/issues/2774/comments","events_url":"https://api.github.com/repos/edx/configuration/issues/2774/events","html_url":"https://github.com/edx/configuration/pull/2774","id":133681635,"number":2774,"title":"Populate ENABLE_OAUTH2_PROVIDER in [cl]ms.env.yaml.","user":{"login":"agaylard","id":6389399,"avatar_url":"https://avatars.githubusercontent.com/u/6389399?v=3","gravatar_id":"","url":"https://api.github.com/users/agaylard","html_url":"https://github.com/agaylard","followers_url":"https://api.github.com/users/agaylard/followers","following_url":"https://api.github.com/users/agaylard/following{/other_user}","gists_url":"https://api.github.com/users/agaylard/gists{/gist_id}","starred_url":"https://api.github.com/users/agaylard/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/agaylard/subscriptions","organizations_url":"https://api.github.com/users/agaylard/orgs","repos_url":"https://api.github.com/users/agaylard/repos","events_url":"https://api.github.com/users/agaylard/events{/privacy}","received_events_url":"https://api.github.com/users/agaylard/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2016-02-15T10:42:43Z","updated_at":"2016-02-15T12:17:21Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/edx/configuration/pulls/2774","html_url":"https://github.com/edx/configuration/pull/2774","diff_url":"https://github.com/edx/configuration/pull/2774.diff","patch_url":"https://github.com/edx/configuration/pull/2774.patch"},"body":"**JIRA tickets**: this PR is part of work done for OC-1382\r\n\r\n**Discussions**: This change adds a variable: ```EDXAPP_ENABLE_OAUTH2_PROVIDER```.\r\n\r\nThis allows the ```FEATURES``` list to be populated with ```ENABLE_OAUTH2_PROVIDER``` set to the value of ``EDXAPP_ENABLE_OAUTH2_PROVIDER``.\r\n\r\nPreviously, it was necessary to edit ``/edx/etc/edxapp/[cl]ms.env.yaml`` and add/set ``ENABLE_OAUTH2_PROVIDER`` manually. \r\n\r\nAnsible users needing to control ``ENABLE_OAUTH2_PROVIDER`` can now set ``EDXAPP_ENABLE_OAUTH2_PROVIDER`` to \"true\" to override the default, which is \"false\".\r\n\r\n\r\n**Dependencies**: None\r\n\r\n**Screenshots**: Not applicable.\r\n\r\n**Sandbox URL**: Not applicable.\r\n\r\n**Partner information**: Not applicable.\r\n\r\n**Testing instructions**:\r\n\r\n1. Run ``sudo /edx/bin/ansible-playbook -c local -i \"localhost,\" edx_sandbox.yml `` and confirm successfule completion.\r\n2. Run ``grep OAUTH2 /edx/etc/edxapp/*yaml``; the result should be\r\n\r\n```/edx/etc/edxapp/cms.env.yaml:    ENABLE_OAUTH2_PROVIDER: false```\r\n```/edx/etc/edxapp/lms.env.yaml:    ENABLE_OAUTH2_PROVIDER: false```\r\n(Prior to this change, grep would return no output.)\r\n\r\n**Reviewers**\r\n- [ ] Code: omarkhan\r\n- [ ] Product: (TBD)\r\n- [ ] Platform: (TBD)\r\n- [ ] UX: Not applicable.\r\n- [ ] TNL: Not applicable.\r\n- [ ] a11y: Not applicable.\r\n- [ ] doc/strings: Not applicable.\r\n"},"comment":{"url":"https://api.github.com/repos/edx/configuration/issues/comments/184184549","html_url":"https://github.com/edx/configuration/pull/2774#issuecomment-184184549","issue_url":"https://api.github.com/repos/edx/configuration/issues/2774","id":184184549,"user":{"login":"edx-webhook","id":8964148,"avatar_url":"https://avatars.githubusercontent.com/u/8964148?v=3","gravatar_id":"","url":"https://api.github.com/users/edx-webhook","html_url":"https://github.com/edx-webhook","followers_url":"https://api.github.com/users/edx-webhook/followers","following_url":"https://api.github.com/users/edx-webhook/following{/other_user}","gists_url":"https://api.github.com/users/edx-webhook/gists{/gist_id}","starred_url":"https://api.github.com/users/edx-webhook/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/edx-webhook/subscriptions","organizations_url":"https://api.github.com/users/edx-webhook/orgs","repos_url":"https://api.github.com/users/edx-webhook/repos","events_url":"https://api.github.com/users/edx-webhook/events{/privacy}","received_events_url":"https://api.github.com/users/edx-webhook/received_events","type":"User","site_admin":false},"created_at":"2016-02-15T12:17:21Z","updated_at":"2016-02-15T12:17:21Z","body":"Thanks for the pull request, @agaylard! I've created [OSPR-1147](https://openedx.atlassian.net/browse/OSPR-1147) to keep track of it in JIRA. JIRA is a place for product owners to prioritize feature reviews by the engineering development teams.\n\nFeel free to add as much of the following information to the ticket:\n- supporting documentation\n- edx-code email threads\n- timeline information (\"this must be merged by XX date\", and why that is)\n- partner information (\"this is a course on edx.org\")\n- any other information that can help Product understand the context for the PR\n\nAll technical communication about the code itself will still be done via the GitHub pull request interface. As a reminder, [our process documentation is here](http://edx-developer-guide.readthedocs.org/en/latest/process/overview.html).\n\n\n\nIf you like, you can add yourself to the [AUTHORS](https://github.com/edx/configuration/blob/master/AUTHORS)      file for this repo, though that isn't required.     Please see the [CONTRIBUTING](https://github.com/edx/edx-platform/blob/master/CONTRIBUTING.rst)     file for more information."}}