Project

Profile

Help

Task #2232

Create Django post_save signal for Task model

Added by pcreech about 3 years ago. Updated 6 months ago.

Status:
CLOSED - WONTFIX
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
Start date:
Due date:
% Done:

0%

Platform Release:
Blocks Release:
Backwards Incompatible:
No
Groomed:
No
Sprint Candidate:
No
Tags:
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:

Description

The 2.y Task system sends an amqp signal during the model's save() message.

This functionality needs to be moved to a Django post_save signal for pulp 3

The method signature/attribute for the signal handler:

@receiver(post_save, sender=Task)
my_handler(sender, instance, created)

Django documenation:

https://docs.djangoproject.com/en/1.8/topics/signals/#connecting-to-signals-sent-by-specific-senders
https://docs.djangoproject.com/en/1.8/ref/signals/#post-save

History

#1 Updated by fdobrovo about 3 years ago

  • Blocked by Task #2300: Add serializer for the task models. added

#2 Updated by fdobrovo about 3 years ago

As far as I understand it first we need to have serialisation of models:
https://github.com/pulp/pulp/blob/3.0-dev/server/pulp/server/async/emit.py#L29

And also ported the function wchic send amqp signal:
https://github.com/pulp/pulp/blob/3.0-dev/server/pulp/server/async/emit.py#L10-L41

#3 Updated by bmbouter about 3 years ago

fdobrovo wrote:

As far as I understand it first we need to have serialisation of models:
https://github.com/pulp/pulp/blob/3.0-dev/server/pulp/server/async/emit.py#L29

And also ported the function wchic send amqp signal:
https://github.com/pulp/pulp/blob/3.0-dev/server/pulp/server/async/emit.py#L10-L41

Both of these statements are correct.

#4 Updated by dalley over 1 year ago

  • Blocked by deleted (Task #2300: Add serializer for the task models.)

#5 Updated by dalley over 1 year ago

  • Status changed from NEW to CLOSED - WONTFIX

#6 Updated by daviddavis 6 months ago

  • Sprint/Milestone set to 3.0

#7 Updated by bmbouter 6 months ago

  • Tags deleted (Pulp 3)

Please register to edit this issue

Also available in: Atom PDF