Project

Profile

Help

Refactor #4346

closed

Simplify the plugin api for writing stages

Added by mdellweg almost 6 years ago. Updated almost 5 years ago.

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

0%

Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Quarter:

Description

Problem

With the invent of content with futures, that are able to feed back in earlier stages of the pipeline, the need came up for some sort of flow control in the batching of content process to prevent deadlocks [0].

To keep full authority over any future implementation of this flow control, i suggest to refactor the stages api with a simple but strict definition, what the implementation of a stage must do / can rely on.

Solution

  1. A stage must override async def run(self):.
  2. In run, items can be retrieved through either self.items or self.batches iterator.
  3. ..., items must be forwarded by self.put.

Further discussion can be found here [1].

[0] https://pulp.plan.io/issues/4296
[1] https://github.com/pulp/pulpcore-plugin/pull/35

Actions #1

Updated by gmbnomis almost 6 years ago

+1 to the alternative using run; one thing less that stage writers may forget (calling finish)

Actions #2

Updated by bmbouter almost 6 years ago

+1 also to the run approach.

It looks like that is the current code pushed to this PR https://github.com/pulp/pulpcore-plugin/pull/35/files is that right? Does that mean https://github.com/pulp/pulpcore-plugin/pull/35/files is ready for review?

Actions #3

Updated by mdellweg almost 6 years ago

Yes, that is the actual version (i just could not wait.). Ready for review.

Actions #4

Updated by daviddavis almost 6 years ago

  • Tags Pulp 3, Pulp 3 RC Blocker added
Actions #5

Updated by bmbouter almost 6 years ago

  • Status changed from NEW to POST
  • Assignee set to mdellweg

Added by dalley almost 6 years ago

Revision 5ed0431d | View on GitHub

Adapt to changes in the stages API

re #4346 https://pulp.plan.io/issues/4346

Actions #6

Updated by bmbouter over 5 years ago

  • Description updated (diff)
  • Status changed from POST to MODIFIED

All the work for this is done so I'm moving to MODIFIED and rewriting with its final solution

Actions #7

Updated by daviddavis over 5 years ago

  • Sprint/Milestone set to 3.0.0
Actions #8

Updated by bmbouter over 5 years ago

  • Tags deleted (Pulp 3, Pulp 3 RC Blocker)
Actions #9

Updated by bmbouter almost 5 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF