Skip to content

storage location not respected on course import #636

Description

@samuelallan72

When a course with video transcripts is imported, the path location is not respected; it goes in the root of the storage. Everything else (eg. reading transcripts) appears to correctly respect the location.

For example, if I have the following config (tutor default + tutor-contrib-s3):

MEDIA_ROOT = "/openedx/media/"
VIDEO_TRANSCRIPTS_SETTINGS["STORAGE_KWARGS"]["location"] = MEDIA_ROOT
STORAGES['default']['BACKEND'] = "storages.backends.s3boto3.S3Boto3Storage"
VIDEO_TRANSCRIPTS_SETTINGS["STORAGE_KWARGS"]["location"] = VIDEO_TRANSCRIPTS_SETTINGS["STORAGE_KWARGS"]["location"].lstrip("/")

And I import a course with transcripts, the transcripts will be saved to video-transcripts/ in s3, but it will try reading the transcripts from openedx/media/video-transcripts.

I believe I've been able to reproduce similar behaviour with the new STORAGES based setting, although I think maybe the transcript file didn't get uploaded at all. Perhaps more testing is required there:

STORAGES["video_transcripts"] = {
      "BACKEND": "storages.backends.s3boto3.S3Boto3Storage",
      "OPTIONS": {
          "bucket_name": "{{ S3_STORAGE_BUCKET }}",
          "location": "",
      },
}

This is on Verawood.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions