Skip to content

Conversation

@rohinlohe
Copy link
Contributor

@rohinlohe rohinlohe commented Apr 3, 2025

Screenshot 2025-04-07 at 11 22 59 AM

Documentation checklist

  • The documentation style guide has been adhered to.
  • If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.
  • Files which have changed name or location have been allocated redirects.

@hyperlint-ai
Copy link
Contributor

hyperlint-ai bot commented Apr 3, 2025

Howdy and thanks for contributing to our repo. The Cloudflare team reviews new, external PRs within two (2) weeks. If it's been two weeks or longer without any movement, please tag the PR Assignees in a comment.

We review internal PRs within 1 week. If it's something urgent or has been sitting without a comment, start a thread in the Developer Docs space internally.


PR Change Summary

Increased the maximum log events limit for Workers from 128 KB to 256 KB, enhancing logging capabilities and visibility into application behavior.

  • Added a new changelog entry detailing the increase in maximum log events limit to 256 KB.
  • Updated documentation to reflect the new maximum log size for Workers logs.
  • Modified existing documentation to clarify the implications of exceeding the new log size limit.

Modified Files

  • src/content/docs/workers/observability/logs/workers-logs.mdx
  • src/content/docs/workers/platform/limits.mdx

Added Files

  • src/content/changelog/workers/2025-04-07-increase-trace-events-limit.mdx

How can I customize these reviews?

Check out the Hyperlint AI Reviewer docs for more information on how to customize the review.

If you just want to ignore it on this PR, you can add the hyperlint-ignore label to the PR. Future changes won't trigger a Hyperlint review.

Note specifically for link checks, we only check the first 30 links in a file and we cache the results for several hours (for instance, if you just added a page, you might experience this). Our recommendation is to add hyperlint-ignore to the PR to ignore the link check for this PR.

Copy link
Contributor

@irvinebroque irvinebroque left a comment

Choose a reason for hiding this comment

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

Something should link to workers logs / point you to how to log stuff from workers?

If I didn't know Workers had this stuff built in, could I discover that from this post?

@@ -0,0 +1,14 @@
---
title: Workers increases maximum log events per invocation to 256 KB
Copy link
Contributor

Choose a reason for hiding this comment

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

Frame in terms of what you can do as developer not what we did

@github-actions github-actions bot added size/s product:workers Related to Workers product labels Apr 3, 2025
@rohinlohe rohinlohe force-pushed the rohin/raise-trace-events-limit branch from c17bb2c to dc3fed4 Compare April 7, 2025 10:23
@rohinlohe rohinlohe merged commit 556742f into production Apr 7, 2025
11 checks passed
@rohinlohe rohinlohe deleted the rohin/raise-trace-events-limit branch April 7, 2025 10:41
RebeccaTamachiro pushed a commit that referenced this pull request Apr 21, 2025
…t to 256 KB (#21376)

* [workers-observability] changelog: increase workers trace events limit to 256 KB

* [workers-observability] changelog: reference workers logs

---------

Co-authored-by: Rohin Lohe <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

product:workers Related to Workers product size/s

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants