Skip to content

Conversation

@eap
Copy link
Collaborator

@eap eap commented Aug 31, 2025

This change introduces the new "test_strategy" flag and replaces "unittest_dependencies" with test dependencies. This allows simplified builds for some "leaf-node" packages such as models. Middle-graph packages can also select more limited builds targeting packages that are known to be affected rather than building everything.

Strategies are as follows

ALL: Used for core dependencies like oops, ioda, ufo, etc. Runs two builds and two tests in this order: unit_build -> unit_test -> integration_build -> integration_test
UNIT: Used for "leaf-node" packages like mpas-jedi and most models. Runs a single build with the specified dependencies and runs package unit test
INTEGRATION: Used for "middle-graph" packages like CRTM. Runs a single build with the specified dependencies and runs all tests for the built packages.

Examples:

Issues

Fixes: #27

Checklist

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have run the unit tests before creating the PR

@eap eap mentioned this pull request Sep 1, 2025
2 tasks
@eap eap requested a review from ashley314 September 9, 2025 18:47
@eap eap marked this pull request as ready for review September 9, 2025 18:48
@eap eap self-assigned this Sep 11, 2025
Copy link
Collaborator

@ashley314 ashley314 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good and changes satisfy the goal of the PR. Thanks!

@eap eap merged commit ee1e539 into develop Sep 12, 2025
1 check passed
@eap eap deleted the feature/downstream-trim branch September 12, 2025 16:09
eap added a commit to JCSDA/CRTMv3 that referenced this pull request Sep 12, 2025
Swap jedi-ci action branch back to deveop now that JCSDA-internal/jedi-ci#21 is merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Trim non-downstream repos from integration tests

3 participants