libero / publisher

The starting point for raising issues for Libero Publisher
MIT License
16 stars 4 forks source link

Publish Docker image for article-store #310

Closed giorgiosironi closed 4 years ago

giorgiosironi commented 4 years ago

Definition of Done:

Related

giorgiosironi commented 4 years ago

In the first run on master, the script ~was skipped~ executed but wrongly detected it didn't need to push:

Run .scripts/github/push-image.sh article-store
  .scripts/github/push-image.sh article-store
  shell: /bin/bash -e {0}
  env:
    IMAGE_TAG: 303714e04272951b3049b0e71c8b260aa930728c
    DOCKER_USERNAME: ***
    DOCKER_PASSWORD: ***
not pushing an image, we are not on master
giorgiosironi commented 4 years ago

Probably a typo in using refs/head/ rather than refs/heads/.

thewilkybarkid commented 4 years ago

https://hub.docker.com/r/liberoadmin/article-store now available.

giorgiosironi commented 4 years ago

https://github.com/libero/reviewer-client/runs/375306068 is green. https://hub.docker.com/layers/liberoadmin/reviewer-client/ee630cdec6f8a9c3514aa38dcd30a058446c6849/images/sha256-98b012c3eb45d2271eb4b037dca3ba8f178888deb5d982460b76a85ba4a3c3e8 is there, it just took a few minutes for https://hub.docker.com/r/liberoadmin/reviewer-client/tags to add it to the list.