Project

Profile

Help

Task #2398

closed

Convert celery task repo.publish to Pulp 3

Added by ttereshc over 7 years ago. Updated over 4 years ago.

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

100%

Estimated time:
Platform Release:
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
Sprint:
Sprint 20
Quarter:

Description

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

1. Query for the instance of the Publisher's Master model that needs to publish.
2. Cast that Publisher master instance to a detail instance
3. Set the working_dir attribute on the detail instance with the working_dir context manager.
4. Call publish (it should take no arguments)

The comments in platform/pulp/plugin/models/publisher.py need to be revised.


Related issues

Related to Pulp - Task #2380: Create a redmine task for each 2.y celery task to be converted to 3.0CLOSED - CURRENTRELEASEttereshc

Actions
Actions #1

Updated by ttereshc over 7 years ago

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

Updated by ttereshc over 7 years ago

  • Tags Pulp 3 added
Actions #3

Updated by amacdona@redhat.com almost 7 years ago

  • Description updated (diff)
  • Groomed changed from No to Yes

We probably ought to do some business checks related to preventing relative_path collisions as well.

Actions #4

Updated by bmbouter almost 7 years ago

wrote:

We probably ought to do some business checks related to preventing relative_path collisions as well.

This sounds good, but I can't picture what it looks like. Can you describe the check a little more?

Actions #5

Updated by dkliban@redhat.com almost 7 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to dkliban@redhat.com
  • Sprint/Milestone set to 38
  • Sprint Candidate changed from No to Yes
Actions #6

Updated by dkliban@redhat.com almost 7 years ago

  • Status changed from ASSIGNED to POST
Actions #7

Updated by amacdona@redhat.com almost 7 years ago

bmbouter wrote:

wrote:

We probably ought to do some business checks related to preventing relative_path collisions as well.

This sounds good, but I can't picture what it looks like. Can you describe the check a little more?

We do it in rpm now, but I think it should be a general tool.

https://github.com/pulp/pulp_rpm/blob/8720816fc489e6dd18e075896ed7ba0bf70541d8/plugins/pulp_rpm/plugins/distributors/yum/configuration.py#L494

Actions #8

Updated by mhrivnak almost 7 years ago

  • Sprint/Milestone changed from 38 to 39

Added by dkliban@redhat.com almost 7 years ago

Revision bf743323 | View on GitHub

Problem: API does not support publishing a repository

Solution: Add API endpoint to dispatch a publish with a specific publisher.

closes #2398

Added by dkliban@redhat.com almost 7 years ago

Revision bf743323 | View on GitHub

Problem: API does not support publishing a repository

Solution: Add API endpoint to dispatch a publish with a specific publisher.

closes #2398

Actions #9

Updated by dkliban@redhat.com almost 7 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100
Actions #10

Updated by bmbouter about 6 years ago

  • Sprint set to Sprint 20
Actions #11

Updated by bmbouter about 6 years ago

  • Sprint/Milestone deleted (39)
Actions #12

Updated by daviddavis almost 5 years ago

  • Sprint/Milestone set to 3.0.0
Actions #13

Updated by bmbouter almost 5 years ago

  • Tags deleted (Pulp 3)
Actions #14

Updated by bmbouter over 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF