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.

Sign up for GitHub

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

Jump to bottom

[edxec] Add missing health checks to app service #483

Open
jmaupetit opened this issue Apr 3, 2020 · 2 comments
Open

[edxec] Add missing health checks to app service #483

jmaupetit opened this issue Apr 3, 2020 · 2 comments
Assignees
@jmaupetit
Labels
feature request

Comments

@jmaupetit
Copy link
Contributor

jmaupetit commented Apr 3, 2020 •
edited
Loading

Purpose

For now, we don't have health checks implemented in the edxec app service.

Proposal

Add & configure Mozilla's Dockerflow 2019.5.0 (more recent releases dropped support for Django 1.8) in our ecommerce's fork and use health monitoring views as we usually do for other apps like Richie.
@jmaupetit jmaupetit self-assigned this Apr 3, 2020
@jmaupetit
Copy link
Contributor Author

Note that we will need to change health check URL to make it generic and compatible with the official ecommerce distribution: see https://github.com/edx/ecommerce/blob/496c053dd05b10eac1179c6303651ab316e2a715/ecommerce/core/views.py#L29

@jmaupetit
Copy link
Contributor Author

jmaupetit commented Apr 3, 2020 •
edited
Loading

As an alternative, instead of integrating Mozilla's Dockerflow, we may consider backporting the official health view from edx to our fork.

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

@jmaupetit jmaupetit

Labels
feature request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jmaupetit