Fuel for OpenStack

Refactor url_access_checker/api.py to use idiomatic python instead of list of tuples.

Bug #1488946 reported by Maciej Kwiek
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
Low
Fuel Python (Deprecated)
Fuel for OpenStack 9.0

Bug Description

check_urls function from fuel-web/network_checker/url_access_checker/api.py uses list of tuples. It could use dictionary, the code would be nicer then.

Also, whole api module should be refactored into a class.

Sylwester Brzeczkowski (sbrzeczkowski)
Changed in fuel:
status: New → Confirmed
Łukasz Oleś (loles)
Changed in fuel:
milestone: none → 8.0
Dmitry Pyzhov (dpyzhov)
tags: added: tech-debt
Dmitry Pyzhov (dpyzhov)
tags: added: area-python
Revision history for this message
Dmitry Pyzhov (dpyzhov) wrote :

We passed SCF in 8.0. Moving the bug to 9.0.

Changed in fuel:
milestone: 8.0 → 9.0
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to network-checker (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/278071

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to network-checker (master)

Reviewed: https://review.openstack.org/278071
Committed: https://git.openstack.org/cgit/openstack/network-checker/commit/?id=04760b3557104f8adf754354364f0b8261efb3fd
Submitter: Jenkins
Branch: master

commit 04760b3557104f8adf754354364f0b8261efb3fd
Author: tivaliy <email address hidden>
Date: Tue Feb 9 22:13:28 2016 +0200

    Use dictionary instead of list of tuples in url access checker

    Refactor url_access_checker/api.py to use idiomatic python
    instead of list of tuple

    Change-Id: Ia2a6a5f301b51b88eda8a4086f7a769230a6c055
    Related-Bug: #1488946

Sergey Kulanov (skulanov)
Changed in fuel:
status: Confirmed → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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