Skip to content

feat(S3): Conditional s3 upload and VersionID #247

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Lorenzovagliano
Copy link
Collaborator

@Lorenzovagliano Lorenzovagliano commented Mar 19, 2025

Requires simultaneous changes in SCOAP3: SCOAP3/scoap3#291

ref: cern-sis/issues-scoap3#401

@Lorenzovagliano Lorenzovagliano force-pushed the repo-make-article-files-versionable-#401 branch 4 times, most recently from 2274ac3 to e138928 Compare March 19, 2025 14:55
@Lorenzovagliano Lorenzovagliano changed the title feat(S3): Add versioning and conditional upload feat(S3): Conditional S3 upload Mar 19, 2025
@Lorenzovagliano Lorenzovagliano force-pushed the repo-make-article-files-versionable-#401 branch 2 times, most recently from e94d33b to 0e7be58 Compare March 19, 2025 15:20
@Lorenzovagliano Lorenzovagliano marked this pull request as ready for review March 19, 2025 15:32
@Lorenzovagliano Lorenzovagliano force-pushed the repo-make-article-files-versionable-#401 branch from 0e7be58 to 585d5d1 Compare March 20, 2025 12:15
@Lorenzovagliano Lorenzovagliano changed the title feat(S3): Conditional S3 upload feat(S3): Conditional s3 upload and VersionID Mar 20, 2025
@Lorenzovagliano Lorenzovagliano force-pushed the repo-make-article-files-versionable-#401 branch from 585d5d1 to 82f4c04 Compare March 20, 2025 12:21
Only upload to S3 when file checksum is different.

Requires simultaneous changes in SCOAP3.

Signed-off-by: [email protected]
@Lorenzovagliano Lorenzovagliano force-pushed the repo-make-article-files-versionable-#401 branch from 82f4c04 to 180c8a4 Compare March 20, 2025 12:50
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.

1 participant