)]}'
{"magnumclient/v1/baymodels_shell.py":[{"author":{"_account_id":21660,"name":"Stephen Watson","email":"stephen.watson@intel.com","username":"swatson"},"change_message_id":"ab946748933bee2c6bec8ba403d124ccce2637d8","unresolved":false,"context_lines":[{"line_number":104,"context_line":"           action\u003d\u0027store_true\u0027, default\u003dFalse,"},{"line_number":105,"context_line":"           help\u003d_(\u0027Disable TLS in the Bay.\u0027))"},{"line_number":106,"context_line":"@utils.arg(\u0027--verify-ca\u0027,"},{"line_number":107,"context_line":"           help\u003d_(\u0027Verify certificate authority.\u0027),"},{"line_number":108,"context_line":"           type\u003dlambda v: strutils.bool_from_string(v, True),"},{"line_number":109,"context_line":"           default\u003dTrue)"},{"line_number":110,"context_line":"@utils.arg(\u0027--public\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"dfeb2761_506351dd","line":107,"updated":"2017-03-31 19:17:54.000000000","message":"You can avoid the lambda here by using \"action\u003d\u0027store_true\u0027\" as in some of our other decorators.","commit_id":"79b970d2d54441256a5efc77fb36bbf24449ec46"},{"author":{"_account_id":7230,"name":"Vijendar Komalla","email":"vijendar.komalla@rackspace.com","username":"vijendar"},"change_message_id":"5a33f74b661bcfc54f32cc937183a0d51feb1961","unresolved":false,"context_lines":[{"line_number":104,"context_line":"           action\u003d\u0027store_true\u0027, default\u003dFalse,"},{"line_number":105,"context_line":"           help\u003d_(\u0027Disable TLS in the Bay.\u0027))"},{"line_number":106,"context_line":"@utils.arg(\u0027--verify-ca\u0027,"},{"line_number":107,"context_line":"           help\u003d_(\u0027Verify certificate authority.\u0027),"},{"line_number":108,"context_line":"           type\u003dlambda v: strutils.bool_from_string(v, True),"},{"line_number":109,"context_line":"           default\u003dTrue)"},{"line_number":110,"context_line":"@utils.arg(\u0027--public\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"dfeb2761_9021d950","line":107,"in_reply_to":"dfeb2761_506351dd","updated":"2017-03-31 19:25:45.000000000","message":"Thanks Steve for reviewing. That logic does not seem to work here since the flag default value is \u0027True\u0027.\n\n1. Here default value set to \u0027True\u0027, that means if the switch is not present in the command line args \u0027True\u0027 will be set\n2. If the switch is provided (it\u0027s not possible to provide true/false value here), the value will be set to \u0027True\u0027 because action is set to \u0027store_true\u0027\n\nLet me know your thoughts.","commit_id":"79b970d2d54441256a5efc77fb36bbf24449ec46"}]}