)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"32a44ea91f67f901f2db2a0d29ab6914358d78cf","unresolved":false,"context_lines":[{"line_number":10,"context_line":"have identical \"created_at\" values. This can occur when creating"},{"line_number":11,"context_line":"instance using the \"min/max count\" option. The reverse sort does not work"},{"line_number":12,"context_line":"correctly as the server list returned from nova is not an exact"},{"line_number":13,"context_line":"reverse as the forward sort. In order to resolve this, a second"},{"line_number":14,"context_line":"key \"display_name\" is added. This ensures the forward and reverse"},{"line_number":15,"context_line":"sort are exact opposites."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Closes-Bug #1856243"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3fa7e38b_dd2cd6b3","line":14,"range":{"start_line":13,"start_character":29,"end_line":14,"end_character":28},"updated":"2019-12-24 23:14:32.000000000","message":"Three comments;\n\n- Could you explain what this means? You don\u0027t explain why adding  the new key \"display_name\" addresses the issue.\n- As I suggested in the previous comment, \"uuid\" is required.\n- \"display_name\" itself does not solve the issue perfectly and adding \"uuid\" is enough to solve the issue. If you still would like  to have \"display_name\" could you explain the reason for \"display_name\"?","commit_id":"99d08aceba19017ee51a7e6519daf1450135b91f"}],"openstack_dashboard/api/nova.py":[{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"48057b2416c2c95d4b04c453c5d19d91832e9bfb","unresolved":false,"context_lines":[{"line_number":595,"context_line":"        view_marker \u003d \u0027possibly_deleted\u0027 if deleted and marker else \u0027ok\u0027"},{"line_number":596,"context_line":"        search_opts[\u0027marker\u0027] \u003d deleted if deleted else marker"},{"line_number":597,"context_line":"        search_opts[\u0027limit\u0027] \u003d page_size + 1"},{"line_number":598,"context_line":"        search_opts[\u0027sort_dir\u0027] \u003d sort_dir"},{"line_number":599,"context_line":"        servers \u003d [Server(s, request)"},{"line_number":600,"context_line":"                   for s in nova_client.servers.list(detailed, search_opts)]"},{"line_number":601,"context_line":"        if view_marker \u003d\u003d \u0027possibly_deleted\u0027:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_d2c62a4b","side":"PARENT","line":598,"updated":"2019-12-19 18:15:54.000000000","message":"note that this option is limited to single sort_key and its corresponding sort_dir","commit_id":"f7313b74c909030210df9dae717008bc7e005ce2"},{"author":{"_account_id":12156,"name":"Jeremy Moffitt","email":"jmoffitt@suse.com","username":"jmoffitt"},"change_message_id":"825cd0a22556882c2a4e6069bbb6f8122ad48628","unresolved":false,"context_lines":[{"line_number":595,"context_line":"        view_marker \u003d \u0027possibly_deleted\u0027 if deleted and marker else \u0027ok\u0027"},{"line_number":596,"context_line":"        search_opts[\u0027marker\u0027] \u003d deleted if deleted else marker"},{"line_number":597,"context_line":"        search_opts[\u0027limit\u0027] \u003d page_size + 1"},{"line_number":598,"context_line":"        search_opts[\u0027sort_dir\u0027] \u003d sort_dir"},{"line_number":599,"context_line":"        servers \u003d [Server(s, request)"},{"line_number":600,"context_line":"                   for s in nova_client.servers.list(detailed, search_opts)]"},{"line_number":601,"context_line":"        if view_marker \u003d\u003d \u0027possibly_deleted\u0027:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_4d5b7ff8","side":"PARENT","line":598,"in_reply_to":"3fa7e38b_d2c62a4b","updated":"2019-12-19 19:09:22.000000000","message":"I suspect thats why we werent getting the right behavior with just the sort_keys change, as there was a mismatch in the number of sort parameters...  hence the switch to passing sort_dirs as well (https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/servers.py#L857)","commit_id":"f7313b74c909030210df9dae717008bc7e005ce2"}]}
