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

Add progress_bar option to image pull #340

Merged
merged 1 commit into from
Nov 13, 2023

Conversation

umohnani8
Copy link
Member

Add a new arg option called progress_bar that when set to true will display a progress bar with the image pull progress. This uses the compat endpoint as that is the data we are able to parse to displat the progress bar.

@openshift-ci openshift-ci bot added the approved label Nov 9, 2023
@umohnani8
Copy link
Member Author

umohnani8 commented Nov 9, 2023

This is what the progress bar looks like - it is basically a bar for each layer being pulled and switches from Downloading to Download complete when done.

Screenshot from 2023-11-09 12-59-18

@umohnani8
Copy link
Member Author

Gating will fail till #339 is merged.

I also don't see a way of adding tests for this.

@jwhonce @baude @rhatdan @lsm5 PTAL

@rhatdan
Copy link
Member

rhatdan commented Nov 9, 2023

Nice work

LGTM

@rhatdan
Copy link
Member

rhatdan commented Nov 9, 2023

/approve

@umohnani8 umohnani8 force-pushed the pull-progress branch 5 times, most recently from 6250a3c to 912b41d Compare November 10, 2023 12:05
Add a new arg option called progress_bar that when set to
true will display a progress bar with the image pull progress.
This uses the compat endpoint as that is the data we are able
to parse to displat the progress bar.

Signed-off-by: Urvashi Mohnani <[email protected]>
@umohnani8
Copy link
Member Author

@jwhonce @baude @cevich @lsm5 tests are green! PTAL

Copy link
Member

@lsm5 lsm5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/hold

Copy link
Contributor

openshift-ci bot commented Nov 13, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lsm5, rhatdan, umohnani8

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:
  • OWNERS [lsm5,rhatdan,umohnani8]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@rhatdan
Copy link
Member

rhatdan commented Nov 13, 2023

/hold cancel

@openshift-merge-bot openshift-merge-bot bot merged commit 47ce668 into containers:main Nov 13, 2023
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants