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

Task context manager not properly clearing old task results #4599

Closed
bjester opened this issue Jun 27, 2024 · 0 comments · Fixed by #4602
Closed

Task context manager not properly clearing old task results #4599

bjester opened this issue Jun 27, 2024 · 0 comments · Fixed by #4602
Assignees
@ozer550
Labels
DEV: backend P0 - critical Priority: Release blocker or regression TAG: regression
Milestone
Studio: Q2-24 Rel...

Comments

@bjester
Copy link
Member

bjester commented Jun 27, 2024 •
edited
Loading

Observed behavior

The context manager for managing task progress along with a change event does not pass an existing task ID properly to a deletion query.

Expected behavior

The code should properly pass the existing task_id to the deletion query, instead of the queryset

User-facing consequences

Repeated channel publishing encounters this error.

Errors and logs

image

Steps to reproduce the issue

  1. Create a channel
  2. Add content
  3. Publish the channel
  4. Add more content
  5. Try to publish again

Usage Details

  • unstable branch
@bjester bjester added P0 - critical Priority: Release blocker or regression DEV: backend TAG: regression labels Jun 27, 2024
@bjester bjester added this to the Studio: Q2-24 Release milestone Jun 27, 2024
@ozer550 ozer550 self-assigned this Jun 27, 2024
Keshav123454 added a commit to Keshav123454/studio that referenced this issue Jul 6, 2024
@Keshav123454
learningequality#4599 - context manager send task_id queryset only to…
69419ef 
… delete task_id
This was referenced Jul 7, 2024
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ozer550 ozer550

Labels
DEV: backend P0 - critical Priority: Release blocker or regression TAG: regression
Projects
None yet
Milestone
Studio: Q2-24 Release
Development

Successfully merging a pull request may close this issue.

Fix regression with publish task ozer550/studio
2 participants
@bjester @ozer550