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

Is there any way to disable progress bar shown during pushing of images ? #666

Closed
tarun818 opened this issue May 15, 2024 · 5 comments
Closed
Labels
discussion This issue is not a bug or feature and a conversation is needed to find an appropriate resolution

Comments

@tarun818
Copy link

I am using below snippet in my code base to push images

`// Create a CopyOptions object for imgpkg's copyOptions.Run()
copyOptions := imgpkg.NewCopyOptions(goUi.NewConfUI(goUi.NewNoopLogger()))
//...
//...
// Set fields according to parameters and default imgpkg configurations, as well as registry auth
copyOptions.RepoDst =
copyOptions.Concurrency = 1
//... and so on and finally, I call Run as shown below

copyOptions.Run()
`
This copyOptions.Run() will start a progress bar as shown here

I want to disable it. is this possible ?

@tarun818 tarun818 added the carvel triage This issue has not yet been reviewed for validity label May 15, 2024
@renuy renuy added discussion This issue is not a bug or feature and a conversation is needed to find an appropriate resolution and removed carvel triage This issue has not yet been reviewed for validity labels May 17, 2024
@praveenrewar
Copy link
Member

@tarun818 I don't think there's a way to disable the progress bar while using the Run function. Unfortunately we don't have apis for the copy command yet which would have been helpful here.

cc @joaopapereira

@joaopapereira
Copy link
Member

Hey
I am currently working on extracting the API for copy. Unfortunately, I do not have a lot of bandwidth to do it. When we extract it, it would be easier for you to call imgpkg.

@tarun818
Copy link
Author

@joaopapereira Thanks for your response.
If possible, can you share a tentative timeline for when we can expect this change?

@joaopapereira
Copy link
Member

I created a PR that will add the v1 API for copy #668. There is another PR that is being reviewed that does change the code in copy and we need to decide if we want to merge this first and add rebase that PR or the reverse.

@praveenrewar
Copy link
Member

The PR is merged. @tarun818 Could you give the copy apis a try now (we haven't made a release yet but you should be able to use the latest commit sha).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion This issue is not a bug or feature and a conversation is needed to find an appropriate resolution
Projects
Archived in project
Development

No branches or pull requests

4 participants