imbajin commented on code in PR #3343: URL: https://github.com/apache/hugegraph/pull/3343#discussion_r4267312054
########## 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: Confirmed from Codecov's recorded-parent lookup and the live commit API. Missing master commit records can prevent a fresh selective PR from inheriting the weekly seed; the carryforward observed within this PR does not prove that scenario works. We are retaining smoke-only master pushes to preserve the CI budget. Both docs/ci.md and the PR description now explicitly state this limitation: weekly validation seeds its own commit, and complete totals are not guaranteed for future selective PRs. Flags remain useful when a recorded parent report exists. No extra runner, repeated master tests, synthetic coverage or threshold relaxation was added. This is a documented limitation, not a claimed end-to-end baseline fix. -- 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]
