Skip to content
This repository has been archived by the owner on Sep 5, 2019. It is now read-only.

Better story for Build logs #9

Open
mattmoor opened this issue Feb 4, 2018 · 7 comments
Open

Better story for Build logs #9

mattmoor opened this issue Feb 4, 2018 · 7 comments

Comments

@mattmoor
Copy link
Member

mattmoor commented Feb 4, 2018

Currently accessing Build logs is a poor experience in (at least) two ways.

Accessing any logs currently requires me to break encapsulation and peel your way through the Job to the Pod running, and then access the logs of the relevant init container.

For failed steps, this is less than ideal, but works.
For successful steps, you get a container not-found error (I guess it's aggressively cleaned up).

@mattmoor mattmoor added enhancement New feature or request builders/cluster labels Feb 4, 2018
@mattmoor
Copy link
Member Author

I wonder if for logs having a fluentd daemonset is enough?

@mdemirhan
Copy link
Contributor

We should tie this with the overall logging & monitoring story (which is very much in flux at the moment - we have a meeting scheduled next week with Pivotal to discuss this further), but to unblock this, a fluentd daemonset or a fluentd sidecar container + some fixed rules to collect build logs and sending them to an Elastic Search cluster (or any other endpoint) is possible. I can take a stab at this one - let me know what you think.

@mattmoor mattmoor added this to the M3 milestone Mar 19, 2018
@mattmoor
Copy link
Member Author

@mdemirhan Is this something that should be covered by your work thus far?

@mdemirhan
Copy link
Contributor

Yes, it should be. Current policy is setup to collect all build-controller container logs. However; this is not sufficient in this case I am assuming. I will take a look at build-crd code to see what containers I should watch and integrate.

@mattmoor
Copy link
Member Author

So is this done once we merge HEAD into Elafros?

@mdemirhan
Copy link
Contributor

Yes, this is complete as far as getting build logs into ElasticSearch. I tested this with the current test cases for build and also tested this with an init container that crashes. We are getting logs correctly. We probably should add a small paragraph explaining how to get build logs. Any recommendations for the location of that small paragraph?

However; our overall debugging story is still a little complex. We need to have a clear guide on how to debug issues (something failed - where do I start - step by step guide). I will create an uber issue that tracks the entire debugging experience.

vdemeester pushed a commit to vdemeester/knative-build that referenced this issue Mar 29, 2019
vdemeester pushed a commit to vdemeester/knative-build that referenced this issue Apr 3, 2019
vdemeester pushed a commit to vdemeester/knative-build that referenced this issue Apr 3, 2019
vdemeester pushed a commit to vdemeester/knative-build that referenced this issue Apr 3, 2019
vdemeester pushed a commit to vdemeester/knative-build that referenced this issue Apr 3, 2019
@knative-housekeeping-robot

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.\n
If this issue is safe to close now please do so with /close.\n
Send feedback to Knative Productivity Slack channel or knative/test-infra.
/lifecycle stale

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants