Skip to content

CI: ModelDB CI workflow that reuses an existing wheels artifact - #3838

Merged
nrnhines merged 3 commits into
masterfrom
hines-grok/modeldb-reuse-wheels
Aug 12, 2026
Merged

CI: ModelDB CI workflow that reuses an existing wheels artifact#3838
nrnhines merged 3 commits into
masterfrom
hines-grok/modeldb-reuse-wheels

Conversation

@nrnhines

@nrnhines nrnhines commented Aug 7, 2026

Copy link
Copy Markdown
Member

Summary

When a NEURON Release (or other wheel-building run) is green through the
wheel matrix but fails only at ModelDB CI, rebuilding every wheel just to
retest ModelDB is wasteful.

This PR adds a manual workflow, ModelDB CI (reuse wheels), that:

  • Takes an existing GitHub Actions wheels artifact (id or URL)
  • Calls nrn-modeldb-ci with that artifact as neuron_v2
  • Lets the operator choose modeldb_ci_ref (branch/tag/SHA of
    nrn-modeldb-ci to checkout for modeldb-run.yaml / runmodels)
  • Optionally limits the suite via models_to_run (space-separated
    accessions; empty = all)

No wheels are built in this workflow.

Operator flow

  1. Open a prior run that uploaded artifact wheels.
  2. Copy the artifact id (e.g. 8935275231) or a
    https://github.com/neuronsimulator/nrn/actions/artifacts/<id> URL.
  3. Actions → ModelDB CI (reuse wheels) → Run workflow (or gh workflow run
    after this file is on the default branch — GitHub only registers
    workflow_dispatch workflows from master).
  4. Set neuron_v1 (e.g. neuron==9.0.1), modeldb_ci_ref (usually master),
    and optional models_to_run (e.g. 2018023 2020504 for a smoke test).

Scope note

ModelDB installs one compatible wheel from the multi-wheel zip (first that
pip accepts on the job's Python), not the full OS×Python matrix. That matches
existing nrn-modeldb-ci behavior.

Dependency

Requires the ref input on the callable modeldb-ci workflow:

Until that is merged, this workflow temporarily uses:

uses: neuronsimulator/nrn-modeldb-ci/.github/workflows/nrn-modeldb-ci.yaml@hines-grok/modeldb-ci-ref-input

Before merge to nrn master, switch that line to @master once
neuronsimulator/nrn-modeldb-ci#154 is on modeldb-ci master (or land that PR
first, then flip in this PR).

Follow-ups (out of scope here)

Test plan

Add a workflow_dispatch entry point to retest ModelDB against wheels from a
prior run (no wheel rebuild), with modeldb_ci_ref selecting the
nrn-modeldb-ci package tip. Temporarily calls the modeldb-ci branch that
adds the ref input; switch uses@ to master after that lands.
@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

✔️ 6d70d68 -> artifacts URL

@azure-pipelines

Copy link
Copy Markdown

✔️ 6d70d68 -> Azure artifacts URL

@codecov

codecov Bot commented Aug 7, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.38%. Comparing base (cee84a6) to head (64f0f27).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3838      +/-   ##
==========================================
+ Coverage   68.34%   68.38%   +0.03%     
==========================================
  Files         701      701              
  Lines      112267   112267              
==========================================
+ Hits        76731    76774      +43     
+ Misses      35536    35493      -43     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

neuronsimulator/nrn-modeldb-ci#154 landed the callable `ref` input on master.
Drop the temporary pin to hines-grok/modeldb-ci-ref-input.
@sonarqubecloud

Copy link
Copy Markdown

Comment thread .github/workflows/modeldb-ci-reuse-wheels.yml Dismissed
@github-actions

Copy link
Copy Markdown
Contributor

✔️ 64f0f27 -> artifacts URL

@nrnhines
nrnhines requested a review from ramcdougal August 11, 2026 21:10
@azure-pipelines

Copy link
Copy Markdown

✔️ 64f0f27 -> Azure artifacts URL

@nrnhines

Copy link
Copy Markdown
Member Author

@ramcdougal Please approve. Needs to be in master for github actions to list it in "All workflows". At that point , after some testing, if the workflow needs improvment, the workflow interface allows one to choose a controller branch (PR) and increment changes til satisfied (and add documentation as well).

@nrnhines
nrnhines merged commit 1739c01 into master Aug 12, 2026
44 checks passed
@nrnhines
nrnhines deleted the hines-grok/modeldb-reuse-wheels branch August 12, 2026 00:27
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.

3 participants