)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"254cf3a745ec01df0cc1fbb3a46ca235796b8be3","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This patch introduces a new config option \u0027subflows\u0027 under a new"},{"line_number":14,"context_line":"group \u0027import_task\u0027."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I9719616518ad2b665535926681bd73213fde9e33"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ba15a1d1_181bf381","line":15,"updated":"2015-09-22 08:28:18.000000000","message":"Please add DocImpact flag","commit_id":"2ebe20b99c4e969c253233849454d30d40fbe19b"}],"glance/async/flows/base_import.py":[{"author":{"_account_id":7549,"name":"Ekaterina Chernova","email":"katyafervent@yandex.ru","username":"efedorova"},"change_message_id":"ae72d48a27fb7069f4288ecaa82f39a6feb68d72","unresolved":false,"context_lines":[{"line_number":45,"context_line":"import_opts \u003d ["},{"line_number":46,"context_line":"    cfg.ListOpt(\u0027subflows\u0027,"},{"line_number":47,"context_line":"                default\u003d[\u0027convert\u0027, \u0027introspect\u0027],"},{"line_number":48,"context_line":"                help\u003d_(\u0027Name of the import subflows that should be loaded.\u0027))"},{"line_number":49,"context_line":"]"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"CONF \u003d cfg.CONF"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a1a9d01_3a463dc0","line":48,"updated":"2015-09-24 14:08:31.000000000","message":"Name -\u003e list of names\n\nBTW are you sure that you need to declare config variable here instead of glance/common/config.py, where all config options are defined?","commit_id":"160cfc21d16670bd7bfad20b630c13e2d29f232c"},{"author":{"_account_id":7575,"name":"Sabari","email":"smurugesan@vmware.com","username":"sabari"},"change_message_id":"f8177e42da772da934d8fe0c7ec0c159fbe13c85","unresolved":false,"context_lines":[{"line_number":45,"context_line":"import_opts \u003d ["},{"line_number":46,"context_line":"    cfg.ListOpt(\u0027subflows\u0027,"},{"line_number":47,"context_line":"                default\u003d[\u0027convert\u0027, \u0027introspect\u0027],"},{"line_number":48,"context_line":"                help\u003d_(\u0027Name of the import subflows that should be loaded.\u0027))"},{"line_number":49,"context_line":"]"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"CONF \u003d cfg.CONF"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a1a9d01_f83f5bd4","line":48,"in_reply_to":"9a1a9d01_3a463dc0","updated":"2015-09-24 21:23:57.000000000","message":"opts are usually defined where they are relevant. Also, import is considered an extension for flows. Hence, this is appropriate place for it.\n\nThanks Ekaterina, I will fix the doc string.","commit_id":"160cfc21d16670bd7bfad20b630c13e2d29f232c"}]}