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

fix: add PROFILE_IMAGE_BACKEND settings for k8s #50

Merged
merged 1 commit into from
Oct 31, 2024

Conversation

Faraz32123
Copy link
Collaborator

@Faraz32123 Faraz32123 commented Oct 10, 2024

  • Add PROFILE_IMAGE_BACKEND settings in tutor-minio using patch named
    openedx-lms-production-settings so that profile images persist
    in k8s deployment of openedx and profile images can work now for both
    local(will use minio storage if minio plugin is enabled) and dev environment.
  • In case of dev environment, backend expects the valid S3 base_url
    key in the PROFILE_IMAGE_BACKEND settings. It will be using openedx DEFAULT_FILE_STORAGE.
  • close PROFILE_IMAGE_BACKEND settings disappeared after v16.0.4 #46

@Faraz32123 Faraz32123 self-assigned this Oct 10, 2024
@Faraz32123 Faraz32123 marked this pull request as draft October 17, 2024 10:08
@Faraz32123 Faraz32123 force-pushed the fix/add_profile_image_backend_for_k8s_deployment branch from 268de10 to 68b3c12 Compare October 17, 2024 12:25
@Faraz32123 Faraz32123 force-pushed the fix/add_profile_image_backend_for_k8s_deployment branch 2 times, most recently from 13410d8 to d0bb7c4 Compare October 29, 2024 12:11
@Faraz32123 Faraz32123 marked this pull request as ready for review October 30, 2024 07:02
@Faraz32123
Copy link
Collaborator Author

Profile image is now working! This PR is tested with both local and dev environment and with K8s environment as well.
@regisb and @DawoudSheraz can u guys have a look at it again. Thanks.

Copy link
Contributor

@regisb regisb left a comment

Choose a reason for hiding this comment

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

Looks good! Can you just add a comment in the PR to explain how to actually use this private bucket?

Copy link
Contributor

@DawoudSheraz DawoudSheraz left a comment

Choose a reason for hiding this comment

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

Tested it locally using sumac sandbox MINIO host. I uploaded the profile pic, stopped k8s, and then re-launched. The image was there after re-launch and could also be previewed from minio.

- Add PROFILE_IMAGE_BACKEND settings in tutor-minio using patch named
`openedx-lms-production-settings` so that profile images persist
in k8s deployment of openedx and profile images can work now for both
local(will use minio storage if minio plugin is enabled) and dev environment.
- In case of dev environment, backend expects the valid S3 `base_url`
key in the PROFILE_IMAGE_BACKEND settings. It will be using openedx `DEFAULT_FILE_STORAGE`.
@Faraz32123 Faraz32123 force-pushed the fix/add_profile_image_backend_for_k8s_deployment branch from d0bb7c4 to 15efa14 Compare October 31, 2024 13:46
@Faraz32123 Faraz32123 merged commit 163f955 into master Oct 31, 2024
2 checks passed
@Faraz32123 Faraz32123 deleted the fix/add_profile_image_backend_for_k8s_deployment branch October 31, 2024 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

PROFILE_IMAGE_BACKEND settings disappeared after v16.0.4
3 participants