[Notes] [Git][BuildStream/buildstream][bst-1.2] 3 commits: Log not-found

c09852c9
by Benjamin Schubert at 2018-08-29T16:58:43Z
```
Log not-found objects in the cache as SKIPPED
```

cba9208f

by Benjamin Schubert at 2018-08-29T16:58:51Z

Add tests that not-found objects in cache are shown as SKIPPED

4a644deb

by Tristan Van Berkom at 2018-08-29T17:24:18Z

Merge branch 'bschubert/backport-log-missed-cache' into 'bst-1.2'

Bschubert/backport log missed cache

See merge request BuildStream/buildstream!752

@@ -249,6 +249,13 @@ class CASCache(ArtifactCache):
                  if e.code() != grpc.StatusCode.NOT_FOUND:
                      raise ArtifactError("Failed to pull artifact {}: {}".format(
                          element._get_brief_display_key(), e)) from e
 +                else:
 +                    self.context.message(Message(
 +                        None,
 +                        MessageType.SKIPPED,
 +                        "Remote ({}) does not have {} cached".format(
 +                            remote.spec.url, element._get_brief_display_key())
 +                    ))
          return False

@@ -338,3 +338,22 @@ def test_pull_missing_blob(cli, tmpdir, datafiles):
          # Assert that no artifacts were pulled
          assert len(result.get_pulled_elements()) == 0
++
++
 +@pytest.mark.datafiles(DATA_DIR)
 +def test_pull_missing_notifies_user(caplog, cli, tmpdir, datafiles):
 +    project = os.path.join(datafiles.dirname, datafiles.basename)
 +    caplog.set_level(1)
++
 +    with create_artifact_share(os.path.join(str(tmpdir), 'artifactshare')) as share:
++
 +        cli.configure({
 +            'artifacts': {'url': share.repo}
 +        })
 +        result = cli.run(project=project, args=['build', 'target.bst'])
++
 +        result.assert_success()
 +        assert not result.get_pulled_elements(), \
 +            "No elements should have been pulled since the cache was empty"
++
 +        assert "SKIPPED Remote ({}) does not have".format(share.repo) in result.stderr

[Notes] [Git][BuildStream/buildstream][bst-1.2] 3 commits: Log not-found objects in the cache as SKIPPED