-
Notifications
You must be signed in to change notification settings - Fork 20
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
feat: add istio example (#29) #32
Conversation
Hi @marcelloale Could you please include the other fields in the input data of the example? I see that you've included only Please run also |
Hi, @marcelloale Here are some examples of Istio/Envoy attributes for CEL: https://gist.github.com/matheusfm/31e96d49159937266e5bb88d69fddb91 This gist contains three YAML-formatted examples, where Could you please consider using this example as input data? PS: A big thank you to @howardjohn for their help with these examples. |
Hi @matheusfm , I've been occupied over the past week, but I intend to make the necessary adjustments as soon as possible this weekend. I want to thank @matheusfm and @howardjohn for the examples. |
Hi @matheusfm, I've added the new input data, but I removed some 'null' fields to reduce the size of the example. If necessary, I can commit the original example without any alterations. |
Thank you @marcelloale for the collaboration! Could you resolve git conflicts please? Then we can merge these changes. |
@matheusfm done |
Thank you very much @marcelloale for your contribution and collaboration. We really appreciate it. |
Description
Add Istio example access log filtering using CEL
Linked Issues
#29