ExaWorks / SDK

ExaWorks SDK
11 stars 12 forks source link

CI: Fix deploy of the `latest` tag #65

Closed SteVwonder closed 3 years ago

SteVwonder commented 3 years ago

It is hard to test the deploy action since it only runs after a PR merge. So a failure occurred with a message:

Run docker push exaworks/sdk-base:latest
The push refers to repository [docker.io/exaworks/sdk-base]
tag does not exist: exaworks/sdk-base:latest
Error: Process completed with exit code 1.

This includes a fix for that error by tagging the already built centos7 images with the latest tag before pushing.

Also includes a fix for some error messages that I was seeing in the building of the centos8 image in CI

dongahn commented 3 years ago

LGTM.

@mtitov: could you take a quick look at d9b6aa6 before I manually merge this in?

SteVwonder commented 3 years ago

Looks like the deploy action worked this time :tada: Thanks @mtitov and @dongahn for the review.