fix: add new logging

Author: thomasrockhu-codecov

codecov_cli/helpers/folder_searcher.py

@@ -72,27 +72,7 @@ def search_files(
         _is_excluded, filename_exclude_regex, multipart_exclude_regex
     )
     for dirpath, dirnames, filenames in os.walk(folder_to_search):
-        logger.debug(
-            "search_files.os_walk",
-            extra=dict(
-                extra_log_attributes=dict(
-                    folder_to_search=folder_to_search,
-                    dirpath=dirpath,
-                    dirnames=dirnames,
-                    filenames=filenames,
-                ),
-            ),
-        )
         dirs_to_remove = set(d for d in dirnames if d in folders_to_ignore)
-        logger.debug(
-            "search_files.dirs_to_remove",
-            extra=dict(
-                extra_log_attributes=dict(
-                    dirs_to_remove=dirs_to_remove,
-                    folders_to_ignore=folders_to_ignore,
-                ),
-            ),
-        )
 
         if multipart_exclude_regex is not None:
             dirs_to_remove.union(
@@ -108,16 +88,6 @@ def search_files(
             # This is the documented way of doing this on python docs
             dirnames.remove(directory)
 
-        logger.debug(
-            "search_files.post_dir_removal",
-            extra=dict(
-                extra_log_attributes=dict(
-                    dirnames=dirnames,
-                    search_for_directories=search_for_directories,
-                ),
-            ),
-        )
-
         if search_for_directories:
             for directory in dirnames:
                 dir_path = pathlib.Path(dirpath) / directory
@@ -126,18 +96,6 @@ def search_files(
         else:
             for single_filename in filenames:
                 file_path = pathlib.Path(dirpath) / single_filename
-                logger.debug(
-                    "search_files.search_for_filenames",
-                    extra=dict(
-                        extra_log_attributes=dict(
-                            single_filename=single_filename,
-                            file_path=file_path,
-                            is_excluded=this_is_excluded(file_path),
-                            is_included=this_is_included(file_path),
-                            yield_it=(not this_is_excluded(file_path) and this_is_included(file_path)),
-                        ),
-                    ),
-                )
                 if not this_is_excluded(file_path) and this_is_included(file_path):
                     yield file_path
 

codecov_cli/helpers/versioning_systems.py

@@ -220,7 +220,8 @@ def list_relevant_files(
             extra=dict(
                 extra_log_attributes=dict(
                     dir_to_use=dir_to_use,
-                    files=files,
+                    len_files=len(files),
+                    files=files[:100],
                 ),
             ),
         )
@@ -233,7 +234,8 @@ def list_relevant_files(
             extra=dict(
                 extra_log_attributes=dict(
                     dir_to_use=dir_to_use,
-                    return_files=return_files,
+                    len_return_files=len(return_files),
+                    return_files=return_files[:100],
                 ),
             ),
         )

codecov_cli/services/upload/__init__.py

@@ -166,7 +166,8 @@ def do_upload_logic(
             text="Data NOT sent to Codecov because of dry-run option",
         )
         with open('coverage_report.txt', 'w+') as f:
-            print(upload_data)
+            print(f"Network Files Sample: {upload_data.network[:100]}")
+            print(f"Total network files: {len(upload_data.network)}")
             f.write('\n'.join(upload_data.network))
             f.write("<<< network")
             f.write('\n'.join(upload_data.file_fixes))