Restructure test_and_docs.yml into two-stage DR-008 Option 4 flow#18
Restructure test_and_docs.yml into two-stage DR-008 Option 4 flow#18Subramanian-K812 wants to merge 21 commits into
Conversation
|
@PiotrKorkus
|
PiotrKorkus
left a comment
There was a problem hiding this comment.
We need a class that will contain resolved deps versions from ref_int root and interface that will allow to scan MODULE file and overwrite version based on that resolved class.
Then we execute UTs, coverage directly in that locations. Collect all the reports, and fill the summary tables as currently.
The main entry point and place where module interact with ref_int by providing any information needs to be known_good.json
| # mechanism — each module is tested against the versions resolved in Stage 1, | ||
| # without modifying the module's released sources. |
There was a problem hiding this comment.
how is it used? I dont see any usage of lockfile from previous job
| @@ -126,10 +134,145 @@ jobs: | |||
| path: github-pages.tar | |||
| retention-days: 3 | |||
| if-no-files-found: error | |||
There was a problem hiding this comment.
why do we install it as last step? not needed?
| matrix: | ||
| module: | ||
| - score_baselibs | ||
| - score_baselibs_rust | ||
| - score_communication | ||
| - score_persistency | ||
| - score_orchestrator | ||
| - score_kyron | ||
| - score_lifecycle_health | ||
| - score_logging |
There was a problem hiding this comment.
dont hardcode it, we need to take it from know_good always
| uses: actions/checkout@v4 | ||
| - name: Execute Unit Tests with Coverage Analysis | ||
| run: | | ||
| python ./scripts/quality_runners.py --modules-to-test ${{ matrix.module }} |
There was a problem hiding this comment.
the goal of Option 4 is that we overwrite deps from resolved deps in ref_int and execute UTs and coverage directly in the modules - via bazel directories (preferable) or git checkout if there is a reason to not use bazel.
…kout, resolved-dep injection, in-module UT/coverage
dc36dc0 to
cfad495
Compare
…orch config, bazel_patches injection, genhtml errors
…erride) injected as full set in Stage 2
…ore/launch_manager
…stead of .to_string()
…on with ref_int's pin
…2_2_0_posix for Stage 2 module build
… markers, simplify imports
No description provided.