Skip to content

Commit 2c02e85

Browse files
authored
Merge pull request #144 from ligangty/copy-to
Chore: adjust logging
2 parents fb5b94f + 35261f1 commit 2c02e85

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

charon/pkgs/maven.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -316,17 +316,16 @@ def handle_maven_uploading(
316316
root=top_level
317317
)
318318
logger.info("Files uploading done\n")
319-
320319
succeeded = True
321320
for target in targets:
322321
# 5. Do manifest uploading
323-
manifest_folder = target[0]
324-
logger.info("Start uploading manifest to s3 bucket %s", manifest_bucket_name)
325322
if not manifest_bucket_name:
326323
logger.warning(
327324
'Warning: No manifest bucket is provided, will ignore the process of manifest '
328325
'uploading\n')
329326
else:
327+
logger.info("Start uploading manifest to s3 bucket %s", manifest_bucket_name)
328+
manifest_folder = target[0]
330329
manifest_name, manifest_full_path = write_manifest(valid_mvn_paths, top_level, prod_key)
331330
s3_client.upload_manifest(
332331
manifest_name, manifest_full_path,

charon/storage.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -287,7 +287,7 @@ async def __copy_between_bucket(
287287
self, source: str, source_key: str,
288288
target: s3.Bucket, target_key: str
289289
) -> bool:
290-
logger.info(
290+
logger.debug(
291291
"Copying file %s from bucket %s to target %s as %s",
292292
source_key, source, target.name, target_key)
293293
copy_source = {

0 commit comments

Comments
 (0)