ci: opt-in config-server pairing for integration tests (sdc-pair)#447
Conversation
Integration tests now use config-server main by default, same-named upstream branch tip only when the PR is labeled sdc-pair, plus optional workflow_dispatch override. Avoids accidental pairing with unrelated branches; drops releases/latest for config-server. Co-authored-by: Cursor <cursoragent@cursor.com>
alexandernorth
left a comment
There was a problem hiding this comment.
Maybe we can decide on the label naming?
Something like 'component-feature-align' or something like this?
Otherwise LGTM
Thats also quite cryptic and also long... I'm flexible what the label is called, but this suggestion I don't like, sorry! |
What do you think about |
|
paired-test? |
Aligns with review consensus on PR #447; repo label should be paired-test. Co-authored-by: Cursor <cursoragent@cursor.com>
|
Resolved: the opt-in integration label is now |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! 🚀 New features to boost your workflow:
|
Summary
sdcio/integration-testsreusable workflow) now receive config-servermainby default instead of the latest GitHub Release tag.sdcio/config-serveris used only when this PR has the labelsdc-pair(avoids accidental pairing with unrelated branches that share a name).workflow_dispatch: optional inputconfigserver_versionpins config-server (checkout + GHCR image tag) when you need an explicit ref (main, full SHA,v0.0.0-PR…, etc.).pull-requests: readis required sopushruns can detect an open PR on this branch and whether it carriessdc-pair(same behaviour as thepull_requestevent).How to use cross-repo (data-server + config-server)
sdc-pairlabel in this repo (Settings → Labels) if it does not exist yet.sdc-pairto this PR when you want integration to resolve config-server to that same-named branch tip; otherwise integration stays on config-servermain.Mirror PR (config-server): sdcio/config-server#455
Test plan
sdc-pairuses config-servermainin integration.sdc-pairand matching upstream branch resolves to expected SHA / passes integration when images exist.configserver_versionset overrides auto resolution.