Repository navigation
fix(ci): reconcile the dependabot train's ansible-floor-sync and CI pin fixes onto main (#16591) - #16636
Merged
Conversation
…manifests (#16591) Vehicle CI (python-suite shard 8/12) caught a real cross-manifest divergence exposed by combining this train's members, not a conflict between two members: three ansible-side hardcoded floors fell behind the manifest versions several dependabot bumps together moved forward, and ansible_requirements_parity_test.py exists precisely to catch this drift regardless of source (dependabot or human). - roles/ai-stack/tasks/main.yml: transformers>=5.16.1 -> >=5.17.0, matching requirements-ai.txt (#16585's bump). - roles/dependency_patching/defaults/main.yml: langchain-core 1.6.1,<2.0.0 -> 1.6.2,<2.0.0 and pypdf 6.17.0 -> 6.18.0, matching what every manifest in the tree now states (#16572/#16574/#16584/ #16586's bumps). Verified directly: both previously-failing tests (test_the_pre_15623_ai_stack_floors_are_a_regression_case, test_every_cross_manifest_divergence_is_in_the_baseline) now pass when called directly against the fixed tree. This fix travels as its own standalone PR (not part of any dependabot member's own diff, and the vehicle itself never merges) so it lands on main alongside the members it makes consistent with.
…tions (#16591) check_dependency_floors.py --strict failed in CI (not the local advisory pre-push version): three CI-installed exact pins had fallen behind the floor autobot-backend/requirements.txt or requirements.txt declares, on unrelated main changes this vehicle's fresh-main merge first exposed together: - alembic 1.19.1 -> 1.19.2 (autobot-backend/requirements.txt:21) - anthropic 1.3.0 -> 1.4.0 (autobot-backend/requirements.txt:40) - pypdf 6.17.0 -> 6.18.0 (requirements.txt:43) websockets also printed in the report but is a named, non-gating KNOWN_CROSS_VENV_EXEMPTIONS entry (#16394) -- left untouched.
Contributor
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: 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. Comment |
Contributor
✅ SSOT Configuration Compliance: Passing🎉 No new hardcoded values of either class — Known backlog in |
Contributor
This was referenced Sep 13, 2026
Merged
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.
Thinking Path
Fast-track base fix: main (tip
b688b3e48, the dependabot train's 10th member landing) is red on two shards because the train's own vehicle CI caught two real cross-cutting issues during validation, and the fix commits for both were still sitting on the disposable vehicle branch when the members got merged -- the vehicle itself was never meant to merge, so its own fix commits needed to travel as this standalone PR, but that PR wasn't opened before the members landed. This closes that gap.Both fixes were already verified against the combined vehicle tree before the members merged (the vehicle's own CI went fully green after these two commits landed on it) -- this PR is exactly those two commits, cherry-picked onto current main with no changes.
What Changed
autobot-slm-backend/ansible/roles/ai-stack/tasks/main.yml:transformers>=5.16.1->>=5.17.0, matchingrequirements-ai.txt(bumped by the train's own chore(deps): bump the all-dependencies group in /autobot-infrastructure/shared/docker/ai-stack with 5 updates #16585 member).autobot-slm-backend/ansible/roles/dependency_patching/defaults/main.yml:langchain-corefloor1.6.1,<2.0.0->1.6.2,<2.0.0andpypdf6.17.0->6.18.0, matching what every manifest in the tree now states (bumped by the train's own chore(deps): bump the all-dependencies group across 1 directory with 9 updates #16572 member).requirements-ci/storage.txt:alembic==1.19.1->1.19.2, matchingautobot-backend/requirements.txt:21's floor.requirements-ci/ai-ml.txt:anthropic==1.3.0->1.4.0, matchingautobot-backend/requirements.txt:40's floor.requirements-ci/document.txt:pypdf==6.17.0->6.18.0, matchingrequirements.txt:43's floor.Verification
check_dependency_floors.py --strict): alembic 1.19.1<1.19.2, anthropic 1.3.0<1.4.0, pypdf 6.17.0<6.18.0 -- all three fixed here.websocketsalso prints in that report but is the pre-existing, non-gatingKNOWN_CROSS_VENV_EXEMPTIONSentry (ci: run SLM backend tests in their own venv, so CI stops forcing the backend's websockets<16 cap onto the SLM #16394), untouched.repo_tests/ansible_requirements_parity_test.py):test_the_pre_15623_ai_stack_floors_are_a_regression_caseandtest_every_cross_manifest_divergence_is_in_the_baseline-- both fixed by the ansible floor sync above.origin/main) ran clean on this exact 5-file changeset.Risks
Low -- pure version-pin/floor mirroring, no behavior change. Diff is exactly 5 lines across 5 files.
Model Used
Claude Sonnet 5 (claude-sonnet-5)
Issue Link
N/A -- CI base-fix for the dependabot train (#16591), no application issue.
Single-issue rationale
N/A -- this is the vehicle's own reconciliation, finishing the existing train per merge-all-first.
Changelog fragment
Checklist
CLAUDE.mdansible_requirements_parity_test.pyandcheck_dependency_floors.py --strictare the tests this fixes)main