Skip to content

Add a link from winston ECS logging to a Cloud tutorial doc #100

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

Merged
merged 2 commits into from
Sep 14, 2021

Conversation

kilfoyle
Copy link
Contributor

Rel: #99
Adds a link at the bottom of this Node.js winston ECS logging page to a similar tutorial in the Cloud docs.

@apmmachine
Copy link
Contributor

apmmachine commented Sep 10, 2021

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2021-09-10T22:31:45.925+0000

  • Duration: 7 min 20 sec

  • Commit: 5d99322

Trends 🧪

Image of Build Times

@github-actions github-actions bot added the agent-nodejs Make available for APM Agents project planning. label Sep 10, 2021
@@ -265,3 +265,5 @@ const logger = winston.createLogger({
// ...
})
----

To learn more, try out our tutorial using Node.js ECS logging with winston: {cloud}/ec-getting-started-search-use-cases-node-logs.html[Ingest logs from a Node.js web application using Filebeat].
Copy link
Member

Choose a reason for hiding this comment

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

I think this would be better in the "Setup" section up higher. Could you move this up to just above these lines:

[float]
[[winston-usage]]
=== Usage

?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good idea. I moved the link up to the "Configure Filebeat" section.

Move the tutorial link up to the `Configure Filebeat` section.
@trentm
Copy link
Member

trentm commented Sep 10, 2021

jenkins run the tests

@trentm
Copy link
Member

trentm commented Sep 13, 2021

@elasticmachine run elasticsearch-ci/docs

@trentm trentm merged commit ae3cd07 into elastic:master Sep 14, 2021
@kilfoyle
Copy link
Contributor Author

Thanks for approving this! I'm not able to merge it but if there's anything I can do to help, let me know.

@kilfoyle
Copy link
Contributor Author

Ah, I see it's merged now. Thanks @trentm!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
agent-nodejs Make available for APM Agents project planning.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants