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

fix: register gogo codec #1

Merged
merged 1 commit into from
Mar 30, 2022
Merged

Conversation

alexmt
Copy link

@alexmt alexmt commented Mar 30, 2022

Signed-off-by: Alexander Matyushentsev AMatyushentsev@gmail.com

Register gogo protobuf grpc codec to fix proto deserialization issue

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
@@ -0,0 +1,27 @@
package grpc
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome file name :D

@leoluz leoluz merged commit da1b9f7 into leoluz:update-grpc Mar 30, 2022
leoluz added a commit that referenced this pull request Apr 22, 2022
* chore: update grpc POC

Signed-off-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>

* fix: register gogo codec (#1)

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>

* Remove gogo annotations from application.proto

Signed-off-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>

* Fix unit tests

Signed-off-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>

* Fix e2e test

Signed-off-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>

* Fix lint

Signed-off-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>

* Fix lint

Signed-off-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>

* Fix unit-test

Signed-off-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>

* Fix LogEntry.Last required field not populated

Signed-off-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>

* Fix LogEntry required fields

Signed-off-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>

* Fix get log content

Signed-off-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>

* Fix app actions list

Signed-off-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>

* Fix ApplicationPodLogsQuery

Signed-off-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>

* Fix RunResourceAction

Signed-off-by: Leonardo Luz Almeida <leonardo_almeida@intuit.com>

Co-authored-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants