Skip to content
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

Added in Region for S3 Sync #19

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

MichaelFBA
Copy link

I noticed the sync script was not working correctly when testing in Frankfurt / Europe. It requires a specific --region REGION

Great work by the way!

I noticed the sync script was not working correctly when testing in Frankfurt / Europe. It requires a specific `--region REGION`

Great work by the way!
@@ -3,6 +3,6 @@ rmdir /Q /S "C:\Program Files (x86)\Steam\steamapps"
md Z:\SteamLibrary\steamapps
cmd /c mklink /j "C:\Program Files (x86)\Steam\steamapps" Z:\SteamLibrary\steamapps
md Z:\Documents
aws s3 sync Z:\Documents s3://BUCKET/Documents
aws s3 sync Z:\Documents s3://BUCKET/Documents --region REGION
if %errorlevel% neq 0 exit /b %errorlevel%
schtasks /Create /RU USERNAME /RP PASSWORD /F /SC MINUTE /MO 1 /TN "Sync Documents with S3" /TR "aws s3 sync Z:\Documents s3://ec2gaming-639801188054/Documents --delete"
Copy link

@jterk jterk Jan 6, 2017

Choose a reason for hiding this comment

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

Do you need the region in the scheduled task as well?

Copy link
Author

Choose a reason for hiding this comment

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

Yep, that's where I needed to add it.

Just thought I would add it to the template file for others

Copy link

Choose a reason for hiding this comment

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

Add --region REGION to the aws s3 invocation on line 8 as well, so that it is added to the scheduled task created by this script.

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