)]}'
{"gnocchi/cli.py":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"b07c412a2bcb130fcbcf6561251a54a942c1d45f","unresolved":false,"context_lines":[{"line_number":39,"context_line":"        cfg.BoolOpt(\"index-only\", default\u003dFalse,"},{"line_number":40,"context_line":"                    help\u003d\"Upgrade index only.\"),"},{"line_number":41,"context_line":"        cfg.BoolOpt(\"storage-only\", default\u003dFalse,"},{"line_number":42,"context_line":"                    help\u003d\"Upgrade storage only.\")"},{"line_number":43,"context_line":"    ])"},{"line_number":44,"context_line":"    conf \u003d service.prepare_service(conf\u003dconf)"},{"line_number":45,"context_line":"    no_option_on_cmd_line \u003d not conf.index_only and not conf.storage_only"}],"source_content_type":"text/x-python","patch_set":4,"id":"ba8a016a_cef642d7","line":42,"updated":"2015-11-16 18:44:57.000000000","message":"to be honest, it seems strange that we call it \u0027only\u0027 when you can set both. i have a slight preference we call it \u0027upgrade-index\u0027, default\u003dTrue and \u0027upgrade-storage\u0027","commit_id":"6cf485c4776cc48e607431463dd089d8fbd97595"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"b100161a88c504b58e98078dc27fe3d6c04cb014","unresolved":false,"context_lines":[{"line_number":39,"context_line":"        cfg.BoolOpt(\"index-only\", default\u003dFalse,"},{"line_number":40,"context_line":"                    help\u003d\"Upgrade index only.\"),"},{"line_number":41,"context_line":"        cfg.BoolOpt(\"storage-only\", default\u003dFalse,"},{"line_number":42,"context_line":"                    help\u003d\"Upgrade storage only.\")"},{"line_number":43,"context_line":"    ])"},{"line_number":44,"context_line":"    conf \u003d service.prepare_service(conf\u003dconf)"},{"line_number":45,"context_line":"    no_option_on_cmd_line \u003d not conf.index_only and not conf.storage_only"}],"source_content_type":"text/x-python","patch_set":4,"id":"ba8a016a_3f37ace4","line":42,"in_reply_to":"ba8a016a_c6cd89b8","updated":"2015-11-17 23:18:41.000000000","message":"So what\u0027s your proposal? :p","commit_id":"6cf485c4776cc48e607431463dd089d8fbd97595"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"dda503c1e148dc1d52d4a2a6c9906ab4334a4f30","unresolved":false,"context_lines":[{"line_number":39,"context_line":"        cfg.BoolOpt(\"index-only\", default\u003dFalse,"},{"line_number":40,"context_line":"                    help\u003d\"Upgrade index only.\"),"},{"line_number":41,"context_line":"        cfg.BoolOpt(\"storage-only\", default\u003dFalse,"},{"line_number":42,"context_line":"                    help\u003d\"Upgrade storage only.\")"},{"line_number":43,"context_line":"    ])"},{"line_number":44,"context_line":"    conf \u003d service.prepare_service(conf\u003dconf)"},{"line_number":45,"context_line":"    no_option_on_cmd_line \u003d not conf.index_only and not conf.storage_only"}],"source_content_type":"text/x-python","patch_set":4,"id":"ba8a016a_c6cd89b8","line":42,"in_reply_to":"ba8a016a_cc4babaf","updated":"2015-11-17 18:50:35.000000000","message":"it seems weird when the command is:\n\n  gnocchi-upgrade --index-only True --storage-only True\n\nit\u0027s the idea that we allow both option that seems weird.","commit_id":"6cf485c4776cc48e607431463dd089d8fbd97595"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"6ca33ef9eabc16a247315a30e9515bf8d341dde6","unresolved":false,"context_lines":[{"line_number":39,"context_line":"        cfg.BoolOpt(\"index-only\", default\u003dFalse,"},{"line_number":40,"context_line":"                    help\u003d\"Upgrade index only.\"),"},{"line_number":41,"context_line":"        cfg.BoolOpt(\"storage-only\", default\u003dFalse,"},{"line_number":42,"context_line":"                    help\u003d\"Upgrade storage only.\")"},{"line_number":43,"context_line":"    ])"},{"line_number":44,"context_line":"    conf \u003d service.prepare_service(conf\u003dconf)"},{"line_number":45,"context_line":"    no_option_on_cmd_line \u003d not conf.index_only and not conf.storage_only"}],"source_content_type":"text/x-python","patch_set":4,"id":"ba8a016a_cc4babaf","line":42,"in_reply_to":"ba8a016a_cef642d7","updated":"2015-11-16 19:17:38.000000000","message":"The idea is to upgrade both by default, and only one if you pass one of those flag. Does that sound weird?","commit_id":"6cf485c4776cc48e607431463dd089d8fbd97595"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"e129704673ea471cd67a79a4aa0c5fb03cb20822","unresolved":false,"context_lines":[{"line_number":36,"context_line":"def upgrade():"},{"line_number":37,"context_line":"    conf \u003d cfg.ConfigOpts()"},{"line_number":38,"context_line":"    conf.register_cli_opts(["},{"line_number":39,"context_line":"        cfg.BoolOpt(\"index-only\", default\u003dFalse,"},{"line_number":40,"context_line":"                    help\u003d\"Upgrade index only.\"),"},{"line_number":41,"context_line":"        cfg.BoolOpt(\"storage-only\", default\u003dFalse,"},{"line_number":42,"context_line":"                    help\u003d\"Upgrade storage only.\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba8a016a_221f5e6d","line":39,"range":{"start_line":39,"start_character":21,"end_line":39,"end_character":26},"updated":"2015-11-18 20:08:13.000000000","message":"maybe --skip-index and --skip-storage? or have one listopt?\n\nif you feel it\u0027s no different, let me know.","commit_id":"0f458ee74d92edc2778140c2003ad3c9b0264ac6"}]}