-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Updated k8s-nginx-ingress parser for NGINX Controller #1869
Conversation
Updated parser to support latest helm release Helm Version: `nginx-ingress-1.28.0` App Version: `0.26.2 ` Signed-off-by: Arjun Sunil <arjun921@gmail.com>
@PettitWesley What can I do to fix all checks? |
Also, not certain (may be these repos get synced on a release?), but I think you should update the parser here too: https://github.com/fluent/fluent-bit-docker-image/tree/1.3 |
@arjun921 would you please also attach a https://rubular.com short link with data sample and the regex ?, that would be useful for the record. |
I will update it for sure!
Here you go @edsiper I'll add it as a comment too in the parser for reference. |
Signed-off-by: Arjun Sunil <arjun921@gmail.com>
@edsiper @PettitWesley Any chance we could merge this now 😅? |
thanks |
Hi, the new regular expression here is missing the |
@robberlang Could you submit a pull request to add that field? Thanks! |
@robberlang Which version of Nginx Ingress Controller are you using? |
Updated parser to support latest helm release
Helm Version:
nginx-ingress-1.28.0
App Version:
0.26.2