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

drop openshift-sdn ports from host-port-registry #1664

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

drop openshift-sdn ports from host-port-registry #1664

wants to merge 5 commits into from

Commits on Aug 16, 2024

  1. Remove some ancient deprecated SDN ports 

    A long time ago there was some confusion about openshift-sdn ports vs
ovn-kubernetes ports vs standalone kube-proxy ports, but that's been
fixed for a long time and continuing to reference it here confuses
things more than it clarifies them. (Also, we still had some of the
details wrong...)
    @danwinship
    danwinship committed Aug 16, 2024
    Configuration menu
    Copy the full SHA
    a6693c6 View commit details
    Browse the repository at this point in the history

  2. Fix "previously allocated" table 

    Whoever added this thought it would be clever to not bother aligning
the columns, and then promptly proceeded to make it clear why the
columns needed to be aligned by accidentally making the body have
more columns than the headings.

Remove the "Protocol" column since it's not really useful for ports
that aren't actually still used, and change "Since" to "Added" and
"Removed".
    @danwinship
    danwinship committed Aug 16, 2024
    Configuration menu
    Copy the full SHA
    239d700 View commit details
    Browse the repository at this point in the history

  3. Remove openshift-sdn ports

    @danwinship
    danwinship committed Aug 16, 2024
    Configuration menu
    Copy the full SHA
    6dbd80a View commit details
    Browse the repository at this point in the history

  4. Remove "ovn-kubernetes only" notes 

    (It's true that these ports aren't used in clusters with third-party
network plugins, but you can figure that out on your own...)
    @danwinship
    danwinship committed Aug 16, 2024
    Configuration menu
    Copy the full SHA
    dd023a0 View commit details
    Browse the repository at this point in the history

  5. Clarify some sdn-related ports

    @danwinship
    danwinship committed Aug 16, 2024
    Configuration menu
    Copy the full SHA
    a0aa2f1 View commit details
    Browse the repository at this point in the history