[Closed] Add standalone convert mode for SOCI index creation without containerd#1864
Closed
prafgup wants to merge 1 commit into
Closed
[Closed] Add standalone convert mode for SOCI index creation without containerd#1864prafgup wants to merge 1 commit into
prafgup wants to merge 1 commit into
Conversation
VVKot
suggested changes
Feb 17, 2026
github-actions
Bot
added
go
prafgup
force-pushed
the
main
branch
2 times, most recently
from
a398342 to
ff06e93
Compare
prafgup
force-pushed
the
main
branch
3 times, most recently
from
3645f13 to
17a2cc6
Compare
prafgup
commented
Feb 18, 2026
Contributor
Author
|
#1867 #1867 (comment) would lead to merge conflicts as its touching the input flags behaviors, I would rebase with main in case that PR is merged soon. Also would refactor flags such as |
prafgup
changed the title
prafgup
commented
Feb 19, 2026
Comment on lines
+92
to
+96
| &cli.BoolFlag{ | ||
| Name: "verbose", | ||
| Aliases: []string{"v"}, | ||
| Usage: "Show detailed progress output", | ||
| }, |
Contributor
Author
There was a problem hiding this comment.
I was also thinking of implementing something like a quiet flag instead which would be the opposite of verbose (used in push command) which would default to "always log" and when quiet is set it would not log these extra info logs.
Currently the convert command (default) anyways always outputs something like -
ztoc skipped - layer sha256:4f4fb700ef54461cfa02571ae0db9a0dc1e0cdb5577484a6d75e68dc38e8acc1 (application/vnd.oci.image.layer.v1.tar+gzip) size 32 is less than min-layer-size 10485760
...
layer sha256:3e9c02b40ec11c1a3cc84f775102e2a3b100700fc7753098adc6d62e3fc6563b -> ztoc sha256:0aa198e2f2f6ff02675b7f7dc27e008a1b81bc15a87a5e4d5f868259a3cdc5f0
thus I kept extra logging as verbose instead of making it a opt-out. Let me know if quiet makes more sense here than verbose.
VVKot
reviewed
Feb 19, 2026
Signed-off-by: Praful Gupta <prafulgupta6@gmail.com>
prafgup
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
New PR - #1870