Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add remote gRPC option for storage plugin #3383

Merged
merged 11 commits into from Dec 10, 2021
Merged

Add remote gRPC option for storage plugin #3383

merged 11 commits into from Dec 10, 2021

Commits on Nov 10, 2021

  1. Add remote gRPC option for storage plugin 

    Adds the option to host a gRPC storage API on a remote endpoint
using regular gRPC. Previously the plugin system only supported
local socket connections through the go-hashicorp plugin system.

Signed-off-by: Matvey Arye <mat@timescale.com>
    @cevian
    cevian committed Nov 10, 2021
    Configuration menu
    Copy the full SHA
    c86a3de View commit details
    Browse the repository at this point in the history

Commits on Nov 13, 2021

  1. Change remote gRPC configuration parameters 

    Use the tlscfg package for the TLS configuration and change the
cli option prefix from `grpc-storage-plugin` to `grpc-storage`.

Signed-off-by: Matvey Arye <mat@timescale.com>
    @cevian
    cevian committed Nov 13, 2021
    Configuration menu
    Copy the full SHA
    d75927e View commit details
    Browse the repository at this point in the history

  2. Add tests for remote gRPC 

    We also created a RegisterServer method on StorageGRPCPlugin to
register be able to easily create remote GRPC servers.

Signed-off-by: Matvey Arye <mat@timescale.com>
    @cevian
    cevian committed Nov 13, 2021
    Configuration menu
    Copy the full SHA
    3714ebe View commit details
    Browse the repository at this point in the history

  3. Fixing naming brought up in PR review 

    Also ran `make fmt`

Signed-off-by: Matvey Arye <mat@timescale.com>
    @cevian
    cevian committed Nov 13, 2021
    Configuration menu
    Copy the full SHA
    fb3e2bc View commit details
    Browse the repository at this point in the history

Commits on Nov 15, 2021

  1. Fixing code nits for GRPC remote storage 

    Signed-off-by: Matvey Arye <mat@timescale.com>
    @cevian
    cevian committed Nov 15, 2021
    Configuration menu
    Copy the full SHA
    44a7ad6 View commit details
    Browse the repository at this point in the history

  2. Refactor to combine the memory storage plugin 

    Signed-off-by: Matvey Arye <mat@timescale.com>
    @cevian
    cevian committed Nov 15, 2021
    Configuration menu
    Copy the full SHA
    903fe1a View commit details
    Browse the repository at this point in the history

Commits on Nov 17, 2021

  1. Merge branch 'master' into add_remote_grpc

    @yurishkuro
    yurishkuro committed Nov 17, 2021
    Configuration menu
    Copy the full SHA
    5d06067 View commit details
    Browse the repository at this point in the history

Commits on Dec 9, 2021

  1. Fix nits in PR review 

    Signed-off-by: Matvey Arye <mat@timescale.com>
    @cevian
    cevian committed Dec 9, 2021
    Configuration menu
    Copy the full SHA
    539dbca View commit details
    Browse the repository at this point in the history

Commits on Dec 10, 2021

  1. Add test for grpc memory plugin 

    Signed-off-by: Matvey Arye <mat@timescale.com>
    @cevian
    cevian committed Dec 10, 2021
    Configuration menu
    Copy the full SHA
    c4a1b01 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'main' into add_remote_grpc

    @yurishkuro
    yurishkuro committed Dec 10, 2021
    Configuration menu
    Copy the full SHA
    f0e8785 View commit details
    Browse the repository at this point in the history

  3. Add test for Close 

    Signed-off-by: Yuri Shkuro <github@ysh.us>
    @yurishkuro
    yurishkuro committed Dec 10, 2021
    Configuration menu
    Copy the full SHA
    5c24f6e View commit details
    Browse the repository at this point in the history