Skip to content

Conversation

forrest-not-gump
Copy link
Contributor

Fixes

Summary

The existing read-only mode JSON IAM policy contains an invalid service name for Amazon EMR, causing errors: "Invalid Service In Action: The service emr:<action> specified in the action does not exist"

Changes

Replaced abbreviated 'emr' service name with full service name 'elasticmapreduce' in read-only IAM policy JSON. 'emr' is not a valid service name for policies in IAM. This change resolves IAM policy errors mentioned above.

User experience

User IAM policy will no longer contain incorrect service:action statements that cause permissions errors when using EMR

Checklist

If your change doesn't seem to apply, please leave them unchecked.

  • [ x] I have reviewed the contributing guidelines
  • [ x] I have performed a self-review of this change
  • [ x] Changes have been tested
  • [ x] Changes are documented

Is this a breaking change? (Y/N) - N

RFC issue number:

Checklist:
N/A

  • Migration process documented
  • Implement warnings (if it can live side by side)

Acknowledgment

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.

Replaced abbreviated 'emr' service name with full service name 'elasticmapreduce' in read-only IAM policy JSON. 'emr' is not a valid service name for policies in IAM
@forrest-not-gump forrest-not-gump changed the title Fix emr IAM policy statements in README.md Docs: Fix emr IAM policy statements in README.md Oct 2, 2025
Copy link

codecov bot commented Oct 2, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.17%. Comparing base (c49497b) to head (15b2db9).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1424      +/-   ##
==========================================
- Coverage   89.45%   89.17%   -0.29%     
==========================================
  Files         726      639      -87     
  Lines       50351    44348    -6003     
  Branches     7951     7079     -872     
==========================================
- Hits        45044    39547    -5497     
+ Misses       3452     3129     -323     
+ Partials     1855     1672     -183     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@forrest-not-gump forrest-not-gump changed the title Docs: Fix emr IAM policy statements in README.md Fix emr IAM policy statements in README.md docs Oct 2, 2025
@forrest-not-gump forrest-not-gump requested a review from a team as a code owner October 6, 2025 15:12
@scottschreckengaust scottschreckengaust added the waiting-for-codeowners Code owners are needed to review label Oct 10, 2025
@scottschreckengaust scottschreckengaust changed the title Fix emr IAM policy statements in README.md docs fix: emr IAM policy statements in docs Oct 10, 2025
@scottschreckengaust scottschreckengaust self-assigned this Oct 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

waiting-for-codeowners Code owners are needed to review

Projects

Status: To triage

Development

Successfully merging this pull request may close these issues.

2 participants