)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"b23e2167744c25539dfefcc27ff636f49b006b8e","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Polina-Gubina \u003cpolinagubina6@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-09-14 10:01:37 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"add missing function to identity resource"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ic2986b107c921752c9e174fe4b7209eec9bf7704"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"ca05a96a_38ce0f04","line":7,"updated":"2022-09-28 10:57:48.000000000","message":"As a general point, it would be better to list the function you\u0027re adding and (ideally) give a quick synopsis of what it does. Just helps reviewers a little bit (it\u0027s a tough job!) 😊","commit_id":"cabfc8da7922537fc3f45e5e83c30eb65ac5a772"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"eb7d4bcc19a91d4872c6cd70825c63ec4d5045b0","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Polina-Gubina \u003cpolinagubina6@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-09-14 10:01:37 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"add missing function to identity resource"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ic2986b107c921752c9e174fe4b7209eec9bf7704"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"b16c364e_c4627d78","line":7,"in_reply_to":"ca05a96a_38ce0f04","updated":"2022-10-18 12:18:48.000000000","message":"Done","commit_id":"cabfc8da7922537fc3f45e5e83c30eb65ac5a772"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"388f72ca22ed989137bebd3b067b50701f568409","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"5f0d1e2c_aebdccb6","updated":"2022-09-28 10:58:52.000000000","message":"Also, can you please add a release note? e.g.\n\n  reno new add-identity-group-users-proxy-method","commit_id":"cabfc8da7922537fc3f45e5e83c30eb65ac5a772"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"b23e2167744c25539dfefcc27ff636f49b006b8e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"9a4f6d96_fb5926bb","updated":"2022-09-28 10:57:48.000000000","message":"Looks pretty good. Couple of small nits. If you can address those, I\u0027m +w","commit_id":"cabfc8da7922537fc3f45e5e83c30eb65ac5a772"}],"openstack/identity/v3/_proxy.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"b23e2167744c25539dfefcc27ff636f49b006b8e","unresolved":true,"context_lines":[{"line_number":407,"context_line":"        return group.check_user(self, user)"},{"line_number":408,"context_line":""},{"line_number":409,"context_line":"    def group_users(self, group, **attrs):"},{"line_number":410,"context_line":"        \"\"\"Users list in group"},{"line_number":411,"context_line":""},{"line_number":412,"context_line":"        :param attrs: Only password_expires_at can be filter for result."},{"line_number":413,"context_line":"        :param group: Either the ID of a group or a"}],"source_content_type":"text/x-python","patch_set":7,"id":"40cce1ed_84d6dd52","line":410,"range":{"start_line":410,"start_character":11,"end_line":410,"end_character":30},"updated":"2022-09-28 10:57:48.000000000","message":"nit: List users in a group","commit_id":"cabfc8da7922537fc3f45e5e83c30eb65ac5a772"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"eb7d4bcc19a91d4872c6cd70825c63ec4d5045b0","unresolved":false,"context_lines":[{"line_number":407,"context_line":"        return group.check_user(self, user)"},{"line_number":408,"context_line":""},{"line_number":409,"context_line":"    def group_users(self, group, **attrs):"},{"line_number":410,"context_line":"        \"\"\"Users list in group"},{"line_number":411,"context_line":""},{"line_number":412,"context_line":"        :param attrs: Only password_expires_at can be filter for result."},{"line_number":413,"context_line":"        :param group: Either the ID of a group or a"}],"source_content_type":"text/x-python","patch_set":7,"id":"7da80989_dcb53cbf","line":410,"range":{"start_line":410,"start_character":11,"end_line":410,"end_character":30},"in_reply_to":"40cce1ed_84d6dd52","updated":"2022-10-18 12:18:48.000000000","message":"Done","commit_id":"cabfc8da7922537fc3f45e5e83c30eb65ac5a772"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"b23e2167744c25539dfefcc27ff636f49b006b8e","unresolved":true,"context_lines":[{"line_number":411,"context_line":""},{"line_number":412,"context_line":"        :param attrs: Only password_expires_at can be filter for result."},{"line_number":413,"context_line":"        :param group: Either the ID of a group or a"},{"line_number":414,"context_line":"            :class:`~openstack.identity.v3.group.Group` instance."},{"line_number":415,"context_line":"        :return: :class:`~openstack.identity.v3.user.User`"},{"line_number":416,"context_line":"        \"\"\""},{"line_number":417,"context_line":"        group \u003d self._get_resource(_group.Group, group)"}],"source_content_type":"text/x-python","patch_set":7,"id":"7aeede52_ec3d197a","line":414,"updated":"2022-09-28 10:57:48.000000000","message":"These are backwards.\n\n  :param group: ...\n  :param attrs: ...","commit_id":"cabfc8da7922537fc3f45e5e83c30eb65ac5a772"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"eb7d4bcc19a91d4872c6cd70825c63ec4d5045b0","unresolved":false,"context_lines":[{"line_number":411,"context_line":""},{"line_number":412,"context_line":"        :param attrs: Only password_expires_at can be filter for result."},{"line_number":413,"context_line":"        :param group: Either the ID of a group or a"},{"line_number":414,"context_line":"            :class:`~openstack.identity.v3.group.Group` instance."},{"line_number":415,"context_line":"        :return: :class:`~openstack.identity.v3.user.User`"},{"line_number":416,"context_line":"        \"\"\""},{"line_number":417,"context_line":"        group \u003d self._get_resource(_group.Group, group)"}],"source_content_type":"text/x-python","patch_set":7,"id":"7b15588f_5026f448","line":414,"in_reply_to":"7aeede52_ec3d197a","updated":"2022-10-18 12:18:48.000000000","message":"Done","commit_id":"cabfc8da7922537fc3f45e5e83c30eb65ac5a772"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"b23e2167744c25539dfefcc27ff636f49b006b8e","unresolved":true,"context_lines":[{"line_number":412,"context_line":"        :param attrs: Only password_expires_at can be filter for result."},{"line_number":413,"context_line":"        :param group: Either the ID of a group or a"},{"line_number":414,"context_line":"            :class:`~openstack.identity.v3.group.Group` instance."},{"line_number":415,"context_line":"        :return: :class:`~openstack.identity.v3.user.User`"},{"line_number":416,"context_line":"        \"\"\""},{"line_number":417,"context_line":"        group \u003d self._get_resource(_group.Group, group)"},{"line_number":418,"context_line":"        base_path \u003d utils.urljoin("}],"source_content_type":"text/x-python","patch_set":7,"id":"c792928d_28fcf5b2","line":415,"updated":"2022-09-28 10:57:48.000000000","message":"We return a *list* of these.","commit_id":"cabfc8da7922537fc3f45e5e83c30eb65ac5a772"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"eb7d4bcc19a91d4872c6cd70825c63ec4d5045b0","unresolved":false,"context_lines":[{"line_number":412,"context_line":"        :param attrs: Only password_expires_at can be filter for result."},{"line_number":413,"context_line":"        :param group: Either the ID of a group or a"},{"line_number":414,"context_line":"            :class:`~openstack.identity.v3.group.Group` instance."},{"line_number":415,"context_line":"        :return: :class:`~openstack.identity.v3.user.User`"},{"line_number":416,"context_line":"        \"\"\""},{"line_number":417,"context_line":"        group \u003d self._get_resource(_group.Group, group)"},{"line_number":418,"context_line":"        base_path \u003d utils.urljoin("}],"source_content_type":"text/x-python","patch_set":7,"id":"20bcb2f5_6590fd7f","line":415,"in_reply_to":"c792928d_28fcf5b2","updated":"2022-10-18 12:18:48.000000000","message":"Done","commit_id":"cabfc8da7922537fc3f45e5e83c30eb65ac5a772"}],"openstack/tests/unit/identity/v3/test_proxy.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"b23e2167744c25539dfefcc27ff636f49b006b8e","unresolved":true,"context_lines":[{"line_number":168,"context_line":"            ]"},{"line_number":169,"context_line":"        )"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"    def test_group_users(self):"},{"line_number":172,"context_line":"        self.verify_list(self.proxy.group_users, user.User,"},{"line_number":173,"context_line":"                         method_kwargs\u003d{\"group\": \u0027group\u0027, \"attrs\": 1},"},{"line_number":174,"context_line":"                         expected_kwargs\u003d{\"attrs\": 1})"}],"source_content_type":"text/x-python","patch_set":7,"id":"d3dd781b_e328e289","line":171,"updated":"2022-09-28 10:57:48.000000000","message":"Can you please drop this onto the next line:\n\n  self.verify_list(\n      self.proxy.group_users,\n      user.User,\n      method_kwargs\u003d{\"group\": \u0027group\u0027, \"attrs\": 1},\n      expected_kwargs\u003d{\"attrs\": 1},\n  )","commit_id":"cabfc8da7922537fc3f45e5e83c30eb65ac5a772"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"eb7d4bcc19a91d4872c6cd70825c63ec4d5045b0","unresolved":false,"context_lines":[{"line_number":168,"context_line":"            ]"},{"line_number":169,"context_line":"        )"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"    def test_group_users(self):"},{"line_number":172,"context_line":"        self.verify_list(self.proxy.group_users, user.User,"},{"line_number":173,"context_line":"                         method_kwargs\u003d{\"group\": \u0027group\u0027, \"attrs\": 1},"},{"line_number":174,"context_line":"                         expected_kwargs\u003d{\"attrs\": 1})"}],"source_content_type":"text/x-python","patch_set":7,"id":"9297d24b_f3225c65","line":171,"in_reply_to":"d3dd781b_e328e289","updated":"2022-10-18 12:18:48.000000000","message":"Done","commit_id":"cabfc8da7922537fc3f45e5e83c30eb65ac5a772"}]}