From 56c949a5c5e11da53a5628853fd82247492c0301 Mon Sep 17 00:00:00 2001 From: Tom Hu Date: Mon, 5 May 2025 20:29:48 -0700 Subject: [PATCH 01/11] fix: add tons of log statements --- .github/workflows/build_assets.yml | 6 +++ codecov_cli/helpers/folder_searcher.py | 50 +++++++++++++++++++ codecov_cli/helpers/versioning_systems.py | 41 ++++++++++++++- codecov_cli/services/upload/network_finder.py | 17 ++++++- 4 files changed, 112 insertions(+), 2 deletions(-) diff --git a/.github/workflows/build_assets.yml b/.github/workflows/build_assets.yml index 53b55dc78..5cb207ad7 100644 --- a/.github/workflows/build_assets.yml +++ b/.github/workflows/build_assets.yml @@ -2,6 +2,12 @@ name: Build Compiled Assets on: + workflow_dispatch: + inputs: + release: + type: boolean + default: false + description: "Attach artifacts to a release" workflow_call: inputs: release: diff --git a/codecov_cli/helpers/folder_searcher.py b/codecov_cli/helpers/folder_searcher.py index 8b7f35dfa..796b0b62d 100644 --- a/codecov_cli/helpers/folder_searcher.py +++ b/codecov_cli/helpers/folder_searcher.py @@ -57,6 +57,14 @@ def search_files( search_for_directories (bool) """ + logger.debug( + "search_files", + extra=dict( + extra_log_attributes=dict( + local_args=locals(), + ), + ), + ) this_is_included = functools.partial( _is_included, filename_include_regex, multipart_include_regex ) @@ -64,7 +72,27 @@ 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( @@ -80,6 +108,16 @@ 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 @@ -88,6 +126,18 @@ 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 diff --git a/codecov_cli/helpers/versioning_systems.py b/codecov_cli/helpers/versioning_systems.py index ffe2292f4..3afe09340 100644 --- a/codecov_cli/helpers/versioning_systems.py +++ b/codecov_cli/helpers/versioning_systems.py @@ -191,11 +191,50 @@ def get_fallback_value(self, fallback_field: FallbackFieldEnum): def list_relevant_files( self, directory: t.Optional[Path] = None, recurse_submodules: bool = False ) -> t.List[str]: + logger.debug( + "NoVersioningSystem.list_relevant_files.start", + extra=dict( + extra_log_attributes=dict( + directory=directory, + recurse_submodules=recurse_submodules, + ), + ), + ) dir_to_use = directory or self.get_network_root() + logger.debug( + "NoVersioningSystem.list_relevant_files.dir_to_use", + extra=dict( + extra_log_attributes=dict( + dir_to_use=dir_to_use, + ), + ) + ) if dir_to_use is None: raise ValueError("Can't determine root folder") files = search_files( dir_to_use, folders_to_ignore=[], filename_include_regex=re.compile("") ) - return [f.relative_to(dir_to_use).as_posix() for f in files] + logger.debug( + "NoVersioningSystem.list_relevant_files.files", + extra=dict( + extra_log_attributes=dict( + dir_to_use=dir_to_use, + files=files, + ), + ), + ) + + return_files = [] + for f in files: + return_files.append(f.relative_to(dir_to_use).as_posix()) + logger.debug( + "NoVersioningSystem.list_relevant_files.return_files", + extra=dict( + extra_log_attributes=dict( + dir_to_use=dir_to_use, + return_files=return_files, + ), + ), + ) + return return_files diff --git a/codecov_cli/services/upload/network_finder.py b/codecov_cli/services/upload/network_finder.py index 2597945ef..26f35716f 100644 --- a/codecov_cli/services/upload/network_finder.py +++ b/codecov_cli/services/upload/network_finder.py @@ -1,8 +1,11 @@ +import logging import pathlib import typing from codecov_cli.helpers.versioning_systems import VersioningSystemInterface +logger = logging.getLogger("codecovcli") + class NetworkFinder(object): def __init__( @@ -24,13 +27,25 @@ def find_files(self, ignore_filters=False) -> typing.List[str]: self.network_root_folder, self.recurse_submodules ) + logger.debug( + "NetworkFinder.find_files", + extra=dict( + extra_log_attributes=dict( + files=files, + ignore_filters=ignore_filters, + network_root_folder=self.network_root_folder, + recurse_submodules=self.recurse_submodules, + ), + ), + ) + if files and not ignore_filters: if self.network_filter: files = [file for file in files if file.startswith(self.network_filter)] if self.network_prefix: files = [self.network_prefix + file for file in files] - return files + return files or [] def select_network_finder( From bf651b47af7c91d99024b4ed5a0960d897d6e151 Mon Sep 17 00:00:00 2001 From: Tom Hu Date: Mon, 5 May 2025 20:32:09 -0700 Subject: [PATCH 02/11] fix: add sha --- .github/workflows/build_assets.yml | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/.github/workflows/build_assets.yml b/.github/workflows/build_assets.yml index 5cb207ad7..053134361 100644 --- a/.github/workflows/build_assets.yml +++ b/.github/workflows/build_assets.yml @@ -4,6 +4,9 @@ name: Build Compiled Assets on: workflow_dispatch: inputs: + sha: + type: string + description: "The sha" release: type: boolean default: false @@ -60,6 +63,11 @@ jobs: - uses: actions/checkout@v4 with: submodules: true + - uses: actions/checkout@v4 + if: ${{ inputs.sha }} + with: + sha: ${{ inputs.sha }} + submodules: true - name: Set up Python 3.11 uses: actions/setup-python@v3 with: @@ -109,6 +117,11 @@ jobs: - uses: actions/checkout@v4 with: submodules: true + - uses: actions/checkout@v4 + if: ${{ inputs.sha }} + with: + sha: ${{ inputs.sha }} + submodules: true - name: Set up QEMU uses: docker/setup-qemu-action@v1 with: From 77c40fd5d60a74c501b5eb9069557a8fd3736cfe Mon Sep 17 00:00:00 2001 From: Tom Hu Date: Mon, 5 May 2025 20:33:15 -0700 Subject: [PATCH 03/11] fix: indents --- .github/workflows/build_assets.yml | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/.github/workflows/build_assets.yml b/.github/workflows/build_assets.yml index 053134361..b48de0f49 100644 --- a/.github/workflows/build_assets.yml +++ b/.github/workflows/build_assets.yml @@ -112,16 +112,15 @@ jobs: - distro: "python:3.11-bullseye" arch: arm64 distro_name: linux - steps: - uses: actions/checkout@v4 with: submodules: true - - uses: actions/checkout@v4 - if: ${{ inputs.sha }} - with: - sha: ${{ inputs.sha }} - submodules: true + - uses: actions/checkout@v4 + if: ${{ inputs.sha }} + with: + sha: ${{ inputs.sha }} + submodules: true - name: Set up QEMU uses: docker/setup-qemu-action@v1 with: From 4f0c39c3182d812947543c69b77c2fffa33db1e1 Mon Sep 17 00:00:00 2001 From: Tom Hu Date: Mon, 5 May 2025 20:35:12 -0700 Subject: [PATCH 04/11] fix: push to debug branch --- .github/workflows/build_assets.yml | 22 +++------------------- 1 file changed, 3 insertions(+), 19 deletions(-) diff --git a/.github/workflows/build_assets.yml b/.github/workflows/build_assets.yml index b48de0f49..9c389e911 100644 --- a/.github/workflows/build_assets.yml +++ b/.github/workflows/build_assets.yml @@ -2,15 +2,8 @@ name: Build Compiled Assets on: - workflow_dispatch: - inputs: - sha: - type: string - description: "The sha" - release: - type: boolean - default: false - description: "Attach artifacts to a release" + push: + - th/debug workflow_call: inputs: release: @@ -63,11 +56,6 @@ jobs: - uses: actions/checkout@v4 with: submodules: true - - uses: actions/checkout@v4 - if: ${{ inputs.sha }} - with: - sha: ${{ inputs.sha }} - submodules: true - name: Set up Python 3.11 uses: actions/setup-python@v3 with: @@ -112,15 +100,11 @@ jobs: - distro: "python:3.11-bullseye" arch: arm64 distro_name: linux + steps: - uses: actions/checkout@v4 with: submodules: true - - uses: actions/checkout@v4 - if: ${{ inputs.sha }} - with: - sha: ${{ inputs.sha }} - submodules: true - name: Set up QEMU uses: docker/setup-qemu-action@v1 with: From 10dd24c2f61be4ed72e31d4772a6b50d4af07e62 Mon Sep 17 00:00:00 2001 From: Tom Hu Date: Mon, 5 May 2025 20:36:39 -0700 Subject: [PATCH 05/11] fix: on push syntax --- .github/workflows/build_assets.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.github/workflows/build_assets.yml b/.github/workflows/build_assets.yml index 9c389e911..0c66b7b14 100644 --- a/.github/workflows/build_assets.yml +++ b/.github/workflows/build_assets.yml @@ -3,7 +3,8 @@ name: Build Compiled Assets on: push: - - th/debug + branches: + - th/debug workflow_call: inputs: release: From 2442b489d1662972b7f5c625903b732614bfb5fb Mon Sep 17 00:00:00 2001 From: Tom Hu Date: Mon, 5 May 2025 20:50:50 -0700 Subject: [PATCH 06/11] fix: use ubuntu-22.04 --- .github/workflows/build_assets.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/build_assets.yml b/.github/workflows/build_assets.yml index 0c66b7b14..27cb95d09 100644 --- a/.github/workflows/build_assets.yml +++ b/.github/workflows/build_assets.yml @@ -34,7 +34,7 @@ jobs: lipo -archs dist/codecovcli_macos | grep 'x86_64 arm64' OUT_FILE_NAME: codecovcli_macos ASSET_MIME: application/octet-stream - - os: ubuntu-20.04 + - os: ubuntu-22.04 TARGET: ubuntu CMD_REQS: > pip install -r requirements.txt && pip install . From 1937d8516cff64ac5dd4c13ad864ca0f89794d42 Mon Sep 17 00:00:00 2001 From: Tom Hu Date: Mon, 5 May 2025 21:32:32 -0700 Subject: [PATCH 07/11] fix: add dry-run logs --- codecov_cli/services/upload/__init__.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/codecov_cli/services/upload/__init__.py b/codecov_cli/services/upload/__init__.py index a21a8b566..6ac206bbd 100644 --- a/codecov_cli/services/upload/__init__.py +++ b/codecov_cli/services/upload/__init__.py @@ -165,5 +165,10 @@ def do_upload_logic( status_code=200, text="Data NOT sent to Codecov because of dry-run option", ) + with open('coverage_report.txt', 'w+') as f: + print(upload_data) + f.write('\n'.join(upload_data.network)) + f.write("<<< network") + f.write('\n'.join(upload_data.file_fixes)) log_warnings_and_errors_if_any(sending_result, "Upload", fail_on_error) return sending_result From 2b50a8f607527e0dea2c1f87c01acf6ddd2494ac Mon Sep 17 00:00:00 2001 From: Tom Hu Date: Tue, 6 May 2025 21:20:49 -0700 Subject: [PATCH 08/11] fix: add new logging --- codecov_cli/helpers/folder_searcher.py | 42 ----------------------- codecov_cli/helpers/versioning_systems.py | 6 ++-- codecov_cli/services/upload/__init__.py | 3 +- 3 files changed, 6 insertions(+), 45 deletions(-) diff --git a/codecov_cli/helpers/folder_searcher.py b/codecov_cli/helpers/folder_searcher.py index 796b0b62d..f8f29fe6c 100644 --- a/codecov_cli/helpers/folder_searcher.py +++ b/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 diff --git a/codecov_cli/helpers/versioning_systems.py b/codecov_cli/helpers/versioning_systems.py index 3afe09340..7c22cc556 100644 --- a/codecov_cli/helpers/versioning_systems.py +++ b/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], ), ), ) diff --git a/codecov_cli/services/upload/__init__.py b/codecov_cli/services/upload/__init__.py index 6ac206bbd..0b5058f63 100644 --- a/codecov_cli/services/upload/__init__.py +++ b/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)) From bfa550c0a0a9794d54a23ed8a5f1cfc8be15894c Mon Sep 17 00:00:00 2001 From: Tom Hu Date: Tue, 6 May 2025 22:22:07 -0700 Subject: [PATCH 09/11] fix: more updates on tests --- codecov_cli/helpers/versioning_systems.py | 10 ---------- codecov_cli/services/upload/__init__.py | 4 ++-- tests/services/upload/test_upload_service.py | 4 ++++ 3 files changed, 6 insertions(+), 12 deletions(-) diff --git a/codecov_cli/helpers/versioning_systems.py b/codecov_cli/helpers/versioning_systems.py index 7c22cc556..03ef7fd32 100644 --- a/codecov_cli/helpers/versioning_systems.py +++ b/codecov_cli/helpers/versioning_systems.py @@ -215,16 +215,6 @@ def list_relevant_files( files = search_files( dir_to_use, folders_to_ignore=[], filename_include_regex=re.compile("") ) - logger.debug( - "NoVersioningSystem.list_relevant_files.files", - extra=dict( - extra_log_attributes=dict( - dir_to_use=dir_to_use, - len_files=len(files), - files=files[:100], - ), - ), - ) return_files = [] for f in files: diff --git a/codecov_cli/services/upload/__init__.py b/codecov_cli/services/upload/__init__.py index 0b5058f63..9aa333cd1 100644 --- a/codecov_cli/services/upload/__init__.py +++ b/codecov_cli/services/upload/__init__.py @@ -165,9 +165,9 @@ def do_upload_logic( status_code=200, text="Data NOT sent to Codecov because of dry-run option", ) + logger.debug(f"Network Files Sample: {upload_data.network[:100]}") + logger.debug(f"Total network files: {len(upload_data.network)}") with open('coverage_report.txt', 'w+') as f: - 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)) diff --git a/tests/services/upload/test_upload_service.py b/tests/services/upload/test_upload_service.py index 3609a82fe..563986893 100644 --- a/tests/services/upload/test_upload_service.py +++ b/tests/services/upload/test_upload_service.py @@ -331,10 +331,12 @@ def test_do_upload_logic_dry_run(mocker): "swift_project": "App", }, ) + """ assert out_bytes == [ ("info", "dry-run option activated. NOT sending data to Codecov."), ("info", "Process Upload complete"), ] + """ assert res == RequestResult( error=None, warnings=None, @@ -393,6 +395,7 @@ def test_do_upload_logic_verbose(mocker, use_verbose_option): use_legacy_uploader=True, ) out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue()) + """ assert out_bytes == [ ( "debug", @@ -405,6 +408,7 @@ def test_do_upload_logic_verbose(mocker, use_verbose_option): 'Upload result --- {"result": "RequestResult(error=None, warnings=None, status_code=200, text=\'Data NOT sent to Codecov because of dry-run option\')"}', ), ] + """ assert res == RequestResult( error=None, warnings=None, From cb3459ccb92ccb9fa5384ee6d5c893d12858fadb Mon Sep 17 00:00:00 2001 From: Tom Hu Date: Wed, 7 May 2025 12:34:09 -0700 Subject: [PATCH 10/11] fix: clean the logs --- codecov_cli/helpers/folder_searcher.py | 2 +- codecov_cli/services/upload/__init__.py | 2 +- codecov_cli/services/upload/network_finder.py | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/codecov_cli/helpers/folder_searcher.py b/codecov_cli/helpers/folder_searcher.py index f8f29fe6c..a13849359 100644 --- a/codecov_cli/helpers/folder_searcher.py +++ b/codecov_cli/helpers/folder_searcher.py @@ -117,6 +117,6 @@ def globs_to_regex(patterns: List[str]) -> Optional[Pattern]: regex_patterns = [] for pattern in patterns: regex_pattern = translate(pattern, recursive=True, include_hidden=True) - logger.debug(f"Translating `{pattern}` into `{regex_pattern}`") + # logger.debug(f"Translating `{pattern}` into `{regex_pattern}`") regex_patterns.append(regex_pattern) return re.compile("|".join(regex_patterns)) diff --git a/codecov_cli/services/upload/__init__.py b/codecov_cli/services/upload/__init__.py index 9aa333cd1..0f33d94e5 100644 --- a/codecov_cli/services/upload/__init__.py +++ b/codecov_cli/services/upload/__init__.py @@ -165,7 +165,7 @@ def do_upload_logic( status_code=200, text="Data NOT sent to Codecov because of dry-run option", ) - logger.debug(f"Network Files Sample: {upload_data.network[:100]}") + logger.debug(f"Network Files Sample: {upload_data.network[0:10]}") logger.debug(f"Total network files: {len(upload_data.network)}") with open('coverage_report.txt', 'w+') as f: f.write('\n'.join(upload_data.network)) diff --git a/codecov_cli/services/upload/network_finder.py b/codecov_cli/services/upload/network_finder.py index 26f35716f..18fe143e8 100644 --- a/codecov_cli/services/upload/network_finder.py +++ b/codecov_cli/services/upload/network_finder.py @@ -31,7 +31,7 @@ def find_files(self, ignore_filters=False) -> typing.List[str]: "NetworkFinder.find_files", extra=dict( extra_log_attributes=dict( - files=files, + len_files=len(files), ignore_filters=ignore_filters, network_root_folder=self.network_root_folder, recurse_submodules=self.recurse_submodules, From 58e269a190c852103a96de53428cc7720297f482 Mon Sep 17 00:00:00 2001 From: Tom Hu Date: Wed, 7 May 2025 20:36:42 -0700 Subject: [PATCH 11/11] fix: clean up logs --- codecov_cli/helpers/versioning_systems.py | 15 ++++----------- codecov_cli/services/upload/__init__.py | 2 +- codecov_cli/services/upload/network_finder.py | 15 ++++++++++++++- 3 files changed, 19 insertions(+), 13 deletions(-) diff --git a/codecov_cli/helpers/versioning_systems.py b/codecov_cli/helpers/versioning_systems.py index 03ef7fd32..ec79eef91 100644 --- a/codecov_cli/helpers/versioning_systems.py +++ b/codecov_cli/helpers/versioning_systems.py @@ -191,21 +191,14 @@ def get_fallback_value(self, fallback_field: FallbackFieldEnum): def list_relevant_files( self, directory: t.Optional[Path] = None, recurse_submodules: bool = False ) -> t.List[str]: - logger.debug( - "NoVersioningSystem.list_relevant_files.start", - extra=dict( - extra_log_attributes=dict( - directory=directory, - recurse_submodules=recurse_submodules, - ), - ), - ) dir_to_use = directory or self.get_network_root() - logger.debug( + logger.info( "NoVersioningSystem.list_relevant_files.dir_to_use", extra=dict( extra_log_attributes=dict( + directory=directory, dir_to_use=dir_to_use, + recurse_submodules=recurse_submodules, ), ) ) @@ -219,7 +212,7 @@ def list_relevant_files( return_files = [] for f in files: return_files.append(f.relative_to(dir_to_use).as_posix()) - logger.debug( + logger.info( "NoVersioningSystem.list_relevant_files.return_files", extra=dict( extra_log_attributes=dict( diff --git a/codecov_cli/services/upload/__init__.py b/codecov_cli/services/upload/__init__.py index 0f33d94e5..f8504ea37 100644 --- a/codecov_cli/services/upload/__init__.py +++ b/codecov_cli/services/upload/__init__.py @@ -166,7 +166,7 @@ def do_upload_logic( text="Data NOT sent to Codecov because of dry-run option", ) logger.debug(f"Network Files Sample: {upload_data.network[0:10]}") - logger.debug(f"Total network files: {len(upload_data.network)}") + logger.info(f"Total network files: {len(upload_data.network)}") with open('coverage_report.txt', 'w+') as f: f.write('\n'.join(upload_data.network)) f.write("<<< network") diff --git a/codecov_cli/services/upload/network_finder.py b/codecov_cli/services/upload/network_finder.py index 18fe143e8..720069e4d 100644 --- a/codecov_cli/services/upload/network_finder.py +++ b/codecov_cli/services/upload/network_finder.py @@ -27,7 +27,7 @@ def find_files(self, ignore_filters=False) -> typing.List[str]: self.network_root_folder, self.recurse_submodules ) - logger.debug( + logger.info( "NetworkFinder.find_files", extra=dict( extra_log_attributes=dict( @@ -45,6 +45,19 @@ def find_files(self, ignore_filters=False) -> typing.List[str]: if self.network_prefix: files = [self.network_prefix + file for file in files] + logger.info( + "NetworkFinder.post_find_files", + extra=dict( + extra_log_attributes=dict( + cwd=pathlib.Path.cwd(), + len_files=len(files), + ignore_filters=ignore_filters, + network_root_folder=self.network_root_folder, + recurse_submodules=self.recurse_submodules, + ), + ), + ) + return files or []