Skip to content

Add analysis cascading#4680

Draft
islean wants to merge 2 commits intomasterfrom
add-analysis-cascading
Draft

Add analysis cascading#4680
islean wants to merge 2 commits intomasterfrom
add-analysis-cascading

Conversation

@islean
Copy link
Member

@islean islean commented Oct 30, 2025

Description

Closes https://github.com/Clinical-Genomics/bug-reports/issues/39.

Added

Changed

Fixed

  • Deleting a case cascades to the analysis table and removes entries from there.

How to prepare for test

  • Ssh to relevant server (depending on type of change)
  • Use stage: us
  • Paxa the environment: paxa
  • Install on stage (example for Hasta):
    bash /home/proj/production/servers/resources/hasta.scilifelab.se/update-tool-stage.sh -e S_cg -t cg -b add-analysis-cascading -a

How to test

  • Do ...

Expected test outcome

  • Check that ...
  • Take a screenshot and attach or copy/paste the output.

Review

  • Tests executed by
  • "Merge and deploy" approved by
    Thanks for filling in who performed the code review and the test!

This version is a

  • MAJOR - when you make incompatible API changes
  • MINOR - when you add functionality in a backwards compatible manner
  • PATCH - when you make backwards compatible bug fixes or documentation/instructions

Implementation Plan

  • Document in ...
  • Deploy this branch on ...
  • Inform to ...

@islean
Copy link
Member Author

islean commented Oct 31, 2025

Upgrade successful:

INFO  [alembic.runtime.migration] Will assume non-transactional DDL.
INFO  [alembic.runtime.migration] Running upgrade 2d63ba0d1154 -> 76214cad1cb7, Add analysis cascading
fatal: Not a git repository (or any parent up to mount point /home)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
repository is clean
Logging deploy ...
Getting deployer... done.
Getting last commit message and SHA... done.
Getting version of deploy scripts... /home/isak.ohlsson
done.
Log deploy... done.
cg, version 78.0.5
[isak.ohlsson@hasta:~] [S_base] $ 

@islean
Copy link
Member Author

islean commented Oct 31, 2025

On upgrade it cascades:

image

@islean
Copy link
Member Author

islean commented Oct 31, 2025

image

Does not cascade

image

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 6, 2025

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.

1 participant