bump celery and a related dependency - attempt to fix secondary+core #3287
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Adrian Edwards <adredwar@redhat.com>
Member
|
Merging all of the changes under review into a second load test branch. This branch includes what is in load-test plus the reversion of the facade_tasks.py changes from last week, and bumping the celery version. |
Contributor
Author
doing this just closed this PR, which probably wasnt intentional but I think its worth revisiting our strategies for making changes |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
this bumps celery's version because i noticed in the changelog for 5.4 rc1
I didnt look too deeply into it but i wonder if this could be related to the issue with core and secondary not doing what its supposed to (i.e. the revert in #3284 )
Notes for Reviewers
Signed commits