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

Update Getting Started logging docs for Cloud Logging #10203

Merged
merged 1 commit into from
Jun 23, 2015

Conversation

satnam6502
Copy link
Contributor

Update the Getting Started guides for cluster level logging with Google Cloud Logging and update the env vars for Elasticsearch logging.


### Logging with Fluentd and Elastiscsearch
Cluster level logging for Kubernetes allows us to collect logs which persist beyond the lifetime of the pod’s container images or the lifetime of the pod or even cluster. In this article we assume that a Kubernetes cluster has been created with cluster level logging support for sending logs to Google Cloud Logging. This is an option when creating a Google Container Engine (GKE) cluster, and is enabled by default for the open source Google Compute Engine (GCE) Kubernetes distribution. After a cluster has been created you will have a collection of system pods running that support monitoring, logging and DNS resolution for names of Kubernetes services:
Copy link
Contributor

Choose a reason for hiding this comment

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

I believe Google Cloud Logging is also enabled by default for GKE.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed the line about it being optional.

@j3ffml
Copy link
Contributor

j3ffml commented Jun 22, 2015

cc @a-robinson, @RichieEscarez

@j3ffml
Copy link
Contributor

j3ffml commented Jun 22, 2015

Thanks for adding, Satnam. LGTM, but I'd like @a-robinson to take a look.

@k8s-bot
Copy link

k8s-bot commented Jun 22, 2015

GCE e2e build/test failed for commit 8d44eff99897e9cbc91b7736d407dd4aa700c868.


![Cluster](../../examples/blog-logging/diagrams/cloud-logging.png)

This diagram shows four nodes created on a GCE cluster with the name of each VM node on a purple background. The internal and public IPs of each node are shown on gray boxes and the pods running in each node are shown in green boxes. Each pod box shows the name of the pod and the namespace it runs in, the IP address of the pod and the images which are run as part of the pod’s execution. The default namespace is used to run privileged system services rather than user applications. Here we see that every node is running a fluentd-cloud-logging pod which is collecting the log output of the containers running on the same node and sending them to Google Cloud Logging. A pod which provides a DNS service runs on one of the nodes and a pod which provides monitoring support runs on another node.
Copy link
Contributor

Choose a reason for hiding this comment

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

"The default namespace is used to run privileged system services rather than user applications"

Isn't the default namespace typically used for both?

Copy link
Contributor

Choose a reason for hiding this comment

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

Can you make the "a DNS service" bit say "the cluster DNS service" and link to docs/dns.md?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed the line about the namespace, added link to cluster DNS doc.

@satnam6502 satnam6502 force-pushed the logging-doc branch 2 times, most recently from edc1d22 to 4f3efc7 Compare June 23, 2015 18:13
@satnam6502
Copy link
Contributor Author

Thank you. PTAL.

@satnam6502
Copy link
Contributor Author

I've also updated the blog article with these changes. Thank you.

@a-robinson
Copy link
Contributor

LGTM

Only nit is that it's better to use non-relative paths to other files in the repo rather than relative ones so that it's easier to move files around later, e.g. (/docs/dns.md) instead of (../dns.md)

@a-robinson a-robinson added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 23, 2015
@satnam6502
Copy link
Contributor Author

Changed relative doc paths to absolute.

@k8s-bot
Copy link

k8s-bot commented Jun 23, 2015

GCE e2e build/test passed for commit edc1d22bd8039ed83070ac572253df2ed7e40d70.

@k8s-bot
Copy link

k8s-bot commented Jun 23, 2015

GCE e2e build/test passed for commit 4f3efc73c3433259a2c312033d5cb8425c77d5e1.

@a-robinson
Copy link
Contributor

LGTM

@k8s-bot
Copy link

k8s-bot commented Jun 23, 2015

GCE e2e build/test passed for commit 67991dd673c17b6a3216eddf8b3829c68f52aa64.

j3ffml added a commit that referenced this pull request Jun 23, 2015
Update Getting Started logging docs for Cloud Logging
@j3ffml j3ffml merged commit 8a92232 into kubernetes:master Jun 23, 2015
@k8s-bot
Copy link

k8s-bot commented Jun 23, 2015

GCE e2e build/test passed for commit 90caacb.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants