Skip to content

Add helper script to check s3 object consistency#23

Merged
Watson1978 merged 7 commits intomainfrom
check-s3-object
Mar 26, 2026
Merged

Add helper script to check s3 object consistency#23
Watson1978 merged 7 commits intomainfrom
check-s3-object

Conversation

@kenhys
Copy link
Copy Markdown
Contributor

@kenhys kenhys commented Mar 25, 2026

It will check consistency between local repository mirror and already uploaded objects.

Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
@kenhys kenhys requested a review from Watson1978 March 25, 2026 08:49
kenhys and others added 3 commits March 25, 2026 18:20
Co-authored-by: Shizuo Fujita <fujita@clear-code.com>
Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
Co-authored-by: Shizuo Fujita <fujita@clear-code.com>
Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
Co-authored-by: Shizuo Fujita <fujita@clear-code.com>
Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
Copy link
Copy Markdown
Contributor

@Watson1978 Watson1978 left a comment

Choose a reason for hiding this comment

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

It seems safer to enclose variables that handle paths in double quotes (like "$WORKING_DIR/$relative_dir"),
but I think this PR can be merged.

Co-authored-by: Shizuo Fujita <fujita@clear-code.com>
Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
Co-authored-by: Shizuo Fujita <fujita@clear-code.com>
Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
Copy link
Copy Markdown
Contributor

@Watson1978 Watson1978 left a comment

Choose a reason for hiding this comment

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

👍🏻

@Watson1978 Watson1978 merged commit b4d7d24 into main Mar 26, 2026
34 checks passed
@Watson1978 Watson1978 deleted the check-s3-object branch March 26, 2026 02:20
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