bitflicker64 commented on code in PR #3343: URL: https://github.com/apache/hugegraph/pull/3343#discussion_r4267200834
########## docs/ci.md: ########## @@ -11,11 +11,24 @@ is merged and synchronized by Gitbox; verify live protection then. Failed tests real failed results. Valid selection and Java outputs allow selected tests to run even when the preflight license checks fail. -For migration, preflight keeps the existing `check-license` check name on the same runner. -A temporary `Server memory tests` check verifies the selected Memory result while live protection -still requires that name. It uses one lightweight `ubuntu-slim` runner and repeats no tests. -After Gitbox applies the new configuration, this bridge skips without allocating a runner; -no Infra intervention or change to review protection is needed. +Preflight keeps its real `check-license` name. The temporary `Server memory tests` +protection bridge has been removed after confirming both ASF and org master required +contexts are empty. Actual backend jobs are named `Server memory tests (Java ...)` +and `Server rocksdb tests (Java ...)`; their native checks report real execution. + +## Coverage with selective tests + +Each coverage upload has a stable test-group flag. The [Codecov configuration](../.github/codecov.yml) +enables [carryforward](https://docs.codecov.com/docs/carryforward-flags), which +preserves the last flagged report for unexecuted groups, so skipping unchanged Commons, +PD/Store or backend checks does not remove their previous data from project totals. +The PR comment explicitly marks carried-forward data. It is historical coverage, +not evidence that a skipped test ran; native job failures remain failures. + +New flags need a complete initial baseline. After rollout, the existing weekly full run Review Comment: Important: For most PRs, carryforward has no master baseline, so the PR goal fails once master moves past the weekly run. Since #3337, a master push runs only smoke checks. Run 38459599936 (push of 92af2f8) skipped commons, pd_store, build-server and server-rocksdb and uploaded nothing. 92af2f8 appears in Codecov's public API only because of manual run 38491119811. ce22afe96 and d168089b7 are absent. Codecov carries a missing flag forward from the head commit's parent chain. A PR based on d168089b7, or on any later commit, has a parent Codecov does not know, so its partial report shows the same artificial change. Please upload the affected flagged groups on master pushes. If that cost is not wanted, state the limit here and in the PR description. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
