Skip to content

Conversation

@iphydf
Copy link
Member

@iphydf iphydf commented Jan 26, 2025

We usually don't need a very long history to debug issues. This protects us from having to look at unnecessarily old logs and protects users from sharing too much of their activity history.

The default of 1000 is chosen generously because it'll likely be limited by log line age first. The 1000 protects us from enormous logs.


This change is Reviewable

@iphydf iphydf added this to the v1.18.3 milestone Jan 26, 2025
@github-actions github-actions bot added the enhancement New feature for the user, not a new feature for build script label Jan 26, 2025
@iphydf iphydf force-pushed the limit-log branch 5 times, most recently from 18b68db to b4976d5 Compare January 26, 2025 01:05
@iphydf iphydf marked this pull request as ready for review January 26, 2025 01:07
@iphydf iphydf force-pushed the limit-log branch 3 times, most recently from 8009274 to f79a4a2 Compare January 26, 2025 01:20
@iphydf iphydf changed the title feat(Debug): Allow user to limit the copied log size. feat(Debug): Allow user to limit the copied log size [render preview]. Jan 31, 2025
@iphydf iphydf changed the title feat(Debug): Allow user to limit the copied log size [render preview]. feat(Debug): Allow user to limit the copied log size. Jan 31, 2025
@iphydf iphydf force-pushed the limit-log branch 3 times, most recently from ba31a5c to cbd3e52 Compare February 8, 2025 19:51
@github-actions
Copy link

github-actions bot commented Feb 8, 2025

@iphydf iphydf modified the milestones: v1.18.3, v1.18.4 Feb 20, 2025
We usually don't need a very long history to debug issues. This protects
us from having to look at unnecessarily old logs and protects users from
sharing too much of their activity history.

The default of 1000 is chosen generously because it'll likely be limited
by log line age first. The 1000 protects us from enormous logs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature for the user, not a new feature for build script

Development

Successfully merging this pull request may close these issues.

1 participant