Unit tests should not perform logging,but some tests still use

[jingtao liang]

We shuld remove the logging

[Amrith Kumar (amrith)]

Without more details about this, it is hard to tell what this bug is about. How can code not perform logging just because it is in unit tests? Heck, some tests actually verify that messages are being logged.

[Brant Knudson (blk-u)]

What problem is this causing?

[Dolph Mathews (dolph)]

> Unit tests should not perform logging

Why not? Debugging is useful.

[Amrith Kumar (amrith)]

marking incomplete for Trove per IRC chat with keystone folks.

[Dolph Mathews (dolph)]

Without any presenting any reasoning or justification, this "bug" seems opinionated, wasteful, and counter-productive.

[Ian Cordasco (icordasc)]

I have to agree with Dolph. I'm inclined to mark it as "won't fix" for glance.

[jingtao liang (liang-jingtao)]

@Dolph Mathews (dolph) Ian Cordasco (icordasc)

Hello, i'm sorry to give the reason here so later.But in the cinder HACKING.rst,you can see the Cinder Specific Commandments [C309].You will see :Unit tests should not perform logging.

[David Stanek (dstanek)]

Marking as WONTFIX for now because I don't think this is a keystone issue at all. If you find a place in keystone where there is a problem feel free to reopen.

[Markus Zoeller (markus_z) (mzoeller)]

Nova has no rules in place which forbids that, so it's not a bug. I also don't see a reason to put effort into this.

[Amrith Kumar (amrith)]

This is not a bug (in Trove).

[Launchpad Janitor (janitor)]

[Expired for rack because there has been no activity for 60 days.]

[Launchpad Janitor (janitor)]

[Expired for python-rackclient because there has been no activity for 60 days.]

[Cyril Roelandt (cyril-roelandt)]

This does not seem relevant to Glance.