)]}'
{"tempest/common/skip_utils.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"595290fc615c7267a26a0215113372e04adc8891","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Copyright (c) 2016 Hewlett-Packard Development Company, L.P."},{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":4,"context_line":"#    not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a89bdaa_53937b4f","line":1,"updated":"2016-09-09 13:57:47.000000000","message":"I\u0027d put this into the tempest/common/utils dir instead.","commit_id":"42c118bfc6838e98cb7bb0a81d61d122788bbdb8"}],"tempest/test.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"595290fc615c7267a26a0215113372e04adc8891","unresolved":false,"context_lines":[{"line_number":154,"context_line":"    return decorator"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"def is_extension_enabled(extension_name, service):"},{"line_number":158,"context_line":"    \"\"\"A function that will check the list of enabled extensions from config"},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"    \"\"\""}],"source_content_type":"text/x-python","patch_set":3,"id":"9a89bdaa_131733ee","side":"PARENT","line":157,"updated":"2016-09-09 13:57:47.000000000","message":"According to http://codesearch.openstack.org/?q\u003dis_extension_enabled\u0026i\u003dnope\u0026files\u003d\u0026repos\u003d\n\nthis function is used by several tempest plugins. We should make a deprecation. You can have a look L870 (bottom of this file) to see how we can leverage debtcollector","commit_id":"d00166fb4632a4b49f775d8096ffd6dd1d6c860a"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"2bcb3405c1ad589cbf9c8184bb8ae5eda8eb7147","unresolved":false,"context_lines":[{"line_number":154,"context_line":"    return decorator"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"def is_extension_enabled(extension_name, service):"},{"line_number":158,"context_line":"    \"\"\"A function that will check the list of enabled extensions from config"},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"    \"\"\""}],"source_content_type":"text/x-python","patch_set":3,"id":"9a89bdaa_aac98a7b","side":"PARENT","line":157,"in_reply_to":"9a89bdaa_131733ee","updated":"2016-09-10 13:08:09.000000000","message":"New path also is not stable interface in Tempest, so we cannot say \"you need to use a new path of is_extensions_enabled\".\nIn addition this method contains CONF value, that means we cannot move the method into tempest.lib at this time.","commit_id":"d00166fb4632a4b49f775d8096ffd6dd1d6c860a"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"8b41376a96887524df2e0f643c5bfbe3b160f5dc","unresolved":false,"context_lines":[{"line_number":154,"context_line":"    return decorator"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"def is_extension_enabled(extension_name, service):"},{"line_number":158,"context_line":"    \"\"\"A function that will check the list of enabled extensions from config"},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"    \"\"\""}],"source_content_type":"text/x-python","patch_set":3,"id":"9a89bdaa_d572d37b","side":"PARENT","line":157,"in_reply_to":"9a89bdaa_aac98a7b","updated":"2016-09-12 09:16:02.000000000","message":"Ken\u0027ichi, Matthew moved the function in tempest/common, not tempest/lib so we don\u0027t really care about CONF or interface stability, yet.\n\nMy point was, let\u0027s try not to break many plugins, even if we never committed to interface stability outside tempest/lib.","commit_id":"d00166fb4632a4b49f775d8096ffd6dd1d6c860a"}]}