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.

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

Migrate telemetry ingestion service #1979

Open
2 tasks
Tracked by #1525
tltran-legion opened this issue Jun 6, 2022 · 1 comment
Open
2 tasks
Tracked by #1525

Migrate telemetry ingestion service #1979

tltran-legion opened this issue Jun 6, 2022 · 1 comment
Assignees
Labels
area: analytics Analytics, telemetry, monitoring, observability area: online Related to online or distributed functionalities priority: P1 High type: enhancement Improvement of existing feature
Milestone

Comments

@tltran-legion
Copy link

tltran-legion commented Jun 6, 2022

Summary

Goal is to migrate the Telemetry service to OpenAPI using the Code Generation framework.

Current State

At the moment, the Telemetry service is using gRPC we need it to follow the defined new API convention.

Work Items

  • Using the Code Generation Framework to generate the service skeleton.
  • Port of the actual business logic of the service into the new skeleton.
@tltran-legion tltran-legion added area: online Related to online or distributed functionalities type: enhancement Improvement of existing feature labels Jun 6, 2022
@ndubois-legion ndubois-legion added this to the M3 milestone Jun 7, 2022
@ndubois-legion ndubois-legion added the area: analytics Analytics, telemetry, monitoring, observability label Jun 7, 2022
@mad-legion mad-legion changed the title Migrate telemetry service Migrate telemetry ingestion service Jun 7, 2022
@mad-legion
Copy link

mad-legion commented Jun 7, 2022

The telemetry ingestion service already supports http and grpc.
Most the work will be to move the rust telemetry client to use http (the unreal client is already using http).
Not sure OpenAPI will do much good here as part of the protocol is binary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: analytics Analytics, telemetry, monitoring, observability area: online Related to online or distributed functionalities priority: P1 High type: enhancement Improvement of existing feature
Projects
None yet
Development

No branches or pull requests

5 participants