Merge pull request #162 from SubstraFoundation/grpc-nolimit

No limit on grpc client
Substra · Feb 28, 2020 · d52acd1 · d52acd1
2 parents b20d5f3 + f416651
commit d52acd1
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 7 deletions.
diff --git a/backend/backend/settings/deps/ledger.py b/backend/backend/settings/deps/ledger.py
@@ -141,8 +141,9 @@ def update_client_with_discovery(client, discovery_results):
                 peer.init_with_bundle({
                     'url': url,
                     'grpcOptions': {
-                        'grpc-max-send-message-length': 15,
-                        'grpc.ssl_target_name_override': peer_info['endpoint'].split(':')[0]
+                        'grpc.ssl_target_name_override': peer_info['endpoint'].split(':')[0],
+                        'grpc.max_send_message_length': -1,
+                        'grpc.max_receive_message_length': -1
                     },
                     'tlsCACerts': {'path': tls_root_cert.name},
                     'clientKey': {'path': LEDGER['peer']['clientKey']},  # use peer creds (mutual tls)
@@ -164,8 +165,9 @@ def update_client_with_discovery(client, discovery_results):
         orderer.init_with_bundle({
             'url': f"{orderer_info[0]['host']}:{orderer_info[0]['port']}",
             'grpcOptions': {
-                'grpc-max-send-message-length': 15,
-                'grpc.ssl_target_name_override': orderer_info[0]['host']
+                'grpc.ssl_target_name_override': orderer_info[0]['host'],
+                'grpc.max_send_message_length': -1,
+                'grpc.max_receive_message_length': -1
             },
             'tlsCACerts': {'path': tls_root_cert.name},
             'clientKey': {'path': LEDGER['peer']['clientKey']},  # use peer creds (mutual tls)

diff --git a/charts/substra-backend/Chart.yaml b/charts/substra-backend/Chart.yaml
@@ -1,7 +1,7 @@
 apiVersion: v1
 name: substra-backend
 home: https://substra.org/
-version: 1.0.0-alpha.19
+version: 1.0.0-alpha.20
 description: Main package for Substra
 icon: https://avatars1.githubusercontent.com/u/38098422?s=200&v=4
 sources:

diff --git a/charts/substra-backend/templates/configmap-server.yaml b/charts/substra-backend/templates/configmap-server.yaml
@@ -30,8 +30,9 @@ data:
         "clientKey": "/var/hyperledger/tls/client/pair/tls.key",
         "clientCert": "/var/hyperledger/tls/client/pair/tls.crt",
         "grpcOptions": {
-          "grpc-max-send-message-length": 15,
-          "grpc.ssl_target_name_override": "{{ .Values.peer.host }}"
+          "grpc.ssl_target_name_override": "{{ .Values.peer.host }}",
+          "grpc.max_send_message_length": -1,
+          "grpc.max_receive_message_length": -1
         }
       }
     }