Skip to content

Link to AWS CPI IAM policy on main - #939

Merged
aramprice merged 1 commit into
mainfrom
codex/five-bosh-defaults-main
Nov 3, 2026
Merged

aramprice merged 1 commit into
mainfrom
codex/five-bosh-defaults-main

Conversation

@aramprice

@aramprice aramprice commented Nov 3, 2026 •

Copy link
Copy Markdown
Member

The AWS IAM documentation will link to the sample policy on main. This is a documentation follow-up and does not gate the AWS CPI branch rename.

Validation: git diff --check passed.

The link will resolve once the AWS CPI rename is complete.

This PR changes checked-in files only. Deployed pipeline reconfiguration and runtime checks remain operator-owned; no Concourse pipelines or jobs were modified, paused, unpaused, or triggered for this work.

Cutover plan:

Shared community configuration (draft): cloudfoundry/community#1682

@coderabbitai

coderabbitai Bot commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Walkthrough

The sample IAM policy link now points to the main branch instead of master.

Suggested reviewers: ivaylogi98

Merge Risk: Low · up to 566e5

The sample-policy link may be unavailable if these docs publish before the AWS CPI branch rename. Confirm the destination works before publication or retain the redirect-compatible master link.

Pre-merge checks: Passed 4
Check name Status Explanation
Title check Passed The title "Link to AWS CPI IAM policy on main" matches the only change in the PR, which switches the sample IAM policy link in content/aws-iam-users.md from the master branch to main. The title is spe…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Description check Passed The description relates directly to the changeset. It states that the AWS IAM documentation will link to the sample policy on main, which matches the one-line change in content/aws-iam-users.md from…
Full details: Title check

Explanation

The title "Link to AWS CPI IAM policy on main" matches the only change in the PR, which switches the sample IAM policy link in content/aws-iam-users.md from the master branch to main. The title is specific and accurate, so the check passes.

Full details: Description check

Explanation

The description relates directly to the changeset. It states that the AWS IAM documentation will link to the sample policy on main, which matches the one-line change in content/aws-iam-users.md from master to main. It also gives a validation step (git diff --check passed) and a cutover reference. The description is specific enough to pass this lenient check.

Finishing Touches
Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

CodeRabbit Verify

  • Run CodeRabbit Verify to test the changes in this pull request.
Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: cb6fe414-482a-45e2-92f5-788419de4128
Commits

Reviewing files that changed from the base of the PR and between 9496b27 and 566e576.

Files selected for processing: 1
  • content/aws-iam-users.md

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

@coderabbitai coderabbitai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Fix comments: Fix CodeRabbit comments on this PR

Comment thread content/aws-iam-users.md Resolved
@github-project-automation github-project-automation Bot moved this to Waiting for Changes | Open for Contribution in Foundational Infrastructure Working Group Nov 3, 2026
@cf-foundation-community-automation cf-foundation-community-automation Bot moved this from Waiting for Changes | Open for Contribution to Inbox in Foundational Infrastructure Working Group Nov 3, 2026
@github-project-automation github-project-automation Bot moved this from Inbox to Pending Merge | Prioritized in Foundational Infrastructure Working Group Nov 3, 2026
@aramprice
aramprice merged commit 872dbe1 into main Nov 3, 2026
2 checks passed
@aramprice
aramprice deleted the codex/five-bosh-defaults-main branch November 3, 2026 06:23
@github-project-automation github-project-automation Bot moved this from Pending Merge | Prioritized to Done in Foundational Infrastructure Working Group Nov 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

1 participant