Gated LevelX dev changes on all Linux regression configurations - #90
Merged
Merged
Conversation
Regression CI only covered master, fetched moving dependencies, and measured only the default configuration. The legacy collector could select gcov from a different compiler version. Pinned compatible ThreadX and FileX revisions, enabled the dev gate on Ubuntu 24.04 with GCC 14, and merged all ten coverage profiles. Added exhaustive single-bit page ECC tests and coverage-input checks. Enforced measured floors of 68.0% lines and 65.4% branches while retaining uncovered sources. GCC 14 baseline: 30/30 tests, 95.38 seconds summed across configurations. Final matrix: 40/40 tests; complete runner 72.68 seconds. Coverage: 1632/2400 lines and 1596/2439 branches. An injected test failure returned 8 with coverage retained; a missing input returned 1. Empty and unmeasured inputs were rejected. actionlint, ShellCheck and whitespace checks passed. Assisted-by: Codex (GPT-6) <noreply@openai.com>
heurtematte
pushed a commit
to eclipse-threadx/.eclipsefdn
that referenced
this pull request
Sep 25, 2026
ThreadX, NetX Duo, FileX, LevelX, USBX and GUIX have no dev protection rules, so their regression results do not prevent merging failing pull requests. Added dev rulesets using the existing ECA, review and project-lead bypass policy. ThreadX and NetX Duo enforce their established regression checks. The other four rulesets remain disabled until their dev workflows are ready: FileX requires eclipse- threadx/filex#106, LevelX requires eclipse-threadx/levelx#90, USBX needs automatic dev regression runs, and GUIX needs dev triggers plus distinct Studio job names. Each staged ruleset needs an explicit enforcement update after its rollout. Otterdog 1.6.1 schema/model validation passed. Comparing rendered configurations found exactly six added dev rulesets: two active and four disabled. All existing settings, master rulesets and Pages policies are unchanged. Verified the inherited ECA/review/bypass policy and all thirteen configured regression check names. Whitespace checks passed. Assisted-by: Codex (GPT-6) <noreply@openai.com>
This was referenced Sep 25, 2026
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
1 participant
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.
Regression CI only ran for master, fetched moving dependencies and reported default-only coverage. The GCC 14 baseline passed 30/30 tests in 95.38 seconds summed across configurations, with 1,424/2,183 lines (65.23%) and 664/1,113 branches (59.66%) covered. The legacy collector selected an incompatible system gcov; baseline coverage was measured with gcov 14.
Pinned ThreadX and the verified FileX #106 head (still open), enabled all ten Linux configurations on dev, modernized installation, and merged repository-relative coverage. Added exhaustive page ECC tests and input-validation checks. Coverage floors are 68.0% lines and 65.4% branches. Deployment remains limited to master; the called test job retains its narrow token permissions. No Windows job is added. Requiring
run_tests / run_testson dev needs an administrator: GitHub rejected branch-protection and ruleset updates. Dependabot activates when its configuration reaches the default branch.Local GCC 14 validation passed 40/40 tests; the complete runner took 72.68 seconds. Merged coverage is 1,632/2,400 lines (68.00%) and 1,596/2,439 branches (65.44%). All ten nonempty profiles contribute to the exact source-line union. Remaining gaps are 768 lines and 843 branches, chiefly simulator adapters, NOR reclaim/open, NAND data movement and media-error paths; none are excluded. A deliberately failing test returned 8 after every configuration completed and preserved coverage; a missing input returned 1. Empty and unmeasured JSON/XML inputs were rejected. actionlint, ShellCheck and whitespace checks passed. Real CI passed 40/40 tests on GCC/gcov 14.2.0 with identical coverage. Installation took 19 seconds, builds 46 seconds, the test/coverage step 119 seconds, and the Linux job 204 seconds. All ten artifact inputs and the exact merged union were verified. ECA passed; deployment was skipped.