Policy check is missing for FWaaS rules table

Bug #1494216 reported by Tatiana Ovchinnikova
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
High
Tatiana Ovchinnikova

Bug Description

There is firewall policy column in FWaaS rules table:

firewall_policy_id = tables.Column(get_policy_name,
                                       link=get_policy_link,
                                       verbose_name=_("In Policy"))

Method "get_policy_name" checks if there is a policy to display, but "get_policy_link" doesn't, so when we try to add a new firewall rule (which has no policy connection by default), horizon can't display FWaaS tables because of "'NoneType' object has no attribute 'id'".

Changed in horizon:
assignee: nobody → Tatiana Ovchinnikova (tmazur)
importance: Undecided → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (master)

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

Changed in horizon:
status: New → In Progress
description: updated
summary: - Policy check is missing for FWaaS tables
+ Policy check is missing for FWaaS table
summary: - Policy check is missing for FWaaS table
+ Policy check is missing for FWaaS rules table
David Lyle (david-lyle)
Changed in horizon:
milestone: none → liberty-rc1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/222102
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=9022617d9a3faa311842d34e708d42abca8e6a1b
Submitter: Jenkins
Branch: master

commit 9022617d9a3faa311842d34e708d42abca8e6a1b
Author: Tatiana Ovchinnikova <email address hidden>
Date: Thu Sep 10 12:52:55 2015 +0300

    Fix FWaaS Rules table displaying

    When we try to add a new firewall rule, it has no policy connection
    by default: we should use "Insert Rule" from Policies table. However,
    horizon can't display FWaaS tables anymore, breaking with error:
    "'NoneType' object has no attribute 'id'". This patch adds a policy
    check to "get_policy_link" method to show firewall rules without
    policy connections.

    Closes-Bug: #1494216

    Change-Id: Iab283181428ede84cbc020ebf52a171714f56fa1

Changed in horizon:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in horizon:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in horizon:
milestone: liberty-rc1 → 8.0.0
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (stable/kilo)

Fix proposed to branch: stable/kilo
Review: https://review.openstack.org/258290

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on horizon (stable/kilo)

Change abandoned by Dave Walker (<email address hidden>) on branch: stable/kilo
Review: https://review.openstack.org/258290
Reason: Kilo is now approaching EOL. We are in freeze pending the final release and no freeze exception has been raised. Therefore I am abandoning this change, if it is required in the release - please restore and raise a request. Thanks

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

Duplicates of this bug

Other bug subscribers

Remote bug watches

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