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

feat(cli-repl): add configuration to set max log files size MONGOSH-1985 #2361

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

gagik
Copy link
Contributor

@gagik gagik commented Feb 7, 2025

Requires mongodb-js/devtools-shared#504 to be released.

Builds on top of #2354

@gagik gagik requested a review from alenakhineika February 7, 2025 10:37
@gagik gagik force-pushed the gagik/add-log-retention-file-size branch 2 times, most recently from e6e9338 to a39adef Compare February 7, 2025 11:22
@gagik gagik force-pushed the gagik/add-log-retention-file-count branch 2 times, most recently from dc47df4 to 9a0b368 Compare February 7, 2025 11:38
@gagik gagik force-pushed the gagik/add-log-retention-file-size branch from a39adef to 0aec4d5 Compare February 7, 2025 11:42
@gagik
Copy link
Contributor Author

gagik commented Feb 7, 2025

On the topic of unexpected green CI, https://github.com/mongodb-js/mongosh/actions/runs/13199134044/job/36846954054?pr=2361 seems not to actually run tests?

Edit: created MONGOSH-2011

@gagik gagik force-pushed the gagik/add-log-retention-file-count branch 3 times, most recently from 5f2ec09 to a265439 Compare February 10, 2025 08:55
Base automatically changed from gagik/add-log-retention-file-count to main February 10, 2025 12:13
@gagik gagik force-pushed the gagik/add-log-retention-file-size branch from f7251a7 to 96dbf08 Compare February 11, 2025 13:33
@gagik gagik force-pushed the gagik/add-log-retention-file-size branch from 96dbf08 to 82bdc5d Compare February 11, 2025 13:34
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.

2 participants