Stu Tomlinson

> Hi @nosnilmot - thanks for your contribution! Running the test suite on it now. Hi @danielleadams - I see the CI tests have failed, but I do not think...

@danielleadams I have rebased, and the automated CI tests have passed. Could you rerun the test suite please?

> @nosnilmot Thanks for the PR. Would you be able to open an issue with the problem description and link it in this PR? Certainly, I have opened #13534

@sguilhen do I need to do anything else to get this merged? It looks like you have approved the changes but github is still saying 'Review required: At least 1...

After quite some considerable effort I have been able to run the 'Keycloak CI / Quickstarts Tests' tests locally to try to reproduce the failure, and ... it does not...

ping! (excuse the pun) is there any prospect of this feature being implemented? @bassosimone did anything come from your implementatin/evaluation? I'm also interested in having this feature (application level latency-only...

Nothing in this PR actually changes the behavior of ejabberd, most of it is changes to things where order has no significance. The default for `outgoing_s2s_families` is set here and...

> I could see some flag that would make us preffer one address above other That's exactly what `outgoing_s2s_families` is. The (current) [documentation](https://docs.ejabberd.im/admin/configuration/toplevel/#outgoing-s2s-families) for `outgoing_s2s_families` states: > Specify which address...

> @nosnilmot I'm running integration tests, but I would like to follow up with questions for other team members before we merge. I'll keep you posted on the progress of...

@danielleadams ping?