-
Notifications
You must be signed in to change notification settings - Fork 397
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
Pin revive version #3796
Pin revive version #3796
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.
LGTM
e0b4674
to
77163fa
Compare
revive v1.3.5 wrongly includes a replace directive which breaks the go install command. This commit pins revive to e33fb87 that is the fix for this issue.
77163fa
to
27f176a
Compare
- Removes go.mod replace directive for github.com/kubernetes/cri-api. - Bump it to v0.29.0. More info: https://kubernetes.io/blog/2022/12/09/kubernetes-v1-26-release/#cri-v1alpha2-removed
27f176a
to
186bded
Compare
Thanks @rafaeldtinoco. I've made changes addressing @NDStrahilevitz concerns. LMK if it's ok. |
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, thanks for addressing my request :)
Commit 186bded has been reported to break some local environments. This is due to the local use of golang version from 1.21. To avoid this, use the same version (1.20.8) of the github workflow (tests) locally: https://github.com/aquasecurity/tracee/blob/main/.github/actions/build-dependencies/action.yaml#L18 |
Close: #3795
1. Explain what the PR does
186bded chore: bump k8s.io/cri-api
c51481b fix: pin revive to e33fb87
2. Explain how to test it
3. Other comments