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

feat: [TKC-2882] stream service logs #6052

Open
wants to merge 26 commits into
base: main
Choose a base branch
from
Open

feat: [TKC-2882] stream service logs #6052

wants to merge 26 commits into from

Conversation

vsukhin
Copy link
Collaborator

@vsukhin vsukhin commented Nov 25, 2024 •
edited
Loading

Pull request description

  • add api endpoints to stream service logs using websockets and sse
  • add cli command parameters to watch service logs
  • add new client method
  • add grpc method to stream service logs
  • add proto method

Checklist (choose whats happened)

  • breaking change! (describe)
  • tested locally
  • tested on cluster
  • added new dependencies
  • updated the docs
  • added a test

Breaking changes

Changes

Fixes

@vsukhin
feat: api methods for service logs
bd39964 
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin
fix: client for get service logs
2c77962 
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin
fix: change log
c71f5c8 
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin
fix: disable hints
3e70a83 
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin
fix: routing
aa9acb3 
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin
fix: show service logs
68fe09f 
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin
fix: check service name
c55187f 
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin
fix: check service name
93b7ac6 
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin
fix: log comment
cf2dd76 
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin
fix: check for testworkflow service
fff63c0 
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin vsukhin marked this pull request as ready for review November 27, 2024 11:23
@vsukhin vsukhin requested a review from a team as a code owner November 27, 2024 11:23
@vsukhin
fix: friendly error
c7bed19 
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin
fix: add spinner
f34e5d1 
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin
feat: proto for service notifications
07ce308 
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin
feat: add cloud grpc method for server notifications
9c03fe4 
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin
fix: change timeeout
9d85b35 
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin
fix: waiting for service pod
57db7a0 
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin
fix: typo
cccfcb7 
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin
fix: waiting for service pod
c4658c5 
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin
fix: add service name check
3768550 
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin
fix: adjust help
c2fd448 
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin
fix: add method to parallel step
b46bb41 
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin
fix: use retry library
59bdce3 
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin
fix: rename const
4224930 
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin
fix: 0 attempts
bcda47f 
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin
fix: use option
97b5bd3 
Signed-off-by: Vladislav Sukhin <[email protected]>
@vsukhin
fix: remove ctx
3e2c45f 
Signed-off-by: Vladislav Sukhin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@WitoDelnat WitoDelnat WitoDelnat left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vsukhin @WitoDelnat