Glance Code Cleanup Round 1

Bug #713461 reported by Rick Harris
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Glance
Fix Released
Low
Rick Harris

Bug Description

We have a few blemishes in the code-base that would be worth excising:

1. glance/store/backends is empty, we can ax this

2. glance/common/db is empty, we can ax this

3. Django-community gripe caught my attention:

Rick-Harriss-MacBook-Pro:trunk rick$ find . -name util*.py
./glance/common/utils.py
./glance/util.py
./tests/utils.py

I prefer utils, so I vote we rename glance/util -> glance/utils.py

4. ./run_tests -N is now producing tons of log messages even when it succeeds. We should probably silence that.

5. README is a little out of date, we can probably put something a little more useful in there.

6. tests/functional is empty. I vote we remove this directory and add back once we have a functional test. The counter-arg here is that this will gnaw at us so that we actually write a functional test. I don't buy it :)

Related branches

Changed in glance:
assignee: nobody → Rick Harris (rconradharris)
importance: Undecided → Low
Changed in glance:
status: New → Fix Committed
Thierry Carrez (ttx)
Changed in glance:
milestone: none → 2011.2
status: Fix Committed → 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.