Open datadog-agent pin PR when tagging an Agent release branch - #25116
Open datadog-agent pin PR when tagging an Agent release branch#25116iliakur wants to merge 4 commits into
Conversation
`ddev release branch tag` now opens a PR on DataDog/datadog-agent that pins INTEGRATIONS_CORE_VERSION in release.json to the integrations-core commit the tag was placed on. The PR targets the matching Agent release branch (never main) and is built through the async GitHub client, so no local checkout of datadog-agent is required. The command is idempotent: re-running for an existing tag skips tag creation and re-runs the follow-up steps, which is the recovery path for when the Agent release branch did not exist at tag time. Adds get_ref, create_ref, get_content and create_or_update_file_contents to the async GitHub client, with git-data/contents response models.
|
evalya-impact-summaryevalya impact analysis |
…anch-tag-opens-pr-with-pin
Mirror get_ref, create_ref, get_content and create_or_update_file_contents on FakeAsyncGitHubClient (with defaults and call-table entries), and drive the datadog-agent bump PR tests through the fake_async_github fixture instead of a raw MockTransport.
Validation ReportAll 21 validations passed. Show details
|
What does this PR do?
ddev release branch tagnow opens a PR onDataDog/datadog-agentafter tagging a release branch. The PR pinsINTEGRATIONS_CORE_VERSIONinrelease.jsonto the integrations-core commit the tag points at.7.84.x), nevermain.datadog-agentis needed. The command uses the async GitHub client, adding four endpoint methods (get_ref,create_ref,get_content,create_or_update_file_contents) and their response models.Motivation
Pinning integrations-core into the Agent's
release.jsonwas a manual step after tagging. Automating it removes a hand-off and a class of mistakes: wrong commit, wrong branch, or targetingmain. The Agent release branch is cut separately and may not exist at tag time, so the idempotent re-run finishes the job later without re-tagging.Review checklist (to be filled by reviewers)
qa/requiredif this PR needs QA validation, orqa/skip-qaif it does not. Exactly one of the two is required.backport/<branch-name>label to the PR and it will automatically open a backport PR once this one is merged