Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove informdb PAT #200

Merged
merged 28 commits into from
Dec 21, 2023
Merged

Remove informdb PAT #200

merged 28 commits into from
Dec 21, 2023

Conversation

stefpiatek
Copy link
Contributor

Will need to rollback to using the tag but should run the system tests in CI

milanmlft
milanmlft approved these changes Dec 21, 2023
View reviewed changes
Copy link
Member

@milanmlft milanmlft 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, once we figure out why the EHR API test is failing

test/Dockerfile.fake_emap_star Outdated Show resolved Hide resolved
stefpiatek and others added 21 commits December 21, 2023 12:43
@stefpiatek
Avoid test container clash
f256f63 
Also take remove volumes in docker compose
@stefpiatek
Remove system tests docker container orphans
ba598d2
@stefpiatek
Install core and cli separately
7e50da1
@stefpiatek
Install packages in editable mode
c3a9dcf
@stefpiatek
Use system test project name in test scripts
5906a84
@stefpiatek
Don't remove orphans in second call
0dfe904 
Of course this removes the previously upped containers
@stefpiatek
Set the system test name for fake star
290d2a4
@stefpiatek
Add healthcheck for fake VNA
45bf1ee
@stefpiatek
Try quoting in post of dicom image
c8e15af
@stefpiatek
Setup python for system tests in CI
168f119
@stefpiatek
Remove interactive terminal for checking pixl anon
08fc1ed
@stefpiatek
Update expected rows in pixl anon
96aeb1f
@stefpiatek
Update tests system tests that links DICOM and parquet files
d191792
@stefpiatek
Use the correct test resource directory
e879d04
@stefpiatek
Update star data to have parquet patients
438b1ba
@milanmlft
Fix docker shutdown in after system test
81d4b1b
@stefpiatek
Update test setup
5400969
@stefpiatek
Merge remote-tracking branch 'origin/stef/remove_pat' into stef/remov…
2a33130 
…e_pat
@milanmlft
Merge branch 'main' into stef/remove_pat
add09b5
@milanmlft
Deserialise message before callback
05f56c1
@milanmlft
Increase ORTHANC storage limit during tests
9591cf9
@stefpiatek stefpiatek merged commit c662aab into main Dec 21, 2023
8 checks passed
@stefpiatek stefpiatek deleted the stef/remove_pat branch December 21, 2023 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@milanmlft milanmlft milanmlft approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@stefpiatek @milanmlft