-
Notifications
You must be signed in to change notification settings - Fork 2
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
scripts: Fix net-istio version for 1.9 scanning #221
Conversation
Signed-off-by: Kimonas Sotirchos <[email protected]>
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.
Thanks @kimwnasptd, left a comment.
22833f6
to
c76d750
Compare
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.
Looking good overall, just two tiny nits. Thanks @kimwnasptd !
Co-authored-by: Daniela Plascencia <[email protected]>
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.
Thanks @kimwnasptd !
* scripts: Fix net-istio version for 1.9 scanning Signed-off-by: Kimonas Sotirchos <[email protected]> * Add CONTRIBUTING.md * Hardcode the net istio version * Update CONTRIBUTING.md Co-authored-by: Daniela Plascencia <[email protected]> * review: Add extra message for where to look for net-istio * fix typo --------- Signed-off-by: Kimonas Sotirchos <[email protected]> Co-authored-by: Daniela Plascencia <[email protected]>
Closes #219
The PR hardcodes the version of net-istio to be used for the 1.12.4 version of Knative Service (used in CKF 1.9), similarly to how we handled it for CKF 1.8.
To verify: