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

🐛(edxapp) fix edxapp image used in development #103

Merged
merged 1 commit into from
Aug 28, 2018

Conversation

jmaupetit
Copy link
Contributor

Purpose

Celery workers have permission issues in development.

Proposal

ginkgo.1-1.0.3-dev docker image has permission issues to run Celery workers. This issue has been fixed since, so we must upgrade to the latest release to make Celery workers fully functional in development env_type.

ginkgo.1-1.0.3-dev docker image has permission issues to run Celery
workers. This issue has been fixed since, so we must upgrade to the
latest release to make Celery workers fully functionnal in development
env_type.
@jmaupetit jmaupetit force-pushed the fix-edxapp-image-for-dev branch from 39b809c to 65c6c20 Compare August 27, 2018 16:01
@@ -2,7 +2,7 @@
domain_name: "{{ lookup('env', 'OPENSHIFT_DOMAIN') }}.nip.io"

# Use development images in the development environment
edxapp_image_tag: "ginkgo.1-1.0.3-dev"
edxapp_image_tag: "ginkgo.1-1.0.6-dev"
Copy link
Contributor

Choose a reason for hiding this comment

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

I just published 1.0.7

Copy link
Contributor Author

@jmaupetit jmaupetit Aug 28, 2018

Choose a reason for hiding this comment

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

Yes, but it's related to #105, hence, the upgrade to 1.0.7 should be made in PR #105 (for production and development images). Deal?

@jmaupetit jmaupetit merged commit c29a40c into master Aug 28, 2018
@jmaupetit jmaupetit deleted the fix-edxapp-image-for-dev branch August 28, 2018 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants