Clinical-Genomics / loqusdbapi

A simple REST api for loqusdb
2 stars 0 forks source link

Push image to Docker Hub prod and stage #10

Closed northwestwitch closed 3 years ago

northwestwitch commented 3 years ago

This PR adds | fixes: fix #9

Review:

This version is a:

northwestwitch commented 3 years ago

Hi @moonso we'd like to deploy and use this repo on the server farm, after moving the LoqusDB database there as well. Eventually Scout would connect to Loqus using this API instead of the command line. To use it via podman and systemd we need to pull it from Docker Hub. This PR should do the pushing to Docker Hub every time we merge a PR to master!

northwestwitch commented 3 years ago

Pushed to Docker hub on push event: 🆗

image

Mropat commented 3 years ago

Nice! We talked about using official action, and I will have to change this for statina and cg in the future, but this was useful anyway

Mropat commented 3 years ago

Someone has to log in as clinicalgenomics and add a deploy key if it doesnt exist yet

northwestwitch commented 3 years ago

Nice! We talked about using official action, and I will have to change this for statina and cg in the future, but this was useful anyway

What's the official action?

northwestwitch commented 3 years ago

Nice! We talked about using official action, and I will have to change this for statina and cg in the future, but this was useful anyway

What's the official action?

Ah OK this one: docker/build-push-action@v2

northwestwitch commented 3 years ago

Now @Mropat ?

henrikstranneheim commented 3 years ago

Well done!

northwestwitch commented 3 years ago

The action to push to to Docker Hub (prod image) wasn't triggered by the new release. We've been missing something.. 🤔