Skip to content

Commit ec9a30f

Browse files
authored
Merge pull request #1796 from cmu-delphi/ndefries/backfill/aws-upload-settings
[Backfill corrections] hone AWS publish behavior
2 parents 1ea75b9 + dfc8f1c commit ec9a30f

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

backfill_corrections/Makefile

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -85,11 +85,13 @@ run:
8585
/bin/bash -c "cp params.host.json params.json && make gurobi.lic && make standardize-dirs && make run-local OPTIONS=\"${OPTIONS}\" LOG_FILE=${LOG_FILE}"
8686

8787
publish:
88-
NUM_FILES=`find $(USR_EXPORT_DIR) -name "*csv.gz" | wc -l`; \
88+
NUM_FILES=`find $(USR_EXPORT_DIR) -name "prediction*.csv.gz" | wc -l`; \
8989
if [[ $$NUM_FILES -gt 0 ]]; then \
9090
aws configure set aws_access_key_id $(AWS_KEY_ID); \
9191
aws configure set aws_secret_access_key $(AWS_SECRET_KEY); \
92-
aws s3 cp $(USR_EXPORT_DIR) $(S3_BUCKET)/ --recursive --exclude "*" --include "*.csv.gz" --acl public-read; \
92+
AWS_MSG=`aws s3 cp $(USR_EXPORT_DIR) $(S3_BUCKET)/ --recursive --exclude "*" --include "*/prediction*.csv.gz"`; \
93+
echo "$${AWS_MSG}"; \
94+
[[ `echo "$${AWS_MSG}" | grep "upload:" | wc -l` -eq $$NUM_FILES ]] || ( echo "ERROR: Not all $${NUM_FILES} expected files were uploaded" && exit 74 ); \
9395
echo "SUCCESS: published $${NUM_FILES} files to the S3 bucket" >> $(LOG_FILE); \
9496
else \
9597
echo "No files in $(USR_EXPORT_DIR) to publish" >> $(LOG_FILE); \

0 commit comments

Comments
 (0)