Skip to content
This repository has been archived by the owner on Oct 9, 2023. It is now read-only.

add server reflection #19

Merged
merged 1 commit into from
Nov 20, 2019
Merged

Conversation

honnix
Copy link
Member

@honnix honnix commented Nov 20, 2019

With server reflection, it would be easier to interact with GRPC service, e.g.

$ grpcurl <host:port> list datacatalog.DataCatalog
datacatalog.DataCatalog.AddTag
datacatalog.DataCatalog.CreateArtifact
datacatalog.DataCatalog.CreateDataset
datacatalog.DataCatalog.GetArtifact
datacatalog.DataCatalog.GetDataset
datacatalog.DataCatalog.ListArtifacts

or

$ grpcurl -d '{"dataset": {"project": "foo", "domain": "bar", "name": "hello", "version": "1"}}' \
  <host:port> datacatalog.DataCatalog.ListArtifacts

@chanadian
Copy link
Contributor

chanadian commented Nov 20, 2019

This is quite handy, thanks for adding it. Personally, I have been using https://github.com/uw-labs/bloomrpc to quickly connect and make grpc calls through a UI. This is 👍 for cli tooling.

@chanadian chanadian merged commit b365a10 into flyteorg:master Nov 20, 2019
@honnix honnix deleted the server-reflection branch November 20, 2019 19:48
wild-endeavor pushed a commit to flyteorg/flyteadmin that referenced this pull request Nov 25, 2019
The same as flyteorg/datacatalog#19

With [server reflection](https://github.com/grpc/grpc-go/blob/master/Documentation/server-reflection-tutorial.md), it would be easier to interact with GRPC service using e.g. `grpcurl`.

flyteadmin does come with HTTP endpoint as well, but interacting with GRPC directly makes it more handy.
chanadian pushed a commit that referenced this pull request Feb 10, 2020
pingsutw pushed a commit to pingsutw/flyte-monorepo that referenced this pull request Apr 4, 2023
The same as flyteorg/datacatalog#19

With [server reflection](https://github.com/grpc/grpc-go/blob/master/Documentation/server-reflection-tutorial.md), it would be easier to interact with GRPC service using e.g. `grpcurl`.

flyteadmin does come with HTTP endpoint as well, but interacting with GRPC directly makes it more handy.
eapolinario pushed a commit that referenced this pull request Sep 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants