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

202402 logs tutorial #68

Merged
merged 57 commits into from
Mar 1, 2024
Merged

202402 logs tutorial #68

merged 57 commits into from
Mar 1, 2024

Conversation

petermarshallio
Copy link
Collaborator

A new notebook, based on Imply's Druid Logs course, that introduces the configuration for logging in Druid.

Initial version of the workbook, using labs 1 and 2 of Imply's logs course as a basis.
Standardised the intro and moved some stuff.
Transcription from course.
Added the "treasure hunt" from the course, and an example on losing zookeeper.
Added section on retention policies.
Clear up ready for people to look at.
Updated the example to include the hostname as this is a more common possible usage.
Copy link
Contributor

@sergioferragut sergioferragut left a comment

Choose a reason for hiding this comment

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

this is a great intro to configuring logs.
Left some suggestions on how to make it more user friendly.

@petermarshallio petermarshallio marked this pull request as ready for review February 22, 2024 11:59
petermarshallio and others added 22 commits February 29, 2024 15:03
@petermarshallio
Copy link
Collaborator Author

Thank you so much @ektravel !!!

Copy link
Collaborator

@writer-jill writer-jill left a comment

Choose a reason for hiding this comment

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

LGTM

@petermarshallio petermarshallio merged commit 6cfb220 into main Mar 1, 2024
1 check passed
@petermarshallio petermarshallio deleted the 202402-logs-tutorial branch March 1, 2024 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants