tree cbdaaa5d1ed081ee0699f4ffe1dee3685d4acd16
parent a94b8f4c7e9094c2d3784cfc1a969446feb704c1
author Raphael Kubo da Costa <raphael.kubo.da.costa@intel.com> 1585670915 +0200
committer Raphael Kubo da Costa <raphael.kubo.da.costa@intel.com> 1585670915 +0200

cas: Stop assuming BatchUpdateBlobsResponse_Response.Status is never nil

The code was always accessing Status.Code, but Status itself can be nil
depending on the gRPC implementation used by the remote end.

Namely, https://github.com/grpc/grpc-go/pull/2929, present since gRPC
1.28.0, causes status.Convert(nil).Proto() to return nil rather than an
empty Status with code set to OK.

Make the code in Upload() be more resilient by using status.FromProto(),
GetStatus() and Code() rather than accessing members directly.

Change-Id: I051bf9cbb3dc70292243ad84f1109eb30554b93d