docs(collector): fix AWS CUR console steps and manual IAM role policy - #845
Merged
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates the AWS integration documentation to align with the latest AWS Console UI for Data Exports and expands the manual IAM role creation guide with detailed trust and permissions policies. The review feedback recommends updating a button label to match the verbatim AWS UI and replacing the retired aws-portal:View* permission with the modern billing:View* equivalent.
saiprasadpotdar
force-pushed
the
docs/aws-cur-and-manual-role
branch
from
September 7, 2026 07:14
e9802c4 to
5bebe34
Compare
The CUR steps pointed at the retired Cost & Usage Reports page; legacy reports are now created under Data Exports as the Legacy CUR export type. The manual-role section listed 13 actions across four services, while the CloudFormation role grants ReadOnlyAccess plus billing and Logs rights and the collector calls ~50 services, so a role built from it failed discovery with AccessDenied. Add the trust policy, an option mirroring the template, an explicit least-privilege policy derived from the collector's SDK calls, and the Standard-mode write actions. Signed-off-by: Saiprasad Potdar <saipotdar9@gmail.com>
saiprasadpotdar
force-pushed
the
docs/aws-cur-and-manual-role
branch
from
September 7, 2026 07:53
5bebe34 to
8596730
Compare
mayankpande88
approved these changes
Sep 7, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Two parts of the AWS page no longer matched the console or the product:
ReadOnlyAccessplus billing and Logs rights and the collector calls ~50 services. A role built from the old policy authenticated and read CUR but failed discovery for EC2, RDS, Lambda, ECS and everything else with AccessDenied. It also gave no trust policy and did not say that Standard access mode needs write actions the doc never listed. The section now has: a trust policy with where to find NudgeBee's principal ARN; Option A mirroring the template (ReadOnlyAccess+ inline billing/logs/CUR); Option B, an explicit least-privilege policy derived from every SDK call incollector-server/cloud-collector/providers/aws; and a table of Standard-mode write actions.Also tightened the Method 2 sentence, which put
sts:AssumeRolein the permissions policy instead of the trust policy.Type of change
Verification
nudgebee-aws-cloud-formation.json(read-only branch). Option B generated from a sweep of<service>.<Method>Inputcalls in the collector, api-server crawl and llm-server cloud tools, mapped to IAM action names.Checklist
npm run buildlocally and the site builds without errorsnpm run lintlocally and lint passes (lint targets.tsxonly; no TSX changed)git commit -s)