Story #5444
closed
[Epic] Consolidate ProgressReporting into a single model so it's simpler for plugin writers
100%
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.
Related issues
Updated by fao89 about 5 years ago
- Tracker changed from Issue to Task
- % Done set to 0
Added by Fabricio Aguiar about 5 years ago
Updated by fao89 about 5 years ago
- Status changed from NEW to POST
- Assignee set to fao89
- pulpcore - https://github.com/pulp/pulpcore/pull/293
- pulpcore-plugin - https://github.com/pulp/pulpcore-plugin/pull/127
- pulp_file - https://github.com/pulp/pulp_file/pull/277
- plugin_template - https://github.com/pulp/plugin_template/pull/105
- pulp-2to3-migrate - https://github.com/pulp/pulp-2to3-migrate/pull/13
- pulp_ansible - #5468 - https://github.com/pulp/pulp_ansible/pull/210
- pulp_deb - #5477 - https://github.com/pulp/pulp_deb/pull/117
- pulp_rpm - #5469 - https://github.com/pulp/pulp_rpm/pull/1447
- pulp_python - #5471 - https://github.com/pulp/pulp_python/pull/255
- pulp_docker - #5470 - https://github.com/pulp/pulp_docker/pull/414
Added by Fabricio Aguiar about 5 years ago
Revision 52c1eae3 | View on GitHub
replace ProgressBar with ProgressReport
Updated by fao89 about 5 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
Updated by fao89 about 5 years ago
- Related to Issue #5164: Progress Reports incorrectly update counts in error or canceled situations added
Added by Fabricio Aguiar about 5 years ago
Revision 130fb201 | View on GitHub
Replace ProgressBar with ProgressReport
ref #5444 https://pulp.plan.io/issues/5444 Required PR: https://github.com/pulp/pulpcore/pull/293 Required PR: https://github.com/pulp/pulpcore-plugin/pull/127 Required PR: https://github.com/pulp/pulp_file/pull/277
Added by Fabricio Aguiar about 5 years ago
Revision 130fb201 | View on GitHub
Replace ProgressBar with ProgressReport
ref #5444 https://pulp.plan.io/issues/5444 Required PR: https://github.com/pulp/pulpcore/pull/293 Required PR: https://github.com/pulp/pulpcore-plugin/pull/127 Required PR: https://github.com/pulp/pulp_file/pull/277
Added by Fabricio Aguiar about 5 years ago
Revision 130fb201 | View on GitHub
Replace ProgressBar with ProgressReport
ref #5444 https://pulp.plan.io/issues/5444 Required PR: https://github.com/pulp/pulpcore/pull/293 Required PR: https://github.com/pulp/pulpcore-plugin/pull/127 Required PR: https://github.com/pulp/pulp_file/pull/277
Added by Fabricio Aguiar about 5 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
Updated by Anonymous about 5 years ago
- Status changed from POST to MODIFIED
- % Done changed from 20 to 100
Applied in changeset pulpcore|e7dfa2c5e9739df1fc51b8f4ae2f6859a02e34ab.
Updated by bmbouter almost 5 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
.
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