load-balancer_listener annotations value is incorrectly encoded as integer

Bug #1662333 reported by Ananth Suryanarayana
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
Trunk
Fix Committed
Medium
Sachin Bansal

Bug Description

Note value is 443, a number. But in schema, it is defined as a string
controller/src/schema/vnc_cfg.xsd

  2656
  2657 <xsd:complexType name="KeyValuePair">
  2658 <xsd:all>
  2659 <xsd:element name="key" type="xsd:string"/>
  2660 <xsd:element name="value" type="xsd:string"/>
  2661 </xsd:all>
  2662 </xsd:complexType>
  2663

0x7fffc4005de8: "{\"loadbalancer_listener\":{\"fq_name\":[\"default-domain\", \"default\", \"3f89b632-ecab-11e6-bd7d-027175576f13-kubernetes-TCP-443\"],\"parent_type\":\"project\",\"display_name\":\"3f89b632-ecab-11e6-bd7d-027175576f1"...
0x7fffc4005eb0: "3-kubernetes-TCP-443\",\"id_perms\":{\"enable\": true, \"uuid\": {\"uuid_mslong\": 4578390826034074086, \"uuid_lslong\": 13654072331580436243}, \"created\": \"2017-02-06T20:31:33.430311\", \"description\": null, \"crea"...
0x7fffc4005f78: "tor\": null, \"user_visible\": true, \"last_modified\": \"2017-02-06T20:31:33.452224\", \"permissions\": {\"owner\": \"cloud-admin\", \"owner_access\": 7, \"other_access\": 7, \"group\": \"cloud-admin-group\", \"group_acce"...
0x7fffc4006040: "ss\": 7}},\"loadbalancer_listener_properties\":{\"default_tls_container\": null, \"protocol\": \"TCP\", \"connection_limit\": null, \"admin_state\": true, \"sni_containers\": [], \"protocol_port\": 443},\"perms2\":{\"own"...
0x7fffc4006108: "er\": \"cloud-admin\", \"owner_access\": 7, \"global_access\": 0, \"share\": []},\"loadbalancer_refs\":[{\"to\":\"default-domain:default:kubernetes\", \"uuid\":\"38351eee-ecab-11e6-817e-027175576f13\"}],\"annotations\":{\""...
0x7fffc40061d0: "key_value_pair\":[{\"key\": \"targetPort\", \"value\": 443}] }}}"

Tags: config
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/28492
Submitter: Sachin Bansal (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/28520
Submitter: Yuvaraja Mariappan

Nischal Sheth (nsheth)
tags: removed: contrail-control
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/28492
Submitter: Sachin Bansal (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/28492
Committed: http://github.org/Juniper/contrail-controller/commit/e8e5bb649a0d9e408a8392619d46d5f62353afd8
Submitter: Zuul (<email address hidden>)
Branch: master

commit e8e5bb649a0d9e408a8392619d46d5f62353afd8
Author: Sachin Bansal <email address hidden>
Date: Mon Feb 6 18:05:24 2017 -0800

Add type validation in prop-collection-post method

Without this, user could set invalid values for prop-map or
prop-list type fields. Also fixed the corresponding routine
in api lib.

Change-Id: I18c4ef9377e9b594e12622a1495672f8c9aaef4e
Closes-Bug: 1662333

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.