Repository navigation
ci: FND-4522 onboard Renovate on the fleet preset, retire Dependabot - #1141
Open
vaibhavatlan wants to merge 2 commits into
Open
vaibhavatlan wants to merge 2 commits into
vaibhavatlan wants to merge 2 commits into
Conversation
Extend application-sdk's fleet Renovate preset, adding pyatlan-specific overrides. Remove .github/dependabot.yml so the two bots don't open duplicate PRs. Overrides on the preset: - platformAutomerge off: main has no required status checks, so GitHub native auto-merge would merge red PRs. Renovate's own merge waits for every check to be green. - Lock maintenance runs weekly and is reviewed. The preset's bounded-lock command exists only on the self-hosted runner, and this repo runs on Mend. - py3.9 compatibility pins (python_version markers) are left alone. pep621 ignores markers and would bump them past 3.9 support. - CI python-version pins are not bumped. github-actions majors are reviewed, and action releases are held for 3 days. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Auto-merge minor/patch updates in [dependency-groups] (dev, docs). These never ship in the published wheel. With platformAutomerge off, Renovate merges the PR itself, and only when every check on the branch is green. Move [project].dependencies minor/patch updates into their own "runtime python dependencies" group, which stays reviewed. Renovate auto-merges a group only when every member does, so keeping runtime deps in the shared group would block the dev/docs PR. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This branch has not been deployed
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.
What
renovate.json, which extends the same fleet preset application-sdk uses (github>atlanhq/application-sdk//renovate-config/default.json). Overrides cover where pyatlan differs..github/dependabot.yml(daily pip + github-actions version updates) so Renovate and Dependabot don't open duplicate PRs. Dependabot alerts are untouched; Renovate's vulnerability PRs read them.The Mend Renovate app is already installed here. Merging this onboards the repo, and Mend then closes its stale onboarding PR #1029.
What you get from the preset
pyatlan-specific overrides (and why)
platformAutomerge: falseeverywheremainhas no required status checks (the only ruleset is Copilot review). GitHub native auto-merge waits only for required checks, so it would merge red PRs. With this off, Renovate merges itself, and only when every check is green. Flip it back once main requires the pyatlan-pr checks.postUpgradeTasks.commands: []renovate-uv-lock-bounded, which exists only on the self-hosted fleet runner. This repo runs on Mend, which would publish a permanent redrenovate/artifacts(the same override application-sdk carries). Without that bound, transitive packages have no release-age hold, so a human reviews the lock PR.matchJsonata, 14 lines)python_versionmarkers. A lookup dry-run proposed moving the< '3.10'lines to their ≥3.10 twins' versions (pydantic~=2.13.4→~=2.14.2, pytest~=8.4.2→~=9.2.0, …). The universal uv lock would then fail, and so would the whole grouped PR. The ≥3.10 lines keep updating. When a held pin is moved by hand, update its entry.pythondep disabledpython-version3.9/3.11/3.12 → 3.15 in the auto-merged actions group. Those pins are support-matrix decisions.pyatlan-pr.yamlskips the test jobs for.github/-only PRs, so green doesn't mean the bumped action ran. The preset's cooldown covers only PyPI.Verification
renovate-config-validator(renovate@44.3.2, the fleet-pinned version): passes.renovate --platform=local --dry-run=lookupagainst this branch: all 14 marker-held lines and all 11 CIpythonpins reportdisabled. Their ≥3.10 twins and the action bumps are still proposed, and fresh releases are held by the 3-day window.Auto-merge (second commit)
[dependency-groups]) minor/patch auto-merge. They never ship in the wheel. WithplatformAutomerge: false, Renovate merges the PR itself, and only when every check on the branch is green (unit/QA 3.9–3.14, vuln scans, integration tests). A red or pending check blocks it.[project].dependencies) get their own reviewed group (runtime python dependencies). A range change there reaches every consumer. It's a separate group because Renovate auto-merges a group only if every member does. Dry-run confirms the split:renovate/non-critical-python-dependencies(dev/docs) vsrenovate/runtime-python-dependencies.Needs a repo admin
mainrequiring the pyatlan-pr checks (e.g.qa-checks-and-unit-tests (3.9)…(3.14),vulnerability-scan (…)). After that,platformAutomergecan go back totrue.After merge: close the 7 stale Dependabot PRs.
🤖 Generated with Claude Code