Add "test" and "status" to asset entries in `status.yaml` files

jwodder commented 3 months ago

Closes #86.

Upon merging this PR, the status.yaml files need to be migrated as follows:

Stop any running dandiset-healthstatus instances on drogon
In an up-to-date clone of this repository, run python3 tools/migrations/gh-86.py results/ (PyYAML required)
Commit & push the changes
Ensure the dandisets-healthstatus checkout on drogon is up to date before resuming the healthstatus processes

codecov[bot] commented 3 months ago

Codecov Report

Attention: Patch coverage is 54.28571% with 16 lines in your changes missing coverage. Please review.

Project coverage is 60.78%. Comparing base (7a81cc1) to head (a2809b2).

Files	Patch %	Lines
code/src/healthstatus/core.py	54.28%	15 Missing and 1 partial :warning:

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #87 +/- ## ========================================== + Coverage 60.37% 60.78% +0.40% ========================================== Files 11 11 Lines 954 946 -8 Branches 215 207 -8 ========================================== - Hits 576 575 -1 + Misses 355 349 -6 + Partials 23 22 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

jwodder commented 3 months ago

@yarikoptic Ping.

jwodder commented 3 months ago

@yarikoptic Ping.

jwodder commented 3 months ago

@yarikoptic FYI, I've killed the run_loop.sh process on drogon, and now I'm waiting for the current run.sh process to complete. Based on the timestamps for recent runs, I expect it to complete around 7 PM EDT today.

jwodder commented 3 months ago

@yarikoptic Merge & deployment completed.

yarikoptic commented 3 months ago

Thank you!

dandi / dandisets-healthstatus

Add "test" and "status" to asset entries in `status.yaml` files #87

Codecov Report