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
cmd/operator-sdk: add bundle validate
command
#2411
cmd/operator-sdk: add bundle validate
command
#2411
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few nits + fix ci, otherwise it shows fine for me.
Should not the WIP be removed? What is missing get done?
LGTM, just some small comments |
11d3dcf
to
fae504d
Compare
/hold Waiting on operator-framework/operator-registry#146 to update |
bundle validate
commandbundle validate
command
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
Just one question. But it seems like any change to fix that, if necessary, would be in operator-registry
.
logger := log.WithFields(log.Fields{"container-tool": c.imageBuilder}) | ||
log.SetLevel(log.DebugLevel) | ||
val := bundle.NewImageValidator(c.imageBuilder, logger) | ||
if err = val.PullBundleImage(c.imageTag, dir); err != nil { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this command work if I have a bundle image on my local system and I haven't pushed it to a registry yet?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apparently not. I tested this command with a few images: local ones all returning the same error, while registry images were fine. I'll add a note here.
Additionally error messages can be improved in PullBundleImage
, as right now the only message I'm seeing is exit status 1
@exdx @dinhxuanvu.
New changes are detected. LGTM label has been removed. |
CLI wrapper for operator-registry bundle content and format validators go.mod,go.sum: pin operator-registry commit containing validation functions doc/cli: regenerate CHANGELOG.md: add bundle validate addition
8eee978
to
4c28290
Compare
/hold cancel |
Description of the change:
Motivation for the change:
bundle validate
can be run on a bundle image manually built after runningbundle build --generate-only
.