Page MenuHomePhabricator
Differential D17377

[chronik-client] Dockerfile back to latest
ClosedPublic
Actions

Authored by bytesofman on Fri, Dec 20, 03:36.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABCc0764c52ccd0: [chronik-client] Dockerfile back to latest
Summary

Changing to @2.0.0 in the last diff did work. So, there is an issue with having this version potentially cached.

The solution is to run the docker build with --no-cache. However looking into it, docker-run.sh does not support this script. I am not sure why -- mb it just needs to be extended.

At the moment, keeping it @latest is better than keeping it pinned to 2.0.0. Will look to resolve the cache issue on docker build in CI.

Test Plan

docker build -f chronik-client.Dockerfile -t chronik-client_local .

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bytesofman created this revision.Fri, Dec 20, 03:36
Herald added a reviewer: Restricted Project. · View Herald TranscriptFri, Dec 20, 03:36
Harbormaster completed remote builds in B31727: Diff 51693.Fri, Dec 20, 03:36
bytesofman published this revision for review.Fri, Dec 20, 03:36
bytesofman added a comment.Fri, Dec 20, 12:14

ci has been updated to cache-bust

Fabien accepted this revision.Fri, Dec 20, 13:06
Fabien added a subscriber: Fabien.

We could do some hacking of the docker file so we don't have to clear the cache when not needed, I will look in to it

This revision is now accepted and ready to land.Fri, Dec 20, 13:06
Closed by commit rABCc0764c52ccd0: [chronik-client] Dockerfile back to latest (authored by bytesofman). · Explain WhyFri, Dec 20, 13:30
This revision was automatically updated to reflect the committed changes.
bytesofman added a commit: rABCc0764c52ccd0: [chronik-client] Dockerfile back to latest.

Revision Contents
Changeset List

Diff 51703

View Options

chronik-client.Dockerfile

Loading...