Story #2402: As an API user, a call to update an Importer generates a Task - Pulp

Send by e-mail

Story #2402

As an API user, a call to update an Importer generates a Task

Added by ttereshc almost 4 years ago. Updated 10 months ago.

Status:

CLOSED - CURRENTRELEASE

Priority:

Normal

Assignee:

dkliban@redhat.com

Category:

Sprint/Milestone:

3.0.0

Start date:

Due date:

% Done:

100%

Estimated time:

Platform Release:

Groomed:

Yes

Sprint Candidate:

Yes

Tags:

Sprint:

Sprint 21

Quarter:

Description

We need to create an update task which can live in pulp.app.tasks.importer.

Task name in Pulp 2: `pulp.server.managers.repo.importer.update_importer_config`.
Task implementation in Pulp 2: https://github.com/pulp/pulp/blob/3.0-dev/server/pulp/server/controllers/importer.py#L240

Hide closed

Checklist

Ensure the viewset uses the serializer to validate the user's data
Create the pulp.app.tasks.importer.update task
Have the viewset call pulp.app.tasks.importer.update task
Write out some commands to test with so the review can easily test
delete old code from /server

Related issues

Associated revisions

Revision f92a4255 View on GitHub
Added by dkliban@redhat.com about 3 years ago

Problem: importer updates are performed syncronously

Solution: override the update method in the Importer viewset

The update method now dispatches a task to update the importer.

closes #2402 https://pulp.plan.io/issues/2402

Revision f92a4255 View on GitHub
Added by dkliban@redhat.com about 3 years ago

Problem: importer updates are performed syncronously

Solution: override the update method in the Importer viewset

The update method now dispatches a task to update the importer.

closes #2402 https://pulp.plan.io/issues/2402

History

#1 Updated by ttereshc almost 4 years ago

Related to Task #2380: Create a redmine task for each 2.y celery task to be converted to 3.0 added

#2 Updated by ttereshc almost 4 years ago

Tags Pulp 3 added

#3 Updated by mhrivnak almost 4 years ago

Groomed changed from No to Yes
Sprint Candidate changed from No to Yes

#4 Updated by mhrivnak almost 4 years ago

This comment also applies here: https://pulp.plan.io/issues/2409#note-4

#5 Updated by bmbouter over 3 years ago

Tracker changed from Task to Story
Subject changed from Convert celery task repo.importer.update_importer_config to Pulp 3 to As an API user, a call to update an importer generates a Task
Description updated (diff)

Rewriting based on similarity to issue 2756 and discussion on the mailing list: https://www.redhat.com/archives/pulp-dev/2017-May/msg00014.html

#6 Updated by bmbouter over 3 years ago

Subject changed from As an API user, a call to update an importer generates a Task to As an API user, a call to update an Importer generates a Task