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 2021/08/02 17:27:10 UTC
[GitHub] [buildstream] jjardon opened a new pull request #1489: requirements/requirements.in: It depends on grpc >= 1.37
jjardon opened a new pull request #1489:
URL: https://github.com/apache/buildstream/pull/1489
with older version you will get errors like this accessing the CAS:
[--:--:--][][] WARNING Failed to initialize remote https://freedesktop-sdk-cache.codethink.co.uk:11001: Received http2 header with status: 404
--
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
[GitHub] [buildstream] juergbi commented on pull request #1489: requirements/requirements.in: It depends on grpc >= 1.37
Posted by GitBox <gi...@apache.org>.
juergbi commented on pull request #1489:
URL: https://github.com/apache/buildstream/pull/1489#issuecomment-891217077
Is grpcio 1.37 the minimum version you've determined or is this simply a known working version? In CI even for BuildStream master we still have a minimum of 1.10 declared and we're testing with 1.34 in CI. And for the C++ library we even still test with gRPC 1.16 (Debian 10), gRPC 1.25 (CentOS 7.7 and Ubuntu 18.04) and gRPC 1.26 (Fedora 32/33), as far as I can tell.
gRPC 1.37 is fairly recent. Can this be an issue on older distros or is this a non-issue as everyone is pulling it from PyPI and there are also no indirect dependency issues?
In any case, any potential issues with old grpcio versions likely also apply to BuildStream master. And thus, this should first be fixed in master before a backport to bst-1.
--
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
[GitHub] [buildstream] gtristan closed pull request #1489: requirements/requirements.in: It depends on grpc >= 1.30
Posted by GitBox <gi...@apache.org>.
gtristan closed pull request #1489:
URL: https://github.com/apache/buildstream/pull/1489
--
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
[GitHub] [buildstream] jjardon commented on pull request #1489: requirements/requirements.in: It depends on grpc >= 1.37
Posted by GitBox <gi...@apache.org>.
jjardon commented on pull request #1489:
URL: https://github.com/apache/buildstream/pull/1489#issuecomment-891565846
PR for master here (1.34 is already in requirements.txt): https://github.com/apache/buildstream/pull/1490
--
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
[GitHub] [buildstream] gtristan commented on pull request #1489: requirements/requirements.in: It depends on grpc >= 1.30
Posted by GitBox <gi...@apache.org>.
gtristan commented on pull request #1489:
URL: https://github.com/apache/buildstream/pull/1489#issuecomment-903494465
Rebased this in #1497 and merging there
--
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
[GitHub] [buildstream] jjardon edited a comment on pull request #1489: requirements/requirements.in: It depends on grpc >= 1.37
Posted by GitBox <gi...@apache.org>.
jjardon edited a comment on pull request #1489:
URL: https://github.com/apache/buildstream/pull/1489#issuecomment-891353082
To be fair I'm not aware of the exact version that fix the problem but I know 1.37 work correctly. Also I know is definitely a problem in fedora 34, which has grpc 1.26
What do you test in the CI exactly? We are using buildbox-casd as remote cache
--
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
[GitHub] [buildstream] nanonyme commented on pull request #1489: requirements/requirements.in: It depends on grpc >= 1.34
Posted by GitBox <gi...@apache.org>.
nanonyme commented on pull request #1489:
URL: https://github.com/apache/buildstream/pull/1489#issuecomment-891816355
The Fedora version (1.26.0) at least is known not to work correctly with newer caches.
--
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
[GitHub] [buildstream] nanonyme commented on pull request #1489: requirements/requirements.in: It depends on grpc >= 1.34
Posted by GitBox <gi...@apache.org>.
nanonyme commented on pull request #1489:
URL: https://github.com/apache/buildstream/pull/1489#issuecomment-891815463
@juergbi there's a requirement bump induced by RAAPI-based remote cache.
--
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
[GitHub] [buildstream] jjardon edited a comment on pull request #1489: requirements/requirements.in: It depends on grpc >= 1.34
Posted by GitBox <gi...@apache.org>.
jjardon edited a comment on pull request #1489:
URL: https://github.com/apache/buildstream/pull/1489#issuecomment-891353082
To be fair I'm not aware of the exact version that fix the problem but I know 1.37 work correctly. Also I know is definitely a problem in fedora 34, which has grpc 1.26
What do you test in the CI exactly? We are using buildbox-casd as remote cache.
Maybe is because you require 1.34 in master? I will change this PR to relax the requirement to that version
--
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
[GitHub] [buildstream] jjardon edited a comment on pull request #1489: requirements/requirements.in: It depends on grpc >= 1.37
Posted by GitBox <gi...@apache.org>.
jjardon edited a comment on pull request #1489:
URL: https://github.com/apache/buildstream/pull/1489#issuecomment-891353082
To be fair I'm not aware of the exact version that fix the problem but I know 1.37 work correctly. Also I know is definitely a problem in fedora 34, which has grpc 1.26
What do you test in the CI exactly? We are using buildbox-casd as remote cache . Maybe is because you require 1.34 in master? I will change this PR to relax the requirement to that version
--
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
[GitHub] [buildstream] jjardon commented on pull request #1489: requirements/requirements.in: It depends on grpc >= 1.37
Posted by GitBox <gi...@apache.org>.
jjardon commented on pull request #1489:
URL: https://github.com/apache/buildstream/pull/1489#issuecomment-891353082
To be fair I'm not aware of the exact version that fix the problem but I know 1.37 work correctly. Also I know is definitely a problem in fedora 34, which has grpc 1.26
--
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
[GitHub] [buildstream] jjardon commented on pull request #1489: requirements/requirements.in: It depends on grpc >= 1.30
Posted by GitBox <gi...@apache.org>.
jjardon commented on pull request #1489:
URL: https://github.com/apache/buildstream/pull/1489#issuecomment-895094112
As we do not know exactly the first version that works, I have further relaxed the requirement to 1.30, which is available in more distros (like debian)
--
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