[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[GitHub] brooklyn-dist pull request #121: [BROOKLYN-579] Add DNS TTL configuration vi...

GitHub user tbouron opened a pull request:


    [BROOKLYN-579] Add DNS TTL configuration via envvar

    Before, java was not refreshing the DNS records, ever. This is due to the default `networkaddress.cache.ttl` set to `-1`, i.e. cache forever.
    This PR updates the system property `sun.net.inetaddr.ttl` to `60` by default. It uses `sun.net.inetaddr.ttl` because `networkaddress.cache.ttl` **is not** a system property. But this has the desirable effect (see: https://stackoverflow.com/a/17219327)

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/tbouron/brooklyn-dist feature/dns-ttl

Alternatively you can review and apply these changes as the patch at:


To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #121
commit 2962a166979bbead588a172a378bac0678c12ba0
Author: Thomas Bouron <thomas.bouron@...>
Date:   2018-05-24T12:44:46Z

    Add DNS TTL configuration via envvar