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

Watch and provide startup logs #15988

Closed
wants to merge 44 commits into from
Closed

Watch and provide startup logs #15988

wants to merge 44 commits into from

Conversation

sparkoo
Copy link
Member

@sparkoo sparkoo commented Feb 11, 2020

What does this PR do?

Watch the logs of all containers in workspace pod during workspace startup. Feature is disabled by default. It can be enabled on workspace startup REST API with new query parameter debug-workspace-start. If it is set to true, logs will be watched and printed during the startup.

so API call looks like this:
<che-host>/api/workspace/{id}/runtime?debug-workspace-start=true

TODO:

  • create config properties, most important for completely disable the feature, should be default true or false ??? configurable with devfile attribute
  • I've observed some double logs of same container, investigate that or create a bug
  • write tests
  • fix CI
  • better log messages (fix @metlos comments)

What issues does this PR fix or reference?

#15983

Docs PR

this adds an option on che-server API, docs should be part of UI based tasks #16050 #16063

@che-bot che-bot added status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. kind/enhancement A feature request - must adhere to the feature request template. labels Feb 11, 2020
Signed-off-by: Michal Vala <mvala@redhat.com>
Signed-off-by: Michal Vala <mvala@redhat.com>
Signed-off-by: Michal Vala <mvala@redhat.com>
Signed-off-by: Michal Vala <mvala@redhat.com>
Signed-off-by: Michal Vala <mvala@redhat.com>
Signed-off-by: Michal Vala <mvala@redhat.com>
…sted in

Signed-off-by: Michal Vala <mvala@redhat.com>
Signed-off-by: Michal Vala <mvala@redhat.com>
…ool... can this thing get any better?

Signed-off-by: Michal Vala <mvala@redhat.com>
Signed-off-by: Michal Vala <mvala@redhat.com>
Signed-off-by: Michal Vala <mvala@redhat.com>
…w responsible for handling just single log messages

Signed-off-by: Michal Vala <mvala@redhat.com>
Signed-off-by: Michal Vala <mvala@redhat.com>
Signed-off-by: Michal Vala <mvala@redhat.com>
# Conflicts:
#	infrastructures/kubernetes/src/main/java/org/eclipse/che/workspace/infrastructure/kubernetes/KubernetesInternalRuntime.java
Signed-off-by: Michal Vala <mvala@redhat.com>
Signed-off-by: Michal Vala <mvala@redhat.com>
…ethod

Signed-off-by: Michal Vala <mvala@redhat.com>
Signed-off-by: Michal Vala <mvala@redhat.com>
Signed-off-by: Michal Vala <mvala@redhat.com>
Signed-off-by: Michal Vala <mvala@redhat.com>
Signed-off-by: Michal Vala <mvala@redhat.com>
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@sparkoo sparkoo deleted the logs branch February 28, 2020 07:49
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Feb 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement A feature request - must adhere to the feature request template. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants