Commit c0a4142
authored
ci(bedrock): enable OIDC web identity recording (ogx-ai#6169)
## Summary
- add an optional Bedrock recording path for the STS web identity auth
introduced in ogx-ai#5388
- when AWS_BEDROCK_ROLE_ARN is configured, request a GitHub OIDC token
for sts.amazonaws.com and expose
AWS_ROLE_ARN/AWS_WEB_IDENTITY_TOKEN_FILE to the Bedrock provider
- keep the current bearer-token path as fallback when
AWS_BEDROCK_ROLE_ARN is not configured
- pass standard AWS credential-chain env vars through Docker integration
test runs
## Context
ogx-ai#6167 preserved compatibility between AWS_BEDROCK_BEARER_TOKEN and the
legacy AWS_BEARER_TOKEN_BEDROCK name. That PR is now merged.
The remaining question from the failed Bedrock re-record is whether CI
ever moved onto the STS/OIDC auth path from ogx-ai#5388. The provider already
supports SigV4 + web identity, but the record workflow did not prepare
AWS_ROLE_ARN/AWS_WEB_IDENTITY_TOKEN_FILE for Bedrock. This PR wires that
optional path without removing the bearer-token fallback.
To use it, configure an AWS_BEDROCK_ROLE_ARN repository secret for the
role trusted by GitHub Actions OIDC.
## Test Plan
- uv run pytest tests/unit/providers/inference/test_bedrock_config.py
tests/unit/providers/inference/bedrock/test_sigv4_auth.py -q
- uv run pre-commit run check-yaml --files
.github/workflows/record-integration-tests.yml
- uv run pre-commit run actionlint --files
.github/workflows/record-integration-tests.yml
- pre-commit via git commit hooks from original commit
Signed-off-by: Sumanth Kamenani <skamenan@redhat.com>1 parent 0f5e0cc commit c0a4142
2 files changed
Lines changed: 30 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
283 | 283 | | |
284 | 284 | | |
285 | 285 | | |
| 286 | + | |
| 287 | + | |
| 288 | + | |
| 289 | + | |
| 290 | + | |
| 291 | + | |
| 292 | + | |
| 293 | + | |
| 294 | + | |
| 295 | + | |
| 296 | + | |
| 297 | + | |
| 298 | + | |
| 299 | + | |
| 300 | + | |
| 301 | + | |
| 302 | + | |
| 303 | + | |
| 304 | + | |
| 305 | + | |
| 306 | + | |
| 307 | + | |
| 308 | + | |
| 309 | + | |
286 | 310 | | |
287 | 311 | | |
288 | 312 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
506 | 506 | | |
507 | 507 | | |
508 | 508 | | |
| 509 | + | |
| 510 | + | |
| 511 | + | |
| 512 | + | |
| 513 | + | |
| 514 | + | |
509 | 515 | | |
510 | 516 | | |
511 | 517 | | |
| |||
0 commit comments