Skip to content

Conversation

Anemy
Copy link
Member

@Anemy Anemy commented Sep 17, 2025

COMPASS-8920

Previously we were including all of these import-export debugs, which were ~80,000 out of 100,000 lines.

Before
https://parsley.mongodb.com/evergreen/10gen_compass_main_unit_tests_ubuntu_test_electron_9c656bbb8515f6b131daa86d4b0ca94abd6ce71b_25_09_17_14_25_40/0/task?bookmarks=0,91994&shareLine=12945

After
https://parsley.mongodb.com/evergreen/10gen_compass_main_unit_tests_rhel_test_electron_patch_9c656bbb8515f6b131daa86d4b0ca94abd6ce71b_68caf82a3c54490007231169_25_09_17_18_04_27/0/task?bookmarks=0,14492&shareLine=12799

We could instead update our debug in our evergreen to either not include debugs, or filter out import-export there:

DEBUG: ${debug|}

Not sure if we get value out of the other debug calls though so I figured I'd go with the minimum change here.
I don't think that these debug calls are landing anywhere in e2e test logs.

@Copilot Copilot AI review requested due to automatic review settings September 17, 2025 18:04
@Anemy Anemy requested a review from a team as a code owner September 17, 2025 18:04
@Anemy Anemy requested a review from johnjackweir September 17, 2025 18:04
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR disables debug output for CI tests in the compass-import-export package to reduce log verbosity. Previously, import-export debug logs were generating approximately 80,000 out of 100,000 total log lines in CI runs, making it difficult to find relevant information.

  • Disables debug output by setting DEBUG= environment variable for CI test scripts
  • Applies the change to both regular and electron test CI commands

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Anemy Anemy merged commit 1571453 into main Sep 19, 2025
60 checks passed
@Anemy Anemy deleted the COMPASS-8920-fix-import-export-log-amount branch September 19, 2025 16:54
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