Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Multiple A records support for the same FQDN #1475

Merged
merged 5 commits into from
Apr 15, 2020
Merged

Multiple A records support for the same FQDN #1475

merged 5 commits into from
Apr 15, 2020

Commits on Mar 28, 2020

  1. Multiple A records support for the same FQDN 

    * Fixes kubernetes-sigs#1313
* Logic is to have unique key for every of multiple `ep.Targets` and keep
  them around in `ep.Labels` for further consistency
* Unit test is not very strict as testing functions are not ready for
  multiple Services with the same key but is imho good enough
    @ytsarev
    ytsarev committed Mar 28, 2020
    Configuration menu
    Copy the full SHA
    3602c47 View commit details
    Browse the repository at this point in the history

Commits on Mar 30, 2020

  1. Skip not A Target related labels 

    * Otherwise it would lead to unwanted deletion of etcd keys
* Inline func comment fix
    @ytsarev
    ytsarev committed Mar 30, 2020
    Configuration menu
    Copy the full SHA
    83ea480 View commit details
    Browse the repository at this point in the history

Commits on Apr 8, 2020

  1. Fix function comments

    @ytsarev
    ytsarev committed Apr 8, 2020
    Configuration menu
    Copy the full SHA
    4d0af61 View commit details
    Browse the repository at this point in the history

  2. Make findEp return Endpoint directly 

    * Return Endpoint instead of index in `findEp()`
* Simplify the logic within `Records()`
    @ytsarev
    ytsarev committed Apr 8, 2020
    Configuration menu
    Copy the full SHA
    c9f22b4 View commit details
    Browse the repository at this point in the history

Commits on Apr 13, 2020

  1. Fix comments and log messages

    @ytsarev
    ytsarev committed Apr 13, 2020
    Configuration menu
    Copy the full SHA
    d3b4126 View commit details
    Browse the repository at this point in the history