Project

Profile

Help

Story #5444

closed

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

Added by bmbouter over 4 years ago. Updated over 4 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
Actions #1

Updated by fao89 over 4 years ago

  • Tracker changed from Issue to Task
  • % Done set to 0
Actions #2

Updated by bmbouter over 4 years ago

  • Sprint/Milestone changed from 71 to 3.0.0
Actions #3

Updated by bmbouter over 4 years ago

  • Description updated (diff)
Actions #4

Updated by fao89 over 4 years ago

  • Sprint set to Sprint 59

Added by Fabricio Aguiar over 4 years ago

Revision 5f074af7 | View on GitHub

Replace ProgressBar with ProgressReport

ref #5444 Required PR: https://github.com/pulp/pulpcore/pull/293 Required PR: https://github.com/pulp/pulpcore-plugin/pull/127

Added by Fabricio Aguiar over 4 years ago

Revision 52c1eae3 | View on GitHub

replace ProgressBar with ProgressReport

ref #5444 https://pulp.plan.io/issues/5444

Actions #6

Updated by fao89 over 4 years ago

  • Tracker changed from Task to Story
  • Subject changed from Consolidate ProgressReporting into a single model so it's simpler for plugin writers to [Epic] Consolidate ProgressReporting into a single model so it's simpler for plugin writers
Actions #7

Updated by fao89 over 4 years ago

  • Related to Issue #5164: Progress Reports incorrectly update counts in error or canceled situations added

Added by Fabricio Aguiar over 4 years ago

Revision e7dfa2c5 | View on GitHub

ProgressReport for known and unknown items count

closes #5444 https://pulp.plan.io/issues/5444 Required PR: https://github.com/pulp/pulpcore-plugin/pull/127 Required PR: https://github.com/pulp/pulp_file/pull/277

Actions #8

Updated by Anonymous over 4 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 20 to 100
Actions #9

Updated by bmbouter over 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF