Project

Profile

Help

Story #5444

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

Added by bmbouter about 1 month ago. Updated 25 days ago.

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

100%

Platform Release:
Blocks Release:
Backwards Incompatible:
No
Groomed:
No
Sprint Candidate:
No
Tags:
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:
Sprint 59

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.


Subtasks

Ansible Plugin - Task #5468: Replace ProgressBar with ProgressReportMODIFIEDfabricio.aguiar

Actions
RPM Support - Task #5469: Replace ProgressBar with ProgressReportMODIFIEDfabricio.aguiar

Actions
Docker Support - Task #5470: Replace ProgressBar with ProgressReportMODIFIEDfabricio.aguiar

Actions
Python Support - Task #5471: Replace ProgressBar with ProgressReportMODIFIEDfabricio.aguiar

Actions
Debian Support - Task #5477: Replace ProgressBar with ProgressReportMODIFIEDfabricio.aguiar

Actions

Related issues

Related to Pulp - Issue #5164: Progress Reports incorrectly update counts in error or canceled situations CLOSED - NOTABUG Actions

Associated revisions

Revision 5f074af7 View on GitHub
Added by Fabricio Aguiar 28 days ago

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

Revision 52c1eae3 View on GitHub
Added by Fabricio Aguiar 28 days ago

replace ProgressBar with ProgressReport

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

Revision e7dfa2c5 View on GitHub
Added by Fabricio Aguiar 26 days ago

History

#1 Updated by fabricio.aguiar about 1 month ago

  • Tracker changed from Issue to Task
  • % Done set to 0

#2 Updated by bmbouter about 1 month ago

  • Sprint/Milestone changed from 71 to 3.0

#3 Updated by bmbouter 28 days ago

  • Description updated (diff)

#4 Updated by fabricio.aguiar 28 days ago

  • Sprint set to Sprint 59

#6 Updated by fabricio.aguiar 28 days 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

#7 Updated by fabricio.aguiar 28 days ago

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

#8 Updated by Anonymous 25 days ago

  • Status changed from POST to MODIFIED
  • % Done changed from 20 to 100

Please register to edit this issue

Also available in: Atom PDF