)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"3eba94f013a0b43f3087ba17c51538bb0813bdbf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"eff529c3_4d8636fb","updated":"2022-11-25 16:38:57.000000000","message":"As you can see, you have some unit tests to fix. However, this change doesn\u0027t appear to do anything for me. See my comments inline","commit_id":"bccca3c2aa52886fcdb370b0e276e7153fec2ba5"}],"openstack/cloud/_identity.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"3eba94f013a0b43f3087ba17c51538bb0813bdbf","unresolved":true,"context_lines":[{"line_number":1185,"context_line":""},{"line_number":1186,"context_line":"        if user:"},{"line_number":1187,"context_line":"            # use cloud.get_user to save us from bad searching by name"},{"line_number":1188,"context_line":"            data[\u0027user\u0027] \u003d self.get_user(user, **search_args)"},{"line_number":1189,"context_line":"        if group:"},{"line_number":1190,"context_line":"            data[\u0027group\u0027] \u003d self.identity.find_group("},{"line_number":1191,"context_line":"                group, ignore_missing\u003dFalse, **search_args)"}],"source_content_type":"text/x-python","patch_set":2,"id":"0cd1f0cc_481e16f9","line":1188,"range":{"start_line":1188,"start_character":60,"end_line":1188,"end_character":61},"updated":"2022-11-25 16:38:57.000000000","message":"This seems to work just fine as-is.\n\n    \u003e\u003e\u003e import openstack\n    \u003e\u003e\u003e conn \u003d openstack.connect()\n    \u003e\u003e\u003e conn.get_user(\u0027admin\u0027, domain_id\u003d\u0027default\u0027)\n    openstack.identity.v3.user.User(id\u003de81a2d7a033642beae9932ba28f176b8, name\u003dadmin, domain_id\u003ddefault, enabled\u003dTrue, password_expires_at\u003dNone, links\u003d{\u0027self\u0027: \u0027http://10.0.109.180/identity/v3/users/e81a2d7a033642beae9932ba28f176b8\u0027}, location\u003dMunch({\u0027cloud\u0027: \u0027devstack-admin\u0027, \u0027region_name\u0027: \u0027RegionOne\u0027, \u0027zone\u0027: None, \u0027project\u0027: Munch({\u0027id\u0027: \u0027c4fdc1bcf30541048ca086d7332a57e3\u0027, \u0027name\u0027: \u0027admin\u0027, \u0027domain_id\u0027: \u0027default\u0027, \u0027domain_name\u0027: None})}))\n    \u003e\u003e\u003e conn.get_user(\u0027admin\u0027, domain_id\u003d\u0027default2\u0027)\n    \u003e\u003e\u003e conn.get_user(\u0027admin\u0027, filters\u003d{\u0027domain_id\u0027: \u0027default\u0027})\n    openstack.identity.v3.user.User(id\u003de81a2d7a033642beae9932ba28f176b8, name\u003dadmin, domain_id\u003ddefault, enabled\u003dTrue, password_expires_at\u003dNone, links\u003d{\u0027self\u0027: \u0027http://10.0.109.180/identity/v3/users/e81a2d7a033642beae9932ba28f176b8\u0027}, location\u003dMunch({\u0027cloud\u0027: \u0027devstack-admin\u0027, \u0027region_name\u0027: \u0027RegionOne\u0027, \u0027zone\u0027: None, \u0027project\u0027: Munch({\u0027id\u0027: \u0027c4fdc1bcf30541048ca086d7332a57e3\u0027, \u0027name\u0027: \u0027admin\u0027, \u0027domain_id\u0027: \u0027default\u0027, \u0027domain_name\u0027: None})}))\n    \u003e\u003e\u003e conn.get_user(\u0027admin\u0027, filters\u003d{\u0027domain_id\u0027: \u0027default2\u0027})\n    \u003e\u003e\u003e quit()\n\nCould you explain what exactly is the issue here?\n\n(This is using the current HEAD of master, commit 2167d7e29f779e05d1f65d268264ddd65443462e)","commit_id":"bccca3c2aa52886fcdb370b0e276e7153fec2ba5"},{"author":{"_account_id":34379,"name":"Francesco Lombardo","email":"francesco.lombardo@garr.it","username":"zvfvrv"},"change_message_id":"3ee7d712451cc56f0f5910a6e251dc3923a95f7d","unresolved":true,"context_lines":[{"line_number":1185,"context_line":""},{"line_number":1186,"context_line":"        if user:"},{"line_number":1187,"context_line":"            # use cloud.get_user to save us from bad searching by name"},{"line_number":1188,"context_line":"            data[\u0027user\u0027] \u003d self.get_user(user, **search_args)"},{"line_number":1189,"context_line":"        if group:"},{"line_number":1190,"context_line":"            data[\u0027group\u0027] \u003d self.identity.find_group("},{"line_number":1191,"context_line":"                group, ignore_missing\u003dFalse, **search_args)"}],"source_content_type":"text/x-python","patch_set":2,"id":"945cf2f6_927ff58f","line":1188,"range":{"start_line":1188,"start_character":60,"end_line":1188,"end_character":61},"in_reply_to":"0cd1f0cc_481e16f9","updated":"2022-11-25 17:11:49.000000000","message":"Hi, \nI made this change because in my environment I get different output than yours:\n\n\u003e\u003e\u003e conn.get_user(\u0027localmember\u0027, filters\u003d{\u0027domain_id\u0027:\u00272b932823d0dc46799acbfabd18b45ee4\u0027})\n\u003e\u003e\u003e conn.get_user(\u0027localmember\u0027, domain_id\u003d\u00272b932823d0dc46799acbfabd18b45ee4\u0027)\nopenstack.identity.v3.user.User(description\u003dAccount di servizio OpenStack, id\u003d7fe965f925bd4d4e9fe9d602b28a62b7, name\u003dlocalmember, domain_id\u003d2b932823d0dc46799acbfabd18b45ee4, enabled\u003dTrue, password_expires_at\u003dNone, links\u003d{\u0027self\u0027: \u0027https://****************.it:5000/v3/users/7fe965f925bd4d4e9fe9d602b28a62b7\u0027}, location\u003dMunch({\u0027cloud\u0027: \u0027envvars\u0027, \u0027region_name\u0027: \u0027\u0027, \u0027zone\u0027: None, \u0027project\u0027: Munch({\u0027id\u0027: \u0027b28a965069544903bf9af16f4dcb7334\u0027, \u0027name\u0027: \u0027admin\u0027, \u0027domain_id\u0027: None, \u0027domain_name\u0027: \u0027admin_domain\u0027})}))\n\nif it works fine in your environment, I\u0027m assuming the problem is elsewhere at this point","commit_id":"bccca3c2aa52886fcdb370b0e276e7153fec2ba5"}]}
