dnephin / dobi

A build automation tool for Docker applications
https://dnephin.github.io/dobi/
Apache License 2.0
309 stars 36 forks source link

Write imageid to .dobi dir on pull action #196

Closed volfyd closed 4 years ago

volfyd commented 4 years ago

On a pull action, dobi wasn't filling in the imageid field in the .dobi directory, causing a later build action to fail to use the cached version on the check:

if image.ID != record.ImageID || record.Info.ModTime().Before(mtime) {
    t.logger().Debug("Image record older than context")
    return true, nil
}

This change fixes it so dobi writes the imageid on pulls.

CLAassistant commented 4 years ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Leif Huhn seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

lhuhn-wish commented 4 years ago

Too many github accounts...OK let me reopen this with the correct one.