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

Timeout sending data Open edX data to Richie #13

Closed
igobranco opened this issue Apr 18, 2024 · 2 comments
Closed

Timeout sending data Open edX data to Richie #13

igobranco opened this issue Apr 18, 2024 · 2 comments
Assignees

Comments

@igobranco
Copy link
Member

Open edX celery task that synchronizes course information from LMS/Studio to Richie is timing out.

From Sentry it is timing out after 5 seconds. It should be more than enough to Richie update itself.

Changes that need to be made:

  1. This celery task shouldn't raise any error;
  2. Replace the log error with a log warning;
  3. Increase the timeout (from 5 to 20 seconds) and make it configurable;

Screen shot from Sentry with the error:
Image

Error synchronizing course course-v1:INA+EIDCE+2024_T1 to richie site https://www.nau

message
Error synchronizing course course-v1:INA+EIDCE+2024_T1 to richie site https://www.nau.edu.pt/api/v1.0/course-runs-sync/

richie_openedx_sync.tasks
HTTPSConnectionPool(host='www.nau.edu.pt', port=443): Read timed out. (read timeout=5)

richie_openedx_sync.tasks.sync_course_run_information_to_richie

ReadTimeout
 
richie_openedx_sync.tasks.sync_course_run_information_to_richie

HTTPSConnectionPool(host='www.nau.edu.pt', port=443): Read timed out. (read timeout=5)
@igobranco igobranco self-assigned this Apr 18, 2024
@igobranco
Copy link
Member Author

igobranco commented Apr 18, 2024

The timeout is already configurable, like:

RICHIE_OPENEDX_SYNC_COURSE_HOOKS:
  - url: https://some.richie.education/api/v1.0/course-runs-sync/
    secret: my-secret
    timeout: 20

igobranco added a commit to fccn/nau_playbooks that referenced this issue Apr 18, 2024
igobranco added a commit that referenced this issue Apr 18, 2024
On some cases the default timeout is too short.
With this change we prevent the timeout error to be
raised and sent by sentry.
relates to #13
igobranco added a commit that referenced this issue Apr 18, 2024
Change logging integration to Richie from ERROR to
WARNING level.
relates to #13
igobranco added a commit that referenced this issue Apr 18, 2024
On some cases the default timeout is too short.
With this change we prevent the timeout error to be
raised and sent by sentry.
relates to #13
igobranco added a commit that referenced this issue Apr 18, 2024
Change logging integration to Richie from ERROR to
WARNING level.
relates to #13
igobranco added a commit that referenced this issue Apr 18, 2024
On some cases the default timeout is too short.
With this change we prevent the timeout error to be
raised and sent by sentry.
relates to #13
igobranco added a commit that referenced this issue Apr 18, 2024
Change logging integration to Richie from ERROR to
WARNING level.
relates to #13
@igobranco
Copy link
Member Author

We no longer receive those errors on Sentry!

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

No branches or pull requests

1 participant