Repository navigation
Python: Temporarily disable Bedrock integration tests - #14363
Evan Mattson (moonbox3) merged 1 commit into
Conversation
Skip AWS credential setup and exclude Bedrock cases while the integration credentials are invalid. Co-authored-by: Copilot App <[email protected]> Copilot-Session: 22a9cc50-d463-4c1e-829c-a608a8ac779f
There was a problem hiding this comment.
🟢 Approval recommended
The workflow changes are narrowly scoped to bypass a known failing AWS credential validation step and to exclude Bedrock-only test cases without impacting other integration coverage.
Pull request overview
Temporarily stabilizes the Python integration test GitHub Actions workflow by disabling AWS credential configuration (currently failing validation) and excluding AWS Bedrock-related integration tests, while keeping the rest of the Python integration suite running.
Changes:
- Skip the
aws-actions/configure-aws-credentialsstep in relevant Python integration jobs. - Exclude Bedrock-related tests from pre-merge and scheduled integration test runs via
pytest -k "not bedrock".
File summaries
| File | Description |
|---|---|
.github/workflows/python-integration-tests.yml |
Disables AWS credential setup and filters out Bedrock tests in affected Python integration jobs to prevent credential-validation failures. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
MAF Automated Review — Iteration 1
Result: No findings
Scope: full PR (1 commit(s)): 293dff3ee19f
Model: claude-opus-4.8
Overview
This CI-only change to .github/workflows/python-integration-tests.yml temporarily disables Bedrock integration tests by adding if: false to the three active "Configure AWS Credentials" steps and appending -k "not bedrock" to the five pytest invocations that run Bedrock-containing suites. The scope is precise: every Bedrock case carries a bedrock-prefixed parametrize id, class, or module name that pytest -k matches, no non-Bedrock test contains a keyword-matchable "bedrock" substring, and Bedrock is the only AWS consumer in the integration suite, so disabling the credential steps strands no still-running test. YAML parses cleanly, action SHAs are unchanged, and the secret references on the disabled steps are inert no-ops. No publishable defect was found.
Reviewed the supplied pull-request change set across correctness, security/reliability, architecture, and failure behavior.
No publishable findings remained after source verification for this scope.
Summary
Context
The affected jobs fail during AWS credential validation before pytest starts: