Skip to content

Containerization #3

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
Open
ivan-c wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Containerization #3

ivan-c wants to merge 2 commits into from

Conversation

@ivan-c
Copy link
Member

@ivan-c ivan-c commented Jun 29, 2023 •
edited
Loading

  • Add node docker image
    • Use artifacts built (built/) from Typescript codebase
      • Build (multistage) from Typescript in (src/) directly?
  • Add continuous delivery to GHCR

@ivan-c ivan-c force-pushed the feature/containers branch 2 times, most recently from b0e93cc to 3a0b728 Compare June 29, 2023 17:57
@ivan-c ivan-c force-pushed the feature/containers branch from 3a0b728 to 8603b03 Compare June 29, 2023 18:33
@ivan-c
Copy link
Member Author

ivan-c commented Jun 29, 2023

Tested as follows; worked great!

docker run \
    --volume $PWD:/opt/node/app/downloads \
ghcr.io/uwcirg/bulk-data-client:feature-containers \
    --global \
    --fhir-url https://fhir.agg-data.connectathon.dev.cirg.uw.edu/fhir/ \
    --_type Measure,MeasureReport

@ivan-c ivan-c marked this pull request as ready for review June 29, 2023 18:35
@ivan-c ivan-c requested review from daniellrgn and mcjustin June 29, 2023 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mcjustin mcjustin mcjustin approved these changes

@daniellrgn daniellrgn Awaiting requested review from daniellrgn

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ivan-c @mcjustin