[Notes] [Git][BuildStream/buildstream][master] 2 commits: cascache.py: Raise ArtifactError on grpc error

[Tristan Maat <gitlab mg gitlab com>]

Title: GitLab

Tristan Maat pushed to branch master at BuildStream / buildstream

Commits:

• fb222ba1
  by Josh Smith at 2018-07-27T08:50:45Z

  cascache.py: Raise ArtifactError on grpc error
  
  This aims to fix issue #502.
  

• 48916b8a
  by Tristan Maat at 2018-07-27T10:22:41Z

  Merge branch 'Qinusty/502' into 'master'
  
  cascache.py: Raise ArtifactError on grpc error
  
  Closes #502
  
  See merge request BuildStream/buildstream!576

1 changed file:

• buildstream/_artifactcache/cascache.py

Changes:

buildstream/_artifactcache/cascache.py

---

...	...	@@ -240,7 +240,8 @@ class CASCache(ArtifactCache):
240	240
241	241	except grpc.RpcError as e:
242	242	if e.code() != grpc.StatusCode.NOT_FOUND:
243		- raise
	243	+ raise ArtifactError("Failed to pull artifact {}: {}".format(
	244	+ element._get_brief_display_key(), e)) from e
244	245
245	246	return False
246	247
...	...	@@ -285,6 +286,7 @@ class CASCache(ArtifactCache):
285	286
286	287	except grpc.RpcError as e:
287	288	if e.code() != grpc.StatusCode.NOT_FOUND:
	289	+ # Intentionally re-raise RpcError for outer except block.
288	290	raise
289	291
290	292	missing_blobs = {}

—
View it on GitLab.
You're receiving this email because of your account on gitlab.com. If you'd like to receive fewer emails, you can adjust your notification settings.