[naming] Rename "exceptions" module to "errors"

Bug #1232074 reported by Allele Dev on 2013-09-27
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
zaqar
Undecided
Cindy Pallares

Bug Description

Identified during proxy development, the plan is to move the naming of exception-related code to use the term error.

This is still open for discussion, though it makes sense, given that most of the errors in the Python standard exception hiearchy are name XError, and most of the conditions handled by marconi are errorneous rather than exceptional.

Flavio Percoco (flaper87) wrote :

I like the idea and the rational behind this bug.

Changed in marconi:
assignee: nobody → Cindy Pallares (cindy-pallaresq)

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

Changed in marconi:
status: New → In Progress

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

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

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

Reviewed: https://review.openstack.org/54141
Committed: http://github.com/openstack/marconi/commit/5624bf1a1459fb271c6c4dfa87d04eaf070978f6
Submitter: Jenkins
Branch: master

commit 5624bf1a1459fb271c6c4dfa87d04eaf070978f6
Author: cpallares <email address hidden>
Date: Mon Oct 28 10:49:27 2013 -0500

    Renamed "exceptions" module to "errors"

    Changes exceptions.py into error.py

    Renaming exception-related code to errors
    because most of the conditions handled by
    marconi are errorneous rather than exceptional.

    Closes-Bug:#1232074

    Change-Id: Ie3509236b07a372a44d97f97e5f3fe183b409606

Changed in marconi:
status: In Progress → Fix Committed
Thierry Carrez (ttx) on 2013-12-19
Changed in marconi:
milestone: none → icehouse-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2014-04-17
Changed in marconi:
milestone: icehouse-1 → 2014.1
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers