Bug 1389101

Summary: PortalIP should be handled in a backwards compatible manner
Product: OpenShift Container Platform Reporter: Matt Wringe <mwringe>
Component: openshift-controller-managerAssignee: Michal Fojtik <mfojtik>
Status: CLOSED NOTABUG QA Contact: zhou ying <yinzhou>
Severity: low Docs Contact:
Priority: low    
Version: 3.4.0CC: aos-bugs, ccoleman
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-10-27 14:08:03 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

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.