Skip to content

Service port validation #974

Closed
Closed

Description

@bgrant0607

AFAICT, ValidateService doesn't check that a port is specified. What happens when no port is specified?

I also can't find anywhere where we check that 2 services don't ask for the same port.

Additionally, host ports may conflict with service ports. It would be useful to report such conflicts and to prevent the creation of services that conflict with host ports and vice versa.

Activity

  1. added
    priority/important-soonMust be staffed and worked on either currently, or very soon, ideally in time for the next release.
    and removed
    priority/backlogHigher priority than priority/awaiting-more-evidence.
    on Sep 8, 2014
  2. added this to the milestone on Sep 9, 2014
  3. brendandburns commented on Sep 24, 2014

    @brendandburns
    Contributor

    oops...

  4. brendandburns commented on Sep 25, 2014

    @brendandburns
    Contributor

    This issue is largely deprecated. Will close with IP per service lands.

  5. thockin commented on Sep 30, 2014

    @thockin
    Member

    IP-per-service does not eliminate this.

    Even after ip-per-service, some Services will have to use HostPorts (for External Load Balancer, for now). Much reduced, but that just makes a conflict even less common and harder to debug.

    It boils down to: both Pods and Services can use HostPorts. We currently check Pods against other Pods for host-port conflicts, but we do not check Pods against Services, nor vice-versa, nor Services against Services.

    We need some common way for both resource classes to check for conflicts and reserve HostPorts atomically.

  6. brendandburns commented on Oct 21, 2014

    @brendandburns
    Contributor

    Closing this as it is now obsolete with service v2.

  7. craigmcl commented on Oct 28, 2014

    @craigmcl

    This is to be addressed in v0.5.

  8. brendandburns commented on Dec 16, 2014

    @brendandburns
    Contributor

    Closing this, as it is no longer possible for services and pods to have port conflicts, unless a user does something manual (like specify a host IP for the "PublicIP" for a service)

  9. added a commit that references this issue on Apr 6, 2016
  10. added a commit that references this issue on Oct 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    priority/important-soonMust be staffed and worked on either currently, or very soon, ideally in time for the next release.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions