Tam Mach

Results 220 comments of Tam Mach

Thanks a lot for your fix 💯 , this issue was annoying as part of day to day cli usage 🎖️

Thanks all for your detailed review, I have added additional policy for deny tests, so that default deny can be avoided. Extra policies could be a little permissive, but should...

> To me it looks like this PR is now introducing quite a bit more than just deny policy tests, e.g. also tests for named ports and matched expressions. I...

the error looks similar, we have a few fixes below, but it's might not fix for hubble enable CLI. Can you give it a crack with latest master branch and...

Confirmed that it's still happening, just curious if any of you both is planning to contribute. Thanks. ``` $ cilium hubble enable Error: Unable to enable Hubble: unable to retrieve...

Thanks for your help on this, feel free to reach out in Cilium OSS #development channel if you have any questions.

/test Job 'Cilium-PR-K8s-1.25-kernel-net-next' failed: Click to show. ### Test Name ```test-name K8sDatapathConfig Check BPF masquerading with ip-masq-agent DirectRouting ``` ### Failure Output ```failure-output FAIL: Failed to add ip route ```...

Thanks for your review comments all, seems like all reviewers have added comments. Marking this as draft to address/incorporate your comments.

/test Job 'Cilium-PR-K8s-1.16-kernel-4.9' failed: Click to show. ### Test Name ```test-name K8sDatapathVerifier Runs the kernel verifier against Cilium's BPF datapath ``` ### Failure Output ```failure-output FAIL: Failed to load BPF...