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

515 Slow s3 Sync #516

Merged
merged 2 commits into from Jan 11, 2020
Merged

515 Slow s3 Sync #516

merged 2 commits into from Jan 11, 2020

Conversation

@duallain
Copy link

duallain commented Jan 10, 2020

Description

A small improvement to help with the speed of production deploys.

Issue / Bugzilla link

#515

Testing

See the associated issue for information about different approaches to running the sync. This PR is just for co-locating the gitlab runner and the bucket. Hopefully this will speed things up while we work on a different change for the fix itself.

No direct testing possible, the job ought to fail on staging if the runner is unavailable, or has incorrect credentials etc.

Alan Alexander added 2 commits Jan 10, 2020
@duallain duallain requested review from glogiotatidis, jgmize and metadave Jan 10, 2020
@jgmize
jgmize approved these changes Jan 11, 2020
Copy link
Contributor

jgmize left a comment

thanks @duallain!

@jgmize jgmize merged commit d72f8d2 into master Jan 11, 2020
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@metadave metadave deleted the 515_slow_sync branch Jan 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.