Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show total number of items in sync-tasks #537

Open
m-bucher opened this issue Jun 2, 2022 · 3 comments
Open

Show total number of items in sync-tasks #537

m-bucher opened this issue Jun 2, 2022 · 3 comments
Labels
.feature CHANGES/<issue_number>.feature Triage-Needed

Comments

@m-bucher
Copy link
Contributor

m-bucher commented Jun 2, 2022

Is your feature request related to a problem? Please describe.
pulp_deb progress_reports during sync tasks should show the total number of items to process. Currently it just says null:

"progress_reports": [
        {
            "code": "update.release_file",
            "done": 0,
            "message": "Update ReleaseFile units",
            "state": "running",
            "suffix": null,
            "total": null
        },
        {
            "code": "update.packageindex",
            "done": 10,
            "message": "Update PackageIndex units",
            "state": "running",
            "suffix": null,
            "total": null
        },
        {
            "code": "associating.content",
            "done": 544,
            "message": "Associating Content",
            "state": "running",
            "suffix": null,
            "total": null
        },
        {
            "code": "sync.downloading.artifacts",
            "done": 938,
            "message": "Downloading Artifacts",
            "state": "running",
            "suffix": null,
            "total": null
        }
    ]

Describe the solution you'd like
I reckon this could mean iterating over the PackageIndex before actually processing it, maybe a 'nicer' solution is possible.
Nevertheless, it would make the current state of the task more transparent.

@quba42
Copy link
Collaborator

quba42 commented Jun 7, 2022

Do other Pulp plugins implement this?

@mdellweg
Copy link
Member

mdellweg commented Jun 7, 2022

Maybe it is possible to keep the stream-processing, but set the value once the last PackageIndex has been fetched.
If we increased the total value for each item we put into the queue, it would probably be more confusing than helping.

@quba42 quba42 added .feature CHANGES/<issue_number>.feature and removed Feature labels Aug 23, 2023
@quba42
Copy link
Collaborator

quba42 commented Dec 18, 2023

OR-1242

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
.feature CHANGES/<issue_number>.feature Triage-Needed
Projects
None yet
Development

No branches or pull requests

3 participants