Skip to content

docs(readme): clarify aws_region scope (smoke test for DEV-245)#43

Merged
brodkin merged 1 commit into
mainfrom
ryan/smoke-test-pepper-cost-attribution
May 10, 2026
Merged

docs(readme): clarify aws_region scope (smoke test for DEV-245)#43
brodkin merged 1 commit into
mainfrom
ryan/smoke-test-pepper-cost-attribution

Conversation

@brodkin

@brodkin brodkin commented May 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • One-line clarification of the aws_region input description in the inputs table
  • Doubles as the smoke test for DEV-245 — first run of the reusable workflow through the newly tagged Application Inference Profiles after the v1 retag

Smoke-test plan

  • Push triggers pepper-self-review against @v1 (now at d0fdf8d); the new Resolve model step should report arn:…application-inference-profile/cz21awrop223 for review mode
  • After auto-review completes, comment @pepper say hi to exercise on-demand mode and confirm the resolution step reports arn:…application-inference-profile/68jw718dw1jv
  • Verify CloudTrail in 618640261060 shows bedrock:InvokeModel events against both profile ARNs
  • Close this PR without merging once the routing is verified

The previous "Bedrock region" gloss didn't convey that this region also
binds the OIDC role's session — both the role and the inference profiles
must live in this region for OIDC + Bedrock to resolve. Also doubles as a
smoke-test for the new tagged-profile routing now that v1 is updated.
@pepper-pr-review pepper-pr-review Bot added pepper-cooking Pepper is currently reviewing this PR pepper-approved Pepper approved this PR and removed pepper-cooking Pepper is currently reviewing this PR labels May 10, 2026

@pepper-pr-review pepper-pr-review Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Verified against DEV-245 — aligned. The clarification tightens the aws_region description to match reality after #42 introduced Application Inference Profiles: the region isn't just "Bedrock region" anymore, it's where the role and the profiles both live. One line, clean improvement, smoke test yours to run.

When you're ready for another look, comment @pepper review.

— Pepper

@brodkin

brodkin commented May 10, 2026

Copy link
Copy Markdown
Contributor Author

@pepper Print the resolved model ARN you were invoked with by reading this PR's workflow logs, then make no code changes. This is a smoke test for the on-demand cost-attribution profile.

@pepper-pr-review

Copy link
Copy Markdown

Smoke test complete

The on-demand cost-attribution profile resolved to:

arn:aws:bedrock:us-west-2:618640261060:application-inference-profile/68jw718dw1jv

This ARN was successfully used for the Pepper invocation in workflow run #25616191851.

No code changes made as requested.

@brodkin brodkin merged commit 5299b3c into main May 10, 2026
1 check passed
@brodkin brodkin deleted the ryan/smoke-test-pepper-cost-attribution branch May 10, 2026 01:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pepper-approved Pepper approved this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant