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
cilium/cmd: Remove deprecated policy_trace command #23550
Conversation
ffca7b8
to
34032de
Compare
34032de
to
4f61395
Compare
4f61395
to
8a60d4b
Compare
Remove /policy/resolve from swagger and re-generate related file again. Signed-off-by: Tam Mach <tam.mach@cilium.io>
8a60d4b
to
3ae5cc9
Compare
/test |
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.
Looks good. I believe there there is potentially even more code to be removed, i.e. some of the code here:
Lines 17 to 28 in 5feb34e
type Tracing int | |
const ( | |
TRACE_DISABLED Tracing = iota | |
TRACE_ENABLED | |
TRACE_VERBOSE | |
) | |
// TraceEnabled returns true if the SearchContext requests tracing. | |
func (s *SearchContext) TraceEnabled() bool { | |
return s.Trace != TRACE_DISABLED | |
} |
But it might be easier to remove that in a separate PR
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
/test-runtime |
/test-1.16-4.19 |
test-runtime hits #23297 |
/test-1.16-4.19 Re-run due to environment issue |
test-1.16-4.19 hits #23845 |
test-1.16-4.19 hits #23845 :(
|
Required code owner reviews are in, CI are all passed. Marking this ready to merge. |
Signed-off-by: Tam Mach tam.mach@cilium.io