Discource-C
/
discourse
mirror of https://github.com/discourse/discourse.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

FIX: Migrating uploads to S3 could miss files 

The rake task aborted the migration with "Already migrated" when all upload URLs linked to the correct S3 bucket even though the files didn't exist on S3. By removing the first check we force the rake task to check for the existance of uploads on S3.
This commit is contained in:
Gerhard Schlager 2020-03-04 12:46:04 +01:00
parent 9cbed7ee46
commit 93b8b04b06
1 changed files with 0 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
lib/file_store/to_s3_migration.rb
View File

@ -139,9 +139,6 @@ module FileStore
# we don't want have migrated state, ensure we run all jobs here # we don't want have migrated state, ensure we run all jobs here
Jobs.run_immediately! Jobs.run_immediately!
log "Checking if #{@current_db} already migrated..."
return log "Already migrated #{@current_db}!" if migration_successful?
log "*" * 30 + " DRY RUN " + "*" * 30 if @dry_run log "*" * 30 + " DRY RUN " + "*" * 30 if @dry_run
log "Migrating uploads to S3 for '#{@current_db}'..." log "Migrating uploads to S3 for '#{@current_db}'..."