openedx-unsupported / edx-analytics-configuration

GNU Affero General Public License v3.0
8 stars 28 forks source link

Use default job flow role (EMR_EC2_DefaultRole). #20

Closed mtyaka closed 8 years ago

mtyaka commented 8 years ago

If the user doesn't explicitly set a job flow role using ansible vars, the default EMR role should be used instead of an arbitrary 'emr' role.

JIRA ticket: https://openedx.atlassian.net/browse/OSPR-1173

openedx-webhooks commented 8 years ago

Thanks for the pull request, @mtyaka! It looks like you're a member of a company that does contract work for edX. If you're doing this work as part of a paid contract with edX, you should talk to edX about who will review this pull request. If this work is not part of a paid contract with edX, then you should ensure that there is an OSPR issue to track this work in JIRA, so that we don't lose track of your pull request.

To automatically create an OSPR issue for this pull request, just visit this link: https://openedx-webhooks.herokuapp.com/github/process_pr?number=20&repo=edx%2Fedx-analytics-configuration

openedx-webhooks commented 8 years ago

Thanks for the pull request, @mtyaka! I've created OSPR-1173 to keep track of it in JIRA. JIRA is a place for product owners to prioritize feature reviews by the engineering development teams.

Feel free to add as much of the following information to the ticket:

All technical communication about the code itself will still be done via the GitHub pull request interface. As a reminder, our process documentation is here.

If you like, you can add yourself to the AUTHORS file for this repo, though that isn't required. Please see the CONTRIBUTING file for more information.

openedx-webhooks commented 8 years ago

Hello @mtyaka: We are unable to continue with review of your submission at this time. Please see the associated JIRA ticket for more explanation.

mtyaka commented 8 years ago

Ugh, I closed this PR by accident. I don't have permission to reopen it, so I opened a new PR instead: https://github.com/edx/edx-analytics-configuration/pull/23