fluent / fluent-operator

Operate Fluent Bit and Fluentd in the Kubernetes way - Previously known as FluentBit Operator
Apache License 2.0
578 stars 246 forks source link

Remove influxdb host validation in ClusterOutput #1363

Closed smallc2009 closed 1 day ago

smallc2009 commented 2 days ago

What this PR does / why we need it:

This PR removes the validation of the ipv4, ipv6 and hostname. This is because the Kubebuilder doesn't support using multiple format validations for a single field. In original code, there were three formation validations(hostname, ipv4, ipv6), but only the last one(ipv6) would be applied.

This PR also remedies port range. The highest valid port number is 65535, not 65536. Port 0 is a reserved port in TCP/IP

Which issue(s) this PR fixes:

Fixes #1359

Does this PR introduced a user-facing change?

Additional documentation, usage docs, etc.:

wenchajun commented 1 day ago

Thanks for your contribution.