search

netsampler / goflow2

High performance sFlow/IPFIX/NetFlow Collector
BSD 3-Clause "New" or "Revised" License
487 stars 112 forks source link

Do not deprecate srv record #260

Open simPod opened 11 months ago

simPod commented 11 months ago

I find it kind of handy since it simplifies broker management via one central point.

lspgn commented 11 months ago

I think that creates a dependency on DNS and does not track well changes to it (or would create the assumption you just need to update the SRV for GoFlow2 to automatically send to the new brokers). In Kube or with automation, this make the SRV redundant. This said I don't have plans to remove this code just yet.

simPod commented 11 months ago

I think that creates a dependency on DNS and does not track well changes to it (or would create the assumption you just need to update the SRV for GoFlow2 to automatically send to the new brokers).

I understand. Maybe adding a doc could help to battle such assumption. I usually use DNS in brokers list as well so having a dependency on it is not such an issue.