Skip to content
Snippets Groups Projects
  1. Sep 28, 2022
  2. Sep 27, 2022
  3. Sep 15, 2022
  4. Sep 14, 2022
  5. Sep 13, 2022
  6. Sep 12, 2022
    • David Douard's avatar
      Reorganize configuration of the http server building · f2945381
      David Douard authored
      rework the configuration used by `make_app()` so that one can start the
      http server usig an existing grpc server by a setting the configuration
      file accordingly.
      
      This make the `make_app()` function only accept one config structure
      (dict), which structure would be:
      
        graph:
          cls: local
          grpc_server: # config for spawn_java_grpc_server()
            ...
          http_rpc_server: # config for GraphServerApp()
            ...
      
      or:
      
        graph:
          cls: remote
          url: # host:port of the running grpc server
            ...
          http_rpc_server: # config for GraphServerApp()
            ...
      
      This also comes wit a few other refactorings:
      
      - rename swh/graph/rpc/ as swh/graph/grpc/ to prevent confusion with the
        (http) rpc server.
      - similarly, rename swh/graph/rpc_server.py as swh/graph/grpc_server.py
      - and rename swh/graph/http_server.py as swh/graph/http_rpc_server.py
      - remove the "graph" config section from the grpc_server config
        structure (it had only the "path" config entry).
      f2945381
    • David Douard's avatar
  7. Sep 08, 2022
  8. Sep 06, 2022
    • vlorentz's avatar
      Return HTTP 503 on AioRpcError · fcca8787
      vlorentz authored
      AioRpcError exceptions used to trigger the default error handler
      of SWH's RPC framework, which msgpacks the exception.
      
      Under some circumstances, this caused the msgpack to hit
      recursion limits, and logged very large errors to Sentry.
      
      Additionally, SimpleTraversalView now calls wait_for_connection()
      before returning 200; which allows catching some gRPC errors early
      and returning 503 instead.
      
      This does not work all the time though; gRPC errors occuring in the
      middle of the stream still raise `ChunkedEncodingError` on the client
      side, but there is not much we can do about that.
      v1.0.2
      fcca8787
    • vlorentz's avatar
      Kill GraphServerProcess on test teardown · 4a7f3f76
      vlorentz authored
      There is no reason to terminate it nicely.
      4a7f3f76
    • vlorentz's avatar
      conftest: Refactor GraphServerProcess to be more flexible · 81d3deb2
      vlorentz authored
      1. queue is now encapsulated
      2. object in the queue is now a dict instead of a tuple (to allow
         adding more keys easily in the future + for readability)
      
      This will be used in a future commit, to get the queue's object
      twice from different functions.
      81d3deb2
  9. Sep 05, 2022
  10. Aug 30, 2022
  11. Jun 30, 2022
  12. Jun 29, 2022
  13. Jun 24, 2022
  14. Jun 23, 2022
Loading