diff options
author | Josh Smith <qinusty@gmail.com> | 2018-09-07 15:36:43 +0100 |
---|---|---|
committer | Josh Smith <qinusty@gmail.com> | 2018-09-19 10:25:21 +0100 |
commit | ca811a4d5905b61862c230e17925ab9152a78b16 (patch) | |
tree | a9a647cff50c300f1634848a6061f94916c1f3ad /buildstream/_artifactcache | |
parent | 72b5902157316e173de2eec5b3a2772283eec3c7 (diff) | |
download | buildstream-ca811a4d5905b61862c230e17925ab9152a78b16.tar.gz |
Rework Skipped usage
The SKIPPED message type is now used to indicate the end of a task which
was successful without having to perform the given task.
This overhauls the use of `Queue.done()` and therefore queues do not
need to provide a processed/skipped return value from `done()`. Instead
this is replaced with the action of raising a `SkipJob` exception from
within `Queue.process()`.
Diffstat (limited to 'buildstream/_artifactcache')
-rw-r--r-- | buildstream/_artifactcache/cascache.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/buildstream/_artifactcache/cascache.py b/buildstream/_artifactcache/cascache.py index 9cf83a222..1a48c4065 100644 --- a/buildstream/_artifactcache/cascache.py +++ b/buildstream/_artifactcache/cascache.py @@ -253,7 +253,7 @@ class CASCache(ArtifactCache): else: self.context.message(Message( None, - MessageType.SKIPPED, + MessageType.INFO, "Remote ({}) does not have {} cached".format( remote.spec.url, element._get_brief_display_key()) )) @@ -344,7 +344,7 @@ class CASCache(ArtifactCache): else: self.context.message(Message( None, - MessageType.SKIPPED, + MessageType.INFO, "Remote ({}) already has {} cached".format( remote.spec.url, element._get_brief_display_key()) )) |