You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@buildstream.apache.org by GitBox <gi...@apache.org> on 2022/08/18 07:08:29 UTC

[GitHub] [buildstream] gtristan commented on pull request #1725: _artifactcache/cascache.py: Only check push capabilities for push remotes

gtristan commented on PR #1725:
URL: https://github.com/apache/buildstream/pull/1725#issuecomment-1219115804

   > While this is a good thing to have, I think it doesn't fix #1724.
   > 
   > In the case of #1724, we still have an HTTP 404 error which should map to UNIMPLEMENTED. For some reason (likely a bug in grpcio) it is being mapped to some other error (the "stream removed" thing)
   
   I've updated the patch to additionally only check for these push capabilities on push remotes.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@buildstream.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org