Story #5028: RepositoryVersion.create() can't be called outside of a task - Pulp

Actions

Send by e-mail Copy link

Story #5028

closed

RepositoryVersion.create() can't be called outside of a task

Added by dkliban@redhat.com almost 5 years ago. Updated over 4 years ago.

Status:

CLOSED - CURRENTRELEASE

Priority:

Normal

Assignee:

fao89

Category:

Sprint/Milestone:

3.0.0

Start date:

Due date:

% Done:

100%

Estimated time:

Platform Release:

Groomed:

Yes

Sprint Candidate:

Tags:

Sprint:

Sprint 60

Quarter:

Description

Docker push requires creating a repository version without dispatching a task. This is currently impossible because the RepositoryVersion.create() method always creates a CreatedResource[0]. A CreatedResource requires a task_id to be saved to the database.

Solution¶

Have the RepositoryVersion.create() check if it's running inside a task. You can do that with Task.current [1].

[0]: https://github.com/pulp/pulpcore/blob/06c7412525c0196169690125eecd63fad3548752/pulpcore/app/models/repository.py#L435
[1]: https://github.com/pulp/pulpcore/blob/8c3798cee5935c6c01af0cf83bdea2359926bfad/pulpcore/app/models/task.py#L339-L350

Actions

Copy link

Updated by dkliban@redhat.com almost 5 years ago

Tracker changed from Issue to Story
% Done set to 0

Actions

Copy link

Updated by dkliban@redhat.com almost 5 years ago

This was filed as part of the docker push workflow work. A possible alternative solution to the problem has been discussed on IRC. This solution would allow plugin writers to create content sets that are not repository versions. Plugin writers would then be able to dispatch a task to either create a repository version from that whole set or to add that content to an existing repository version.

Actions

Copy link

Updated by dkliban@redhat.com over 4 years ago

We should implement this as written in the description. The Docker API is synchronous. It needs to be able to create repository versions synchronously.

Actions

Copy link

Updated by bmbouter over 4 years ago

I don't think the plugin writer benefits from another parameter to allow this to run outside of a task or not. The plugin writer is creating it, and we should allow them to have it wherever they want.

Actions

Copy link