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.

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

Multiple auto-egress-IP fixes #18808

Merged
merged 5 commits into from
Mar 8, 2018

Commits on Mar 2, 2018

  1. Add assertOVSChanges helper to egressip_test

    This makes the tests a little bit more self-documenting, and makes it
    simpler to add new tests. Also, change it to only look at OVS flows in
    the egress table, so that when there is a mismatch, the error message
    is of a manageable length.
    danwinship committed Mar 2, 2018
    Configuration menu
    Copy the full SHA
    6758ab7 View commit details
    Browse the repository at this point in the history
  2. egressip_test updates

    Split out one test case, add some new (already-passing) test cases,
    add some comments, and remove some double-checking-of-internal-state
    that will break with upcoming rewrites.
    danwinship committed Mar 2, 2018
    Configuration menu
    Copy the full SHA
    bbadc22 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    c854216 View commit details
    Browse the repository at this point in the history
  4. Fix egressip handling when a NetNamespace is updated

    (We handled delete+recreate correctly, but not update.)
    danwinship committed Mar 2, 2018
    Configuration menu
    Copy the full SHA
    b10a138 View commit details
    Browse the repository at this point in the history

Commits on Mar 5, 2018

  1. Rearrange egressip internals, add duplication tests

    There should never be multiple HostSubnets or multiple NetNamespaces
    claiming the same egress IP, but if there are, we need to track them
    carefully so we don't get out sync with reality after things are
    fixed.
    danwinship committed Mar 5, 2018
    Configuration menu
    Copy the full SHA
    a21509b View commit details
    Browse the repository at this point in the history