)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"c937921e430434779bd2612e134763be4b9a7a13","unresolved":false,"context_lines":[{"line_number":24,"context_line":"                }"},{"line_number":25,"context_line":"            ]"},{"line_number":26,"context_line":"        },"},{"line_number":27,"context_line":"        \"adminPass\": \"seekr3t\","},{"line_number":28,"context_line":"        \"created\": \"2012-09-12T17:20:36Z\","},{"line_number":29,"context_line":"        \"flavor\": {"},{"line_number":30,"context_line":"            \"id\": \"1\","}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1ae5cdf2_c39c58c3","line":27,"updated":"2014-06-03 08:06:42.000000000","message":"adminPass is only present if nova is configured with CONF.enable_instance_password enabled (which is the default).\n\nWill need a tempest config variable for testing against Nova setups without this configured.","commit_id":"93f6d704fc42fec92dfcec10e85da91f19aa36d1"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"9337c0625e941c192185c54ac2dab2a5f397edf9","unresolved":false,"context_lines":[{"line_number":24,"context_line":"                }"},{"line_number":25,"context_line":"            ]"},{"line_number":26,"context_line":"        },"},{"line_number":27,"context_line":"        \"adminPass\": \"seekr3t\","},{"line_number":28,"context_line":"        \"created\": \"2012-09-12T17:20:36Z\","},{"line_number":29,"context_line":"        \"flavor\": {"},{"line_number":30,"context_line":"            \"id\": \"1\","}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"baada198_248ad764","line":27,"in_reply_to":"1ae5cdf2_4e49435f","updated":"2014-07-22 12:36:24.000000000","message":"Done.","commit_id":"93f6d704fc42fec92dfcec10e85da91f19aa36d1"},{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"b6d81109ca300d322caddfe50e3d128d2d24e2ca","unresolved":false,"context_lines":[{"line_number":24,"context_line":"                }"},{"line_number":25,"context_line":"            ]"},{"line_number":26,"context_line":"        },"},{"line_number":27,"context_line":"        \"adminPass\": \"seekr3t\","},{"line_number":28,"context_line":"        \"created\": \"2012-09-12T17:20:36Z\","},{"line_number":29,"context_line":"        \"flavor\": {"},{"line_number":30,"context_line":"            \"id\": \"1\","}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1ae5cdf2_4e49435f","line":27,"in_reply_to":"1ae5cdf2_8b4a62aa","updated":"2014-06-03 12:18:05.000000000","message":"yes. I know its a bit awkward on the tempest side, but if CONF.enable_instance_password is set to true on the nova side then adminPass should be required and if its not it definitely should not appear. Its a bit of a #fail on the nova side implementation  but its what we have to live with.","commit_id":"93f6d704fc42fec92dfcec10e85da91f19aa36d1"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"7daf92f801675d31bbde7b00c9eefbcca128b286","unresolved":false,"context_lines":[{"line_number":24,"context_line":"                }"},{"line_number":25,"context_line":"            ]"},{"line_number":26,"context_line":"        },"},{"line_number":27,"context_line":"        \"adminPass\": \"seekr3t\","},{"line_number":28,"context_line":"        \"created\": \"2012-09-12T17:20:36Z\","},{"line_number":29,"context_line":"        \"flavor\": {"},{"line_number":30,"context_line":"            \"id\": \"1\","}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1ae5cdf2_8b4a62aa","line":27,"in_reply_to":"1ae5cdf2_c39c58c3","updated":"2014-06-03 09:42:54.000000000","message":"Thanks Chris for review.\nYes, adminPass depend on the config value, and that\u0027s why we have not defined adminPass as \u0027required\u0027 field. \n\nAre you suggesting to check adminPass based on the conf value? In that case, we need to have condition at service_client side to validate against the 2 different schema.","commit_id":"93f6d704fc42fec92dfcec10e85da91f19aa36d1"}]}