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

Collect pod logs #212

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Collect pod logs #212

wants to merge 10 commits into from

Conversation

kaplanelad
Copy link
Contributor

What type of PR is this?
Backend Feature
UI Feature

What this PR does / why we need it:
Collect pod container logs

Which issue(s) this PR fixes (if exists):

Fixes #195

@kaplanelad
Copy link
Contributor Author

Hey @ortz,
@liorbd and I added the option to collect and see pods/containers in the UI.
Once you will approved it I'll to the changed in the helm chart repo.

See the results
image

image

Copy link
Contributor

@ortz ortz left a comment

Choose a reason for hiding this comment

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

hi @kaplanelad , some comments.. other than that, looks good.

watcher/kubernetes/pods.go Outdated Show resolved Hide resolved
watcher/kubernetes/pods.go Outdated Show resolved Hide resolved
watcher/kubernetes/pods.go Outdated Show resolved Hide resolved
watcher/kubernetes/pods.go Show resolved Hide resolved
@kaplanelad
Copy link
Contributor Author

@liorbd can you please resolve the ui/client/src/components/EventsView/EventViewSelector.js conflicts?

@liorbd liorbd force-pushed the feature/195-collect-pods-logs-v3 branch from 3a5e360 to 6307e44 Compare April 26, 2020 06:43
@liorbd
Copy link
Contributor

liorbd commented Apr 26, 2020

@kaplanelad done

@kaplanelad kaplanelad added the lgtm label May 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Collect pod logs
3 participants