)]}'
{"magnumclient/osc/v1/clusters.py":[{"author":{"_account_id":20498,"name":"Spyros Trigazis","email":"spyridon.trigazis@cern.ch","username":"strigazi"},"change_message_id":"e2d0944b85425a471f6e0ac4b4173ed69f0d7df2","unresolved":false,"context_lines":[{"line_number":72,"context_line":"        parser.add_argument(\u0027--labels\u0027,"},{"line_number":73,"context_line":"                            metavar\u003d\u0027\u003cKEY1\u003dVALUE1,KEY2\u003dVALUE2;KEY3\u003dVALUE3...\u003e\u0027,"},{"line_number":74,"context_line":"                            action\u003d\u0027append\u0027,"},{"line_number":75,"context_line":"                            default\u003dNone,"},{"line_number":76,"context_line":"                            help\u003d_(\u0027Arbitrary labels in the form of key\u003dvalue\u0027"},{"line_number":77,"context_line":"                                   \u0027pairs to associate with a cluster \u0027"},{"line_number":78,"context_line":"                                   \u0027template. May be used multiple times.\u0027))"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f4b6375_7ed343d6","line":75,"range":{"start_line":75,"start_character":28,"end_line":75,"end_character":41},"updated":"2017-10-25 09:37:35.000000000","message":"We need to remove this completely.","commit_id":"f025b5a77f40582cbc445ad38ab14ca28659e715"},{"author":{"_account_id":20498,"name":"Spyros Trigazis","email":"spyridon.trigazis@cern.ch","username":"strigazi"},"change_message_id":"d689727b42bdf0c3ab9ef3951e9c8d5125ac36dd","unresolved":false,"context_lines":[{"line_number":114,"context_line":"            \u0027discovery_url\u0027: parsed_args.discovery_url,"},{"line_number":115,"context_line":"            \u0027docker_volume_size\u0027: parsed_args.docker_volume_size,"},{"line_number":116,"context_line":"            \u0027keypair\u0027: parsed_args.keypair,"},{"line_number":117,"context_line":"            \u0027labels\u0027: parsed_args.labels,"},{"line_number":118,"context_line":"            \u0027master_count\u0027: parsed_args.master_count,"},{"line_number":119,"context_line":"            \u0027name\u0027: parsed_args.name,"},{"line_number":120,"context_line":"            \u0027node_count\u0027: parsed_args.node_count,"}],"source_content_type":"text/x-python","patch_set":4,"id":"5f4e5783_72604f02","line":117,"range":{"start_line":117,"start_character":12,"end_line":117,"end_character":41},"updated":"2017-10-13 11:54:08.000000000","message":"this shouldn\u0027t be here, we add the labels at line 124","commit_id":"f025b5a77f40582cbc445ad38ab14ca28659e715"},{"author":{"_account_id":20498,"name":"Spyros Trigazis","email":"spyridon.trigazis@cern.ch","username":"strigazi"},"change_message_id":"1dfc8b15a0d3adda761e18806c23c79ae7c798b1","unresolved":false,"context_lines":[{"line_number":114,"context_line":"            \u0027discovery_url\u0027: parsed_args.discovery_url,"},{"line_number":115,"context_line":"            \u0027docker_volume_size\u0027: parsed_args.docker_volume_size,"},{"line_number":116,"context_line":"            \u0027keypair\u0027: parsed_args.keypair,"},{"line_number":117,"context_line":"            \u0027labels\u0027: parsed_args.labels,"},{"line_number":118,"context_line":"            \u0027master_count\u0027: parsed_args.master_count,"},{"line_number":119,"context_line":"            \u0027name\u0027: parsed_args.name,"},{"line_number":120,"context_line":"            \u0027node_count\u0027: parsed_args.node_count,"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f4b6375_bec2db57","line":117,"range":{"start_line":117,"start_character":12,"end_line":117,"end_character":41},"in_reply_to":"3f4b6375_9016ee54","updated":"2017-10-25 09:26:19.000000000","message":"With this change the api receives None instead of [1] wtypes.Unset\n\nSo we are sending None instead of no value.\n\n[1] https://review.openstack.org/#/c/473346/22/magnum/api/controllers/v1/cluster.py@411","commit_id":"f025b5a77f40582cbc445ad38ab14ca28659e715"},{"author":{"_account_id":9995,"name":"Ricardo Rocha","email":"rocha.porto@gmail.com","username":"rocha"},"change_message_id":"b4bd3a4eeb0ea74aacd490a30d40747b1d45eed3","unresolved":false,"context_lines":[{"line_number":114,"context_line":"            \u0027discovery_url\u0027: parsed_args.discovery_url,"},{"line_number":115,"context_line":"            \u0027docker_volume_size\u0027: parsed_args.docker_volume_size,"},{"line_number":116,"context_line":"            \u0027keypair\u0027: parsed_args.keypair,"},{"line_number":117,"context_line":"            \u0027labels\u0027: parsed_args.labels,"},{"line_number":118,"context_line":"            \u0027master_count\u0027: parsed_args.master_count,"},{"line_number":119,"context_line":"            \u0027name\u0027: parsed_args.name,"},{"line_number":120,"context_line":"            \u0027node_count\u0027: parsed_args.node_count,"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f4b6375_9016ee54","line":117,"range":{"start_line":117,"start_character":12,"end_line":117,"end_character":41},"in_reply_to":"5f4e5783_72604f02","updated":"2017-10-25 08:33:23.000000000","message":"it needs to be there for the tests to pass when None is given, otherwise args has no \u0027labels\u0027 entry.","commit_id":"f025b5a77f40582cbc445ad38ab14ca28659e715"}],"magnumclient/v1/clusters_shell.py":[{"author":{"_account_id":20498,"name":"Spyros Trigazis","email":"spyridon.trigazis@cern.ch","username":"strigazi"},"change_message_id":"e2d0944b85425a471f6e0ac4b4173ed69f0d7df2","unresolved":false,"context_lines":[{"line_number":106,"context_line":"           type\u003dint,"},{"line_number":107,"context_line":"           help\u003d_(\u0027The size in GB for the docker volume to use\u0027))"},{"line_number":108,"context_line":"@utils.arg(\u0027--labels\u0027, metavar\u003d\u0027\u003cKEY1\u003dVALUE1,KEY2\u003dVALUE2;KEY3\u003dVALUE3...\u003e\u0027,"},{"line_number":109,"context_line":"           action\u003d\u0027append\u0027, default\u003dNone,"},{"line_number":110,"context_line":"           help\u003d_(\u0027Arbitrary labels in the form of key\u003dvalue pairs \u0027"},{"line_number":111,"context_line":"                  \u0027to associate with a cluster. \u0027"},{"line_number":112,"context_line":"                  \u0027May be used multiple times.\u0027))"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f4b6375_1e4d6fc3","line":109,"range":{"start_line":109,"start_character":26,"end_line":109,"end_character":40},"updated":"2017-10-25 09:37:35.000000000","message":"We need to remove this completely.","commit_id":"f025b5a77f40582cbc445ad38ab14ca28659e715"}]}
