Project tags treats entire collection as a single tag

Bug #1752301 reported by Gage Hugo
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
High
Gage Hugo

Bug Description

When backporting Debian stable/queens from release Sid to Stretch, an issue where 3 unit tests were failing due to the entire project tags collection being treated as a single tag was encountered[0].

It was later determined that the hybrid_property.expression implementation was causing this issue[1]. When a quick change was pushed up and tested, the issue appeared to be fixed[2].

The fix for this issue is to drop @hybrid_property usage for @property, which removes the use of tags.expression. Project should always have tags instantiated, so there is not a behavior difference, which is a better fit for @property.

[0] http://eavesdrop.openstack.org/irclogs/%23openstack-keystone/%23openstack-keystone.2018-02-27.log.html#t2018-02-27T11:23:30

[1] http://eavesdrop.openstack.org/irclogs/%23openstack-keystone/%23openstack-keystone.2018-02-27.log.html#t2018-02-27T19:32:42

[2] http://eavesdrop.openstack.org/irclogs/%23openstack-keystone/%23openstack-keystone.2018-02-27.log.html#t2018-02-27T21:24:34

Changed in keystone:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to keystone (stable/queens)

Fix proposed to branch: stable/queens
Review: https://review.openstack.org/548949

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to keystone (master)

Reviewed: https://review.openstack.org/548399
Committed: https://git.openstack.org/cgit/openstack/keystone/commit/?id=94ef9458858506766149761b8f2a9961d6c9def6
Submitter: Zuul
Branch: master

commit 94ef9458858506766149761b8f2a9961d6c9def6
Author: Gage Hugo <email address hidden>
Date: Tue Feb 27 19:55:22 2018 +0000

    Remove @expression from tags

    This change makes tags a property of Project instead of a hybrid_property
    since we will always have a Project contain some list of tags.

    Change-Id: I1033321132cb3ec71bf94b8293cef91dfc6b8272
    Co-Authored-By: Morgan Fainberg <email address hidden>
    Closes-Bug: #1752301

Changed in keystone:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to keystone (stable/queens)

Reviewed: https://review.openstack.org/548949
Committed: https://git.openstack.org/cgit/openstack/keystone/commit/?id=b98748cbfdbcc8918dfb5cbcc0a4d35a99906fdf
Submitter: Zuul
Branch: stable/queens

commit b98748cbfdbcc8918dfb5cbcc0a4d35a99906fdf
Author: Gage Hugo <email address hidden>
Date: Tue Feb 27 19:55:22 2018 +0000

    Remove @expression from tags

    This change makes tags a property of Project instead of a hybrid_property
    since we will always have a Project contain some list of tags.

    Change-Id: I1033321132cb3ec71bf94b8293cef91dfc6b8272
    Co-Authored-By: Morgan Fainberg <email address hidden>
    Closes-Bug: #1752301

tags: added: in-stable-queens
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/keystone 14.0.0.0b1

This issue was fixed in the openstack/keystone 14.0.0.0b1 development milestone.

Changed in keystone:
milestone: none → rocky-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/keystone 13.0.1

This issue was fixed in the openstack/keystone 13.0.1 release.

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.