NaElement translate_struct does not work for non unique tags

Bug #1287643 reported by Julia Portnova on 2014-03-04
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Undecided
Navneet

Bug Description

Method translate_struct of NaElement does not work properly in case of unique tags.

Navneet (singn) on 2014-03-04
Changed in cinder:
assignee: nobody → Navneet (singn)
Navneet (singn) wrote :

The bug relates to properly translating data structures such as dict, list and tuple into NaElement (api) acceptable elements. It may not be related to this method only. I will scan for proper structure and type conversions in the NaElement class.

Changed in cinder:
status: New → Confirmed

Fix proposed to branch: master
Review: https://review.openstack.org/77888

Changed in cinder:
status: Confirmed → In Progress
summary: - netapp-api-trunslate-struct-non-unique
+ netapp-api-translate-struct-non-unique
description: updated
summary: - netapp-api-translate-struct-non-unique
+ NaElement translate_struct does not work for non unique tags

Reviewed: https://review.openstack.org/77888
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=e8fac5eda3309a19ee2e1b36883a4e30476644a9
Submitter: Jenkins
Branch: master

commit e8fac5eda3309a19ee2e1b36883a4e30476644a9
Author: Navneet Singh <email address hidden>
Date: Fri Feb 14 21:21:33 2014 +0530

    NetApp api fix structure conversion methods

    This fixes the issue in api element creation
    when data structures are supplied to be converted
    into NaElements. It could not handle multiple
    keys with same name and also not converted different
    types properly into NaElement. This change fixes the issue.

    Change-Id: I84f1ee97a73bbad88bbb64d4a9ac4e1b29cad741
    Closes-bug: #1287643

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx) on 2014-03-27
Changed in cinder:
milestone: none → icehouse-rc1
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2014-04-17
Changed in cinder:
milestone: icehouse-rc1 → 2014.1
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers