)]}'
{"zun/api/controllers/v1/schemas/containers.py":[{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"768805359d5005ea429e30f423174a26225172a7","unresolved":false,"context_lines":[{"line_number":74,"context_line":"    \u0027type\u0027: \u0027object\u0027,"},{"line_number":75,"context_line":"    \u0027properties\u0027: {"},{"line_number":76,"context_line":"        \u0027force\u0027: parameter_types.boolean_extended,"},{"line_number":77,"context_line":"        \u0027all_projects\u0027: parameter_types.boolean_extended,"},{"line_number":78,"context_line":"        \u0027stop\u0027: parameter_types.boolean_extended"},{"line_number":79,"context_line":"    },"},{"line_number":80,"context_line":"    \u0027additionalProperties\u0027: False"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f91af0f_f7743fd1","line":77,"range":{"start_line":77,"start_character":9,"end_line":77,"end_character":21},"updated":"2018-01-09 03:34:01.000000000","message":"@miaohb,\n\nThe gate failed because tempest was sending \u0027all_tenants\u0027 to server but the schema was changed to \u0027all_projects\u0027 in here.\n\nI think the issue can be fixed by doing the following:\n\n1. Add \u0027all_projects\u0027 in this patch (keep \u0027all_tenants\u0027 unchanged).\n2. Make tempest to send \u0027all_projects\u0027 to server (as you did) in https://review.openstack.org/#/c/531655/ .\n3. Remove \u0027all_tenants\u0027 in server\n\nDo the steps above one by one. Then, the gate should be happy. Let me know if you have any question about that.","commit_id":"0d6cd3ad7e58e054f62e71fba8692b32836d3c4b"}],"zun/common/utils.py":[{"author":{"_account_id":22406,"name":"Kien Nguyen","email":"kiennt2609@gmail.com","username":"kiennt"},"change_message_id":"2f94e47e16e040aca9376e9f6df52f30121faf84","unresolved":false,"context_lines":[{"line_number":343,"context_line":""},{"line_number":344,"context_line":""},{"line_number":345,"context_line":"def is_all_projects(search_opts):"},{"line_number":346,"context_line":"    all_projects \u003d search_opts.get(\u0027all_projects\u0027)"},{"line_number":347,"context_line":"    if all_projects:"},{"line_number":348,"context_line":"        try:"},{"line_number":349,"context_line":"            all_projects \u003d strutils.bool_from_string(all_projects, True)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f91af0f_81726c48","line":346,"range":{"start_line":346,"start_character":19,"end_line":346,"end_character":50},"updated":"2018-01-08 08:44:23.000000000","message":"The patch in Zun-tempest-plugin has to depend on this one. But this patch also needs Zun-tempest-plugin. What a vicious circle!\n\nIMO, we can work around with it like:\n\nall_projects \u003d search_opts.get(\u0027all_projects\u0027) or search_opts.get(\u0027all_tenants\u0027)\n\nThen we will remove in the follow-up, after both zun \u0026 zun-tempest-plugin were merged. What do you think?","commit_id":"df74d0566103be16cff47e278801b3267db5d8b7"},{"author":{"_account_id":23055,"name":"miaohb","email":"miao.hongbao@zte.com.cn","username":"miaohb"},"change_message_id":"6d8206e9bae2006bc84c6fc817d388f9836b54fb","unresolved":false,"context_lines":[{"line_number":343,"context_line":""},{"line_number":344,"context_line":""},{"line_number":345,"context_line":"def is_all_projects(search_opts):"},{"line_number":346,"context_line":"    all_projects \u003d search_opts.get(\u0027all_projects\u0027)"},{"line_number":347,"context_line":"    if all_projects:"},{"line_number":348,"context_line":"        try:"},{"line_number":349,"context_line":"            all_projects \u003d strutils.bool_from_string(all_projects, True)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f91af0f_e4f02600","line":346,"range":{"start_line":346,"start_character":19,"end_line":346,"end_character":50},"in_reply_to":"9f91af0f_81726c48","updated":"2018-01-08 08:59:37.000000000","message":"Good idea! Thank you, Kien!","commit_id":"df74d0566103be16cff47e278801b3267db5d8b7"}]}
