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

build: removing django-celery-results constraint. #30381

Merged
merged 5 commits into from
May 16, 2022
Merged

build: removing django-celery-results constraint. #30381

merged 5 commits into from
May 16, 2022

Conversation

awais786
Copy link
Contributor

@awais786 awais786 commented May 12, 2022 •
edited
Loading 

# django-celery-results==2.3.0 requires psycopg2 as dependency but hasn't explicitly added in package requirements.
# The PR https://github.com/celery/django-celery-results/pull/292 fixing this issue has been merged but
# new version hasn't been released yet with the fix so pinning the version until django-celery-results>2.3.0 is out.
django-celery-results<2.3.0

https://github.com/celery/django-celery-results/pull/292/files

Important PR.
celery/django-celery-results#261

@awais786 awais786 marked this pull request as ready for review May 13, 2022 16:20
@awais786
Copy link
Contributor Author

I did't find any data in prod tables.

 django_celery_results_chordcounter                               |
| django_celery_results_groupresult                                |
| django_celery_results_taskresult

@awais786 awais786 merged commit 9f904e8 into master May 16, 2022
@awais786 awais786 deleted the remove-dj-celery-res-pin branch May 16, 2022 10:00
@edx-pipeline-bot
Copy link
Contributor

EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production.

@edx-pipeline-bot
Copy link
Contributor

EdX Release Notice: This PR has been deployed to the production environment.

1 similar comment
@edx-pipeline-bot
Copy link
Contributor

EdX Release Notice: This PR has been deployed to the production environment.

@edx-pipeline-bot
Copy link
Contributor

EdX Release Notice: This PR has been rolled back from the production environment.

@edx-pipeline-bot
Copy link
Contributor

EdX Release Notice: This PR has been deployed to the production environment.

1 similar comment
@edx-pipeline-bot
Copy link
Contributor

EdX Release Notice: This PR has been deployed to the production environment.

jawad-khan pushed a commit that referenced this pull request Jun 14, 2022
@awais786 @edx-requirements-bot @jawad-khan
build: removing django-celery-results constraint. (#30381)
3621b8b 
* build: removing django-celery-results constraint.

Co-authored-by: edX requirements bot <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@iamsobanjaved iamsobanjaved iamsobanjaved approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@awais786 @edx-pipeline-bot @iamsobanjaved @edx-requirements-bot