Skip to content

Conversation

@udondan
Copy link
Owner

@udondan udondan commented Jan 18, 2026

Adds latest IAM updates from AWS

Summary by CodeRabbit

  • New Features

    • Added support for ECS Model Context Protocol (ecs-mcp) service with read-only tool invocation capabilities.
  • Removals

    • Removed DeepRacer service and all associated IAM actions, resource types, and condition keys.
  • Chores

    • Updated project version to 0.738.0.
    • Updated IAM policy statement generator statistics to reflect service changes.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 18, 2026

Walkthrough

This PR removes AWS DeepRacer IAM service definitions (actions, resources, condition keys) and adds AWS ECS Model Context Protocol (ecs-mcp) IAM service definitions. Project version and documentation statistics are updated to reflect these service changes.

Changes

Cohort / File(s) Summary
Version Bumps
VERSION, docs/source/conf.py
Version updated from 0.737.0 to 0.738.0 across project files
Documentation Updates
README.md, docs/source/index.rst, CHANGELOG/v0.738.0.md
Statistics updated: Actions 20079 → 20028, Resource Types 2137 → 2130, Condition Keys 2235 → 2230; changelog documents service removal/addition
DeepRacer Service Removal
stats/actions/deepracer, stats/conditions/deepracer, stats/resources/deepracer
Removed all DeepRacer IAM definitions: 53 actions, 5 condition keys, 7 resource types (car, evaluation_job, leaderboard, leaderboard_evaluation_job, reinforcement_learning_model, track, training_job)
ECS-MCP Service Addition
stats/actions/ecs-mcp, stats/conditions/ecs-mcp, stats/resources/ecs-mcp
Added new ECS-MCP service with 2 read actions (InvokeReadOnlyTools, UseMcp); empty condition keys and resources files
Service Listing Update
stats/services
Replaced deepracer entry with ecs-mcp entry in service registry

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Adds latest IAM updates from AWS' is generic and vague. While it relates to the changeset (updating IAM policy data), it does not clearly specify the primary change: removing deepracer service support and adding ecs-mcp service support. Consider a more specific title that describes the main service changes, such as 'Remove deepracer service and add ecs-mcp service to IAM policy data' to clearly communicate the primary modification.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@udondan udondan merged commit 8475b18 into main Jan 18, 2026
9 checks passed
@udondan udondan deleted the iam-updates branch January 18, 2026 02:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants