)]}'
{"tempest/cli/simple_read_only/test_trove.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"6817971ebd4914633e6f56b8f5adf6e48c52f93c","unresolved":false,"context_lines":[{"line_number":15,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":16,"context_line":"#    under the License."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"import datetime"},{"line_number":19,"context_line":"from oslo.config import cfg"},{"line_number":20,"context_line":"import subprocess"},{"line_number":21,"context_line":"import tempest.cli"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F9Uhw%3D","line":18,"updated":"2013-12-12 17:23:22.000000000","message":"Need to order the imports correctly, i.e. std lib packages first, then a group of 3rd party packages, e.g. oslo, and then the local packages, i.e. tempest.\n\nhttp://docs.openstack.org/developer/hacking/#imports","commit_id":"9d36e491ba996dd49bd13cd9143268b7bdf86388"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"dd123f2b4c8a792a56e110a5602e7a2d652e24e3","unresolved":false,"context_lines":[{"line_number":21,"context_line":"import tempest.cli"},{"line_number":22,"context_line":"from tempest.openstack.common import log as logging"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"CONF \u003d cfg.CONF"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"LOG \u003d logging.getLogger(__name__)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F9Tag%3D","line":24,"updated":"2013-12-12 20:54:48.000000000","message":"We can\u0027t use the oslo config global namespace in tempest. It causes issues because of how test runners import and run code. To read config variables the base TestClass setups a variable config. Use that instead, also it looks like this isn\u0027t used.","commit_id":"9d36e491ba996dd49bd13cd9143268b7bdf86388"}]}