Project

Profile

Help

Story #5444

closed

[Epic] Consolidate ProgressReporting into a single model so it's simpler for plugin writers

Added by bmbouter about 5 years ago. Updated almost 5 years ago.

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

100%

Estimated time:
(Total: 0:00 h)
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Sprint 59
Quarter:

Description

Problem

Currently the pulpcore.app.models.progress module provides three objects:

ProgressReport
ProgressBar
ProgressSpinner

These are confusing for plugin users and should be combined.

Solution

Delete ProgressBar and ProgressSpinner, and consolidate them into a single model. In the database they are already a single model since ProgressBar and ProgressSpinner are proxy models for ProgressReport.

This will be a plugin API breaking change and needs to be advertised to pulp-dev and also come with a fix for pulp_file.


Sub-issues 5 (0 open5 closed)

Ansible Plugin - Task #5468: Replace ProgressBar with ProgressReportCLOSED - CURRENTRELEASEfao89

Actions
RPM Support - Task #5469: Replace ProgressBar with ProgressReportCLOSED - CURRENTRELEASEfao89

Actions
Container Support - Task #5470: Replace ProgressBar with ProgressReportCLOSED - CURRENTRELEASEfao89

Actions
Python Support - Task #5471: Replace ProgressBar with ProgressReportMODIFIEDfao89

Actions
Debian Support - Task #5477: Replace ProgressBar with ProgressReportCLOSED - CURRENTRELEASEfao89

Actions

Related issues

Related to Pulp - Issue #5164: Progress Reports incorrectly update counts in error or canceled situationsCLOSED - NOTABUGActions

Also available in: Atom PDF