cut_list() needs to work the same as cut_dict()

Bug #1761246 reported by Bob Haddleton
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mistral
Fix Released
Low
Bob Haddleton

Bug Description

The utils.cut_list() method returns a string representation of the provided list, truncated to some number of characters that may or may not be close to the length parameter, depending on the structure and contents of the list provided.

This can lead to hung workflows when the Execution.state_info field is not truncated properly due to the context variables involved

The cut_list() method needs to return a string no longer than what the length parameter specifies to ensure that the state_info field is truncated properly and does not cause hung workflows.

And yes this should have been fixed when I fixed cut_dict()

Changed in mistral:
status: New → In Progress
assignee: nobody → Bob Haddleton (bob-haddleton)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to mistral (master)

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

Dougal Matthews (d0ugal)
Changed in mistral:
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to mistral (master)

Reviewed: https://review.openstack.org/558902
Committed: https://git.openstack.org/cgit/openstack/mistral/commit/?id=d456ba271612f945e2665e6dd7213a6233018583
Submitter: Zuul
Branch: master

commit d456ba271612f945e2665e6dd7213a6233018583
Author: Bob Haddleton <email address hidden>
Date: Wed Apr 4 14:18:57 2018 -0500

    Update cut_list() to return the specified number of characters

    This patch updates cut_list() to ensure that it does not return
    more than the number of characters specified in the limit
    parameter.

    Change-Id: I9dbd061a5ba1976aae9d1acf1e501a3c98782972
    Closes-Bug: 1761246

Changed in mistral:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/mistral 7.0.0.0b1

This issue was fixed in the openstack/mistral 7.0.0.0b1 development milestone.

Dougal Matthews (d0ugal)
Changed in mistral:
milestone: none → rocky-1
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.