l3_proactive_app naming convention is unclear

Bug #1653926 reported by Omer Anson
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
DragonFlow
Fix Released
High
Omer Anson

Bug Description

The names of methods in the L3 proactive application are unclear. They do not explain what the method does, but more like in what context it should be used.

This bug exists so that this will be corrected. There are two action items:
* Rename each method to state its function
* Add pydoc for each method.

Omer Anson (omer-anson)
Changed in dragonflow:
assignee: nobody → Omer Anson (omer-anson)
Omer Anson (omer-anson)
Changed in dragonflow:
importance: Medium → High
Omer Anson (omer-anson)
Changed in dragonflow:
assignee: Omer Anson (omer-anson) → nobody
Omer Anson (omer-anson)
Changed in dragonflow:
assignee: nobody → Omer Anson (omer-anson)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to dragonflow (master)

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

Changed in dragonflow:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to dragonflow (master)

Reviewed: https://review.openstack.org/504047
Committed: https://git.openstack.org/cgit/openstack/dragonflow/commit/?id=3d5bc204307fc52dead5d58a7981548af8062318
Submitter: Jenkins
Branch: master

commit 3d5bc204307fc52dead5d58a7981548af8062318
Author: Omer Anson <email address hidden>
Date: Thu Sep 14 15:33:55 2017 +0300

    Add docstrings to L3 apps

    Also rename some of the functions.

    Change-Id: I5b652d59d1ffb126226fdc846e9ee7d6bcf3aafa
    Closes-Bug: #1653926

Changed in dragonflow:
status: In Progress → Fix Released
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.