)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15168,"name":"Kirill Zaitsev","email":"k.zaitsev@me.com","username":"kzaitsev"},"change_message_id":"41e40e558b476bd45944816528577ed3c5eb804c","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Valerii Kovalchuk \u003cvakovalchuk@mirantis.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-05-19 19:12:00 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add \u0027--exists-action-deps\u0027 argument to murano CLI"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"It allows to apply different default actions for existing main"},{"line_number":10,"context_line":"package and existing dependencies. In case of specifying just"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"bab6814e_191c6111","line":7,"range":{"start_line":7,"start_character":5,"end_line":7,"end_character":25},"updated":"2016-05-19 17:27:35.000000000","message":"--dep-exists-action","commit_id":"387325ef6f77587ad61fcd0f3397a391f1574d4a"}],"muranoclient/v1/shell.py":[{"author":{"_account_id":15168,"name":"Kirill Zaitsev","email":"k.zaitsev@me.com","username":"kzaitsev"},"change_message_id":"ebd491eb69e1853d90d1e7b507eea45781712bc0","unresolved":false,"context_lines":[{"line_number":627,"context_line":"                \u0027(ignored when importing with multiple packages).\u0027)"},{"line_number":628,"context_line":"@utils.arg(\u0027--exists-action\u0027, default\u003d\u0027\u0027, choices\u003d[\u0027a\u0027, \u0027s\u0027, \u0027u\u0027],"},{"line_number":629,"context_line":"           help\u003d\u0027Default action when a package already exists.\u0027)"},{"line_number":630,"context_line":"@utils.arg(\u0027--exists-action-deps\u0027, default\u003d\u0027\u0027, choices\u003d[\u0027a\u0027, \u0027s\u0027, \u0027u\u0027],"},{"line_number":631,"context_line":"           help\u003d\u0027Default action when a dependency package already exists.\u0027)"},{"line_number":632,"context_line":"def do_package_import(mc, args):"},{"line_number":633,"context_line":"    \"\"\"Import a package."}],"source_content_type":"text/x-python","patch_set":1,"id":"bab6814e_5bbc6b6c","line":630,"range":{"start_line":630,"start_character":14,"end_line":630,"end_character":32},"updated":"2016-05-18 14:36:24.000000000","message":"--deps-exist-action is a more sematicaly correct name","commit_id":"7b852cc53347e69f4f7451d7b17b63d2ad3cd651"},{"author":{"_account_id":20773,"name":"Valerii Kovalchuk","email":"kovvalole@gmail.com","username":"vakovalchuk"},"change_message_id":"5a982e075579c721b0265a473fde24c7b51091b3","unresolved":false,"context_lines":[{"line_number":627,"context_line":"                \u0027(ignored when importing with multiple packages).\u0027)"},{"line_number":628,"context_line":"@utils.arg(\u0027--exists-action\u0027, default\u003d\u0027\u0027, choices\u003d[\u0027a\u0027, \u0027s\u0027, \u0027u\u0027],"},{"line_number":629,"context_line":"           help\u003d\u0027Default action when a package already exists.\u0027)"},{"line_number":630,"context_line":"@utils.arg(\u0027--exists-action-deps\u0027, default\u003d\u0027\u0027, choices\u003d[\u0027a\u0027, \u0027s\u0027, \u0027u\u0027],"},{"line_number":631,"context_line":"           help\u003d\u0027Default action when a dependency package already exists.\u0027)"},{"line_number":632,"context_line":"def do_package_import(mc, args):"},{"line_number":633,"context_line":"    \"\"\"Import a package."}],"source_content_type":"text/x-python","patch_set":1,"id":"bab6814e_e13ad0cc","line":630,"range":{"start_line":630,"start_character":14,"end_line":630,"end_character":32},"in_reply_to":"bab6814e_5bbc6b6c","updated":"2016-05-19 15:23:18.000000000","message":"Ok. I thought of it as \"exists action for dependencies\"","commit_id":"7b852cc53347e69f4f7451d7b17b63d2ad3cd651"},{"author":{"_account_id":15168,"name":"Kirill Zaitsev","email":"k.zaitsev@me.com","username":"kzaitsev"},"change_message_id":"02ec2f0e37b9b827408c3426a18099e6adb869ac","unresolved":false,"context_lines":[{"line_number":627,"context_line":"                \u0027(ignored when importing with multiple packages).\u0027)"},{"line_number":628,"context_line":"@utils.arg(\u0027--exists-action\u0027, default\u003d\u0027\u0027, choices\u003d[\u0027a\u0027, \u0027s\u0027, \u0027u\u0027],"},{"line_number":629,"context_line":"           help\u003d\u0027Default action when a package already exists.\u0027)"},{"line_number":630,"context_line":"@utils.arg(\u0027--exists-action-deps\u0027, default\u003d\u0027\u0027, choices\u003d[\u0027a\u0027, \u0027s\u0027, \u0027u\u0027],"},{"line_number":631,"context_line":"           help\u003d\u0027Default action when a dependency package already exists.\u0027)"},{"line_number":632,"context_line":"def do_package_import(mc, args):"},{"line_number":633,"context_line":"    \"\"\"Import a package."}],"source_content_type":"text/x-python","patch_set":1,"id":"bab6814e_4163dcb1","line":630,"range":{"start_line":630,"start_character":14,"end_line":630,"end_character":32},"in_reply_to":"bab6814e_5bbc6b6c","updated":"2016-05-18 14:49:31.000000000","message":"or maybe --dep-exists-action. Sounds better to me.","commit_id":"7b852cc53347e69f4f7451d7b17b63d2ad3cd651"},{"author":{"_account_id":15168,"name":"Kirill Zaitsev","email":"k.zaitsev@me.com","username":"kzaitsev"},"change_message_id":"ebd491eb69e1853d90d1e7b507eea45781712bc0","unresolved":false,"context_lines":[{"line_number":672,"context_line":""},{"line_number":673,"context_line":"    imported_list \u003d []"},{"line_number":674,"context_line":""},{"line_number":675,"context_line":"    ex_action_deps \u003d args.exists_action_deps"},{"line_number":676,"context_line":"    if ex_action_deps \u003d\u003d \u0027\u0027:"},{"line_number":677,"context_line":"        ex_action_deps \u003d args.exists_action"},{"line_number":678,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"bab6814e_669b96ca","line":675,"range":{"start_line":675,"start_character":4,"end_line":675,"end_character":6},"updated":"2016-05-18 14:36:24.000000000","message":"there is no reason to not write out the full name here. ex reads like \u0027previous\u0027, so I\u0027d rather ask you to update it to exists.","commit_id":"7b852cc53347e69f4f7451d7b17b63d2ad3cd651"},{"author":{"_account_id":20773,"name":"Valerii Kovalchuk","email":"kovvalole@gmail.com","username":"vakovalchuk"},"change_message_id":"5a982e075579c721b0265a473fde24c7b51091b3","unresolved":false,"context_lines":[{"line_number":672,"context_line":""},{"line_number":673,"context_line":"    imported_list \u003d []"},{"line_number":674,"context_line":""},{"line_number":675,"context_line":"    ex_action_deps \u003d args.exists_action_deps"},{"line_number":676,"context_line":"    if ex_action_deps \u003d\u003d \u0027\u0027:"},{"line_number":677,"context_line":"        ex_action_deps \u003d args.exists_action"},{"line_number":678,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"bab6814e_2191789a","line":675,"range":{"start_line":675,"start_character":4,"end_line":675,"end_character":6},"in_reply_to":"bab6814e_669b96ca","updated":"2016-05-19 15:23:18.000000000","message":"Done","commit_id":"7b852cc53347e69f4f7451d7b17b63d2ad3cd651"},{"author":{"_account_id":13149,"name":"Dmytro Dovbii","email":"dmanddov@gmail.com","username":"Ddovbii"},"change_message_id":"d4c66934423bc4d6633c6f884074681b6fa8f892","unresolved":false,"context_lines":[{"line_number":676,"context_line":"    if dep_exists_action \u003d\u003d \u0027\u0027:"},{"line_number":677,"context_line":"        dep_exists_action \u003d args.exists_action"},{"line_number":678,"context_line":""},{"line_number":679,"context_line":"    main_package_name \u003d package.manifest[\u0027FullName\u0027]"},{"line_number":680,"context_line":""},{"line_number":681,"context_line":"    for name, package in six.iteritems(total_reqs):"},{"line_number":682,"context_line":"        image_specs \u003d package.images()"}],"source_content_type":"text/x-python","patch_set":2,"id":"bab6814e_bf841cd4","line":679,"updated":"2016-05-19 16:11:59.000000000","message":"this will work only in case when user pass single name of package to `package-import` command","commit_id":"e49281064fb271361c73d1c743ad04a9c51e55fb"},{"author":{"_account_id":20773,"name":"Valerii Kovalchuk","email":"kovvalole@gmail.com","username":"vakovalchuk"},"change_message_id":"e955dab22f6be7377fc1ced752c4f292a67cc75b","unresolved":false,"context_lines":[{"line_number":676,"context_line":"    if dep_exists_action \u003d\u003d \u0027\u0027:"},{"line_number":677,"context_line":"        dep_exists_action \u003d args.exists_action"},{"line_number":678,"context_line":""},{"line_number":679,"context_line":"    main_package_name \u003d package.manifest[\u0027FullName\u0027]"},{"line_number":680,"context_line":""},{"line_number":681,"context_line":"    for name, package in six.iteritems(total_reqs):"},{"line_number":682,"context_line":"        image_specs \u003d package.images()"}],"source_content_type":"text/x-python","patch_set":2,"id":"bab6814e_7fbd9467","line":679,"in_reply_to":"bab6814e_bf841cd4","updated":"2016-05-19 16:13:46.000000000","message":"Good catch, thank you. Done","commit_id":"e49281064fb271361c73d1c743ad04a9c51e55fb"},{"author":{"_account_id":12597,"name":"Lin Yang","email":"lin.a.yang@intel.com","username":"lin.yang"},"change_message_id":"e0bf7798858d5e2f14040d3e59a9292c315477c1","unresolved":false,"context_lines":[{"line_number":628,"context_line":"@utils.arg(\u0027--exists-action\u0027, default\u003d\u0027\u0027, choices\u003d[\u0027a\u0027, \u0027s\u0027, \u0027u\u0027],"},{"line_number":629,"context_line":"           help\u003d\u0027Default action when a package already exists.\u0027)"},{"line_number":630,"context_line":"@utils.arg(\u0027--dep-exists-action\u0027, default\u003d\u0027\u0027, choices\u003d[\u0027a\u0027, \u0027s\u0027, \u0027u\u0027],"},{"line_number":631,"context_line":"           help\u003d\u0027Default action when a dependency package already exists.\u0027)"},{"line_number":632,"context_line":"def do_package_import(mc, args):"},{"line_number":633,"context_line":"    \"\"\"Import a package."},{"line_number":634,"context_line":"    `FILE` can be either a path to a zip file, url or a FQPN."}],"source_content_type":"text/x-python","patch_set":4,"id":"7aa08908_19869ab8","line":631,"updated":"2016-06-15 05:54:57.000000000","message":"Please consider to add explanation for \u0027a\u0027, \u0027s\u0027, \u0027u\u0027 in help message to new user.","commit_id":"43e31036ffee34bf2911d51dc75dbfdf1c54d838"},{"author":{"_account_id":20773,"name":"Valerii Kovalchuk","email":"kovvalole@gmail.com","username":"vakovalchuk"},"change_message_id":"deb47f49c261ba9f802a3735bb626e82d86a651b","unresolved":false,"context_lines":[{"line_number":628,"context_line":"@utils.arg(\u0027--exists-action\u0027, default\u003d\u0027\u0027, choices\u003d[\u0027a\u0027, \u0027s\u0027, \u0027u\u0027],"},{"line_number":629,"context_line":"           help\u003d\u0027Default action when a package already exists.\u0027)"},{"line_number":630,"context_line":"@utils.arg(\u0027--dep-exists-action\u0027, default\u003d\u0027\u0027, choices\u003d[\u0027a\u0027, \u0027s\u0027, \u0027u\u0027],"},{"line_number":631,"context_line":"           help\u003d\u0027Default action when a dependency package already exists.\u0027)"},{"line_number":632,"context_line":"def do_package_import(mc, args):"},{"line_number":633,"context_line":"    \"\"\"Import a package."},{"line_number":634,"context_line":"    `FILE` can be either a path to a zip file, url or a FQPN."}],"source_content_type":"text/x-python","patch_set":4,"id":"7aa08908_94a36cb1","line":631,"in_reply_to":"7aa08908_19869ab8","updated":"2016-06-15 07:53:01.000000000","message":"Done","commit_id":"43e31036ffee34bf2911d51dc75dbfdf1c54d838"},{"author":{"_account_id":7535,"name":"Nikolay Starodubtsev","email":"starodubcevna@gmail.com","username":"Kezar"},"change_message_id":"6636ffa97884f03b88a860c5f2d8f8684a537d55","unresolved":false,"context_lines":[{"line_number":676,"context_line":""},{"line_number":677,"context_line":"    imported_list \u003d []"},{"line_number":678,"context_line":""},{"line_number":679,"context_line":"    dep_exists_action \u003d args.dep_exists_action"},{"line_number":680,"context_line":"    if dep_exists_action \u003d\u003d \u0027\u0027:"},{"line_number":681,"context_line":"        dep_exists_action \u003d args.exists_action"},{"line_number":682,"context_line":""},{"line_number":683,"context_line":"    for name, package in six.iteritems(total_reqs):"},{"line_number":684,"context_line":"        image_specs \u003d package.images()"}],"source_content_type":"text/x-python","patch_set":5,"id":"7aa08908_4e004836","line":681,"range":{"start_line":679,"start_character":0,"end_line":681,"end_character":46},"updated":"2016-06-15 11:04:31.000000000","message":"why not:\n if args.dep_exists_action:\n     dep_exists_action \u003d args.dep_exists_action\n else:\n     dep_exists_action \u003d args.exists_action","commit_id":"8b5c564b210e13ba858416197113c23aedcb4f1a"},{"author":{"_account_id":15168,"name":"Kirill Zaitsev","email":"k.zaitsev@me.com","username":"kzaitsev"},"change_message_id":"ca4862b0dfc670a4c2a79f04a44297782a32784d","unresolved":false,"context_lines":[{"line_number":676,"context_line":""},{"line_number":677,"context_line":"    imported_list \u003d []"},{"line_number":678,"context_line":""},{"line_number":679,"context_line":"    dep_exists_action \u003d args.dep_exists_action"},{"line_number":680,"context_line":"    if dep_exists_action \u003d\u003d \u0027\u0027:"},{"line_number":681,"context_line":"        dep_exists_action \u003d args.exists_action"},{"line_number":682,"context_line":""},{"line_number":683,"context_line":"    for name, package in six.iteritems(total_reqs):"},{"line_number":684,"context_line":"        image_specs \u003d package.images()"}],"source_content_type":"text/x-python","patch_set":5,"id":"7aa08908_a986df13","line":681,"range":{"start_line":679,"start_character":0,"end_line":681,"end_character":46},"in_reply_to":"7aa08908_241ebb5f","updated":"2016-06-16 01:16:44.000000000","message":"+1 do not see anything criminal in current way of handling things","commit_id":"8b5c564b210e13ba858416197113c23aedcb4f1a"},{"author":{"_account_id":20773,"name":"Valerii Kovalchuk","email":"kovvalole@gmail.com","username":"vakovalchuk"},"change_message_id":"13abd447d4564bdab5a6456a6be783b1e9235812","unresolved":false,"context_lines":[{"line_number":676,"context_line":""},{"line_number":677,"context_line":"    imported_list \u003d []"},{"line_number":678,"context_line":""},{"line_number":679,"context_line":"    dep_exists_action \u003d args.dep_exists_action"},{"line_number":680,"context_line":"    if dep_exists_action \u003d\u003d \u0027\u0027:"},{"line_number":681,"context_line":"        dep_exists_action \u003d args.exists_action"},{"line_number":682,"context_line":""},{"line_number":683,"context_line":"    for name, package in six.iteritems(total_reqs):"},{"line_number":684,"context_line":"        image_specs \u003d package.images()"}],"source_content_type":"text/x-python","patch_set":5,"id":"7aa08908_241ebb5f","line":681,"range":{"start_line":679,"start_character":0,"end_line":681,"end_character":46},"in_reply_to":"7aa08908_4e004836","updated":"2016-06-15 11:25:14.000000000","message":"what\u0027s so wrong about the current way of doing it?","commit_id":"8b5c564b210e13ba858416197113c23aedcb4f1a"},{"author":{"_account_id":20364,"name":"Konstantin Snihyr","email":"konstantin.lex@gmail.com","username":"ksnihyr"},"change_message_id":"a91004aafb8c87717bc0db79723b1c491ee3bcd5","unresolved":false,"context_lines":[{"line_number":676,"context_line":""},{"line_number":677,"context_line":"    imported_list \u003d []"},{"line_number":678,"context_line":""},{"line_number":679,"context_line":"    dep_exists_action \u003d args.dep_exists_action"},{"line_number":680,"context_line":"    if dep_exists_action \u003d\u003d \u0027\u0027:"},{"line_number":681,"context_line":"        dep_exists_action \u003d args.exists_action"},{"line_number":682,"context_line":""},{"line_number":683,"context_line":"    for name, package in six.iteritems(total_reqs):"},{"line_number":684,"context_line":"        image_specs \u003d package.images()"}],"source_content_type":"text/x-python","patch_set":5,"id":"5a9d85d2_778864e8","line":681,"range":{"start_line":679,"start_character":0,"end_line":681,"end_character":46},"in_reply_to":"7aa08908_a986df13","updated":"2016-06-20 14:01:40.000000000","message":"More explicit way: declare default action in utils.arg decorator","commit_id":"8b5c564b210e13ba858416197113c23aedcb4f1a"}],"releasenotes/notes/dep-exists-action-9af18bebcc0ef053.yaml":[{"author":{"_account_id":15168,"name":"Kirill Zaitsev","email":"k.zaitsev@me.com","username":"kzaitsev"},"change_message_id":"89debaad128547130ad48bfc84e040fa0c1058d3","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"features:"},{"line_number":4,"context_line":"  - Added \u0027--exists-action-deps\u0027 option to murano command-line interface."},{"line_number":5,"context_line":"    It allows to apply different default actions for existing main"},{"line_number":6,"context_line":"    package and existing dependencies. In case of specifying just"},{"line_number":7,"context_line":"    \u0027--exists-action\u0027, action for dependencies equals to that like it"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"bab6814e_59bb89e5","line":4,"range":{"start_line":4,"start_character":11,"end_line":4,"end_character":31},"updated":"2016-05-19 17:27:54.000000000","message":"--dep-exists-action","commit_id":"387325ef6f77587ad61fcd0f3397a391f1574d4a"},{"author":{"_account_id":20773,"name":"Valerii Kovalchuk","email":"kovvalole@gmail.com","username":"vakovalchuk"},"change_message_id":"d2d336e6712b742a56edf2eb88373b4bbd3b5a8c","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"features:"},{"line_number":4,"context_line":"  - Added \u0027--exists-action-deps\u0027 option to murano command-line interface."},{"line_number":5,"context_line":"    It allows to apply different default actions for existing main"},{"line_number":6,"context_line":"    package and existing dependencies. In case of specifying just"},{"line_number":7,"context_line":"    \u0027--exists-action\u0027, action for dependencies equals to that like it"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"bab6814e_597a69f2","line":4,"range":{"start_line":4,"start_character":11,"end_line":4,"end_character":31},"in_reply_to":"bab6814e_59bb89e5","updated":"2016-05-19 17:33:57.000000000","message":"Done here and in commit message","commit_id":"387325ef6f77587ad61fcd0f3397a391f1574d4a"}]}