-
Notifications
You must be signed in to change notification settings - Fork 111
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
Make OVS_CNI_IMAGE truly optional #683
Make OVS_CNI_IMAGE truly optional #683
Conversation
The `hack/env.sh` script expected that `OVS_CNI_IMAGE` is set, and exited with an error when it was not. There is no good reason for erroring out on missing variable for an optional image. Instead, set the variable to empty string when it is unset. When `SKIP_VAR_SET` is set, the default for `OVS_CNI_IMAGE` remains a valid image.
Thanks for your PR,
To skip the vendors CIs use one of:
|
495f797
to
aa1ba65
Compare
Thanks for your PR,
To skip the vendors CIs use one of:
|
Pull Request Test Coverage Report for Build 8722690509Details
💛 - Coveralls |
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
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
@ykulazhenkov any objections here? |
no, LGTM |
The
hack/env.sh
script expected thatOVS_CNI_IMAGE
is set, and exited with an error when it was not. There is no good reason for erroring out on missing variable for an optional image. Instead, set the variable to empty string when it is unset.When
SKIP_VAR_SET
is set, the default forOVS_CNI_IMAGE
remains a valid image.