[chore] Remove gogoproto annotations from trace_storage.proto and dependency_storage.proto (jaegertracing#6819)

<!--
!! Please DELETE this comment before posting.
We appreciate your contribution to the Jaeger project! 👋🎉
-->

## Which problem is this PR solving?
- Towards jaegertracing#6789

## Description of the changes
- This PR removes the gogoproto annotations from `trace_storage.proto`
and `dependency_storage.proto` to avoid introducing a dependency on gogo
for users. Instead, it uses a similar patching mechanism to api_v3 to
add the annotations when generating the *.pb.go files.
 
## How was this change tested?
- CI

## Checklist
- [x] I have read
https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
- [x] I have signed all commits
- [x] I have added unit tests for the new functionality
- [x] I have run lint and test steps successfully
  - for `jaeger`: `make lint test`
  - for `jaeger-ui`: `npm run lint` and `npm run test`

---------

Signed-off-by: Mahad Zaryab <[email protected]>

Author: mahadzaryab1

internal/storage/v2/grpc/dependency_storage.proto

@@ -9,15 +9,9 @@ option go_package = "storage";
 
 message GetDependenciesRequest {
   // start_time is the start of the time interval to search for the dependencies.
-  google.protobuf.Timestamp start_time = 1 [
-    (gogoproto.stdtime) = true,
-    (gogoproto.nullable) = false
-  ];
+  google.protobuf.Timestamp start_time = 1;
   // end_time is the end of the time interval to search for the dependencies.
-  google.protobuf.Timestamp end_time = 2 [
-    (gogoproto.stdtime) = true,
-    (gogoproto.nullable) = false
-  ];
+  google.protobuf.Timestamp end_time = 2;
 }
 
 // Dependency represents a relationship between two services.
@@ -36,7 +30,7 @@ message Dependency {
 }
 
 message GetDependenciesResponse {
-  repeated Dependency dependencies = 1 [(gogoproto.nullable) = false];
+  repeated Dependency dependencies = 1;
 }
 
 service DependencyReader {

internal/storage/v2/grpc/trace_storage.proto

@@ -2,7 +2,6 @@ syntax = "proto3";
 
 package jaeger.storage.v2;
 
-import "gogoproto/gogo.proto";
 import "google/protobuf/duration.proto";
 import "google/protobuf/timestamp.proto";
 import "opentelemetry/proto/trace/v1/trace.proto";
@@ -17,12 +16,12 @@ message GetTraceParams {
   // start_time is the start of the time interval to search for the trace_id.
   //
   // This field is optional.
-  google.protobuf.Timestamp start_time = 2 [(gogoproto.stdtime) = true];
+  google.protobuf.Timestamp start_time = 2;
 
   // end_time is the end of the time interval to search for the trace_id.
   //
   // This field is optional.
-  google.protobuf.Timestamp end_time = 3 [(gogoproto.stdtime) = true];
+  google.protobuf.Timestamp end_time = 3;
 }
 
 // GetTracesRequest represents a request to retrieve multiple traces.
@@ -49,13 +48,13 @@ message GetOperationsRequest {
   // spans generated in a particular context.
   //
   // This field is optional.
-  opentelemetry.proto.trace.v1.Span.SpanKind span_kind = 2;
+  string span_kind = 2;
 }
 
 // Operation contains information about an operation for a given service.
 message Operation {
   string name = 1;
-  opentelemetry.proto.trace.v1.Span.SpanKind span_kind = 2;
+  string span_kind = 2;
 }
 
 // GetOperationsResponse represents the response for GetOperationsRequest.
@@ -98,22 +97,10 @@ message TraceQueryParameters {
   string service_name = 1;
   string operation_name = 2;
   repeated KeyValue attributes = 3;
-  google.protobuf.Timestamp start_time_min = 4 [
-    (gogoproto.stdtime) = true,
-    (gogoproto.nullable) = false
-  ];
-  google.protobuf.Timestamp start_time_max = 5 [
-    (gogoproto.stdtime) = true,
-    (gogoproto.nullable) = false
-  ];
-  google.protobuf.Duration duration_min = 6 [
-    (gogoproto.stdduration) = true,
-    (gogoproto.nullable) = false
-  ];
-  google.protobuf.Duration duration_max = 7 [
-    (gogoproto.stdduration) = true,
-    (gogoproto.nullable) = false
-  ];
+  google.protobuf.Timestamp start_time_min = 4;
+  google.protobuf.Timestamp start_time_max = 5;
+  google.protobuf.Duration duration_min = 6;
+  google.protobuf.Duration duration_max = 7;
   int32 search_depth = 8;
 }
 
@@ -133,14 +120,8 @@ message FindTracesRequest {
 // It is meant as a rough boundary and may not be populated in all cases.
 message FoundTraceID {
   bytes trace_id = 1;
-  google.protobuf.Timestamp start = 2 [
-    (gogoproto.stdtime) = true,
-    (gogoproto.nullable) = false
-  ];
-  google.protobuf.Timestamp end = 3 [
-    (gogoproto.stdtime) = true,
-    (gogoproto.nullable) = false
-  ];
+  google.protobuf.Timestamp start = 2;
+  google.protobuf.Timestamp end = 3;
 }
 
 // FindTraceIDsResponse represents the response for FindTracesRequest.