Bug 1389101 - PortalIP should be handled in a backwards compatible manner
Summary: PortalIP should be handled in a backwards compatible manner
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: openshift-controller-manager
Version: 3.4.0
Hardware: Unspecified
OS: Unspecified
low
low
Target Milestone: ---
: ---
Assignee: Michal Fojtik
QA Contact: zhou ying
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-10-26 20:52 UTC by Matt Wringe
Modified: 2016-10-27 14:08 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-10-27 14:08:03 UTC
Target Upstream Version:


Attachments (Terms of Use)

Description Matt Wringe 2016-10-26 20:52:26 UTC
In a service definition, the 'portalIP' value has been renamed to 'clusterIP'. 

The problem is that existing configuration files which use this 'portalIP' value will fail when updating to OpenShift 3.4. This is a silent failure where the configuration file will be accepted, it will just not have its the value applied properly.

This means existing components which rely on the behaviour of 'portalIP' will start to fail.

We should continue to handle 'portalIP' values in a backwards compatible manner.

This will fail to properly set the portalIP to 'None':

- apiVersion: v1
  kind: Service
  metadata:
    labels:
      metrics-infra: hawkular-cassandra
      name: hawkular-cassandra
    name: hawkular-cassandra-nodes
  spec:
    portalIP: None
    ports:
    - name: cql-port
      port: 9042
    - name: thift-port
      port: 9160
    - name: tcp-port
      port: 7000
    - name: ssl-port
      port: 7001
    selector:
      type: hawkular-cassandra

While this will properly set the clusterIP to 'None':
- apiVersion: v1
  kind: Service
  metadata:
    labels:
      metrics-infra: hawkular-cassandra
      name: hawkular-cassandra
    name: hawkular-cassandra-nodes
  spec:
    clusterIP: None
    ports:
    - name: cql-port
      port: 9042
    - name: thift-port
      port: 9160
    - name: tcp-port
      port: 7000
    - name: ssl-port
      port: 7001
    selector:
      type: hawkular-cassandra

Comment 1 Clayton Coleman 2016-10-27 14:07:17 UTC
This has been deprecated since 1.0 and we are dropping support for it.


Note You need to log in before you can comment on or make changes to this bug.