)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"c233940635371efe14bb91b645ac03713e4e6289","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"5894f2b8_bb1598e1","updated":"2022-04-28 10:07:03.000000000","message":"Nicely done 👍","commit_id":"d1da77c7e526034ef3c32d22cf69ce049c070886"},{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"f1408f23a0144c308989624fef08093910b0ddc1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1928f76f_7b56c7ee","updated":"2022-04-28 13:51:09.000000000","message":"Please rebase and rename tag in tox_extra_args in .zuul.yaml, so that your tests are actually run.","commit_id":"d1da77c7e526034ef3c32d22cf69ce049c070886"},{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"ca764f6c7911f019ffbb760f497fce376acdf283","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"a2b0e60f_c5429072","updated":"2022-05-02 07:56:32.000000000","message":"Zuul CI wont run this role because check jobs will take the job definition for the base job from latest master branch commit. Hence it will ignore the change to tox_extra_args which are applied in this patch 😭 Thinking about a solution...","commit_id":"fd23ae8343425b7988a29e10caa494bfb25de1a6"},{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"0acbef5f618c0de6df0f29f08bb22996c944ac4d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"c28c1514_68569832","in_reply_to":"4b6e3d00_8adff649","updated":"2022-05-03 11:00:32.000000000","message":"Locally your patch worked fine, lets see what happens when i merge it ^^","commit_id":"fd23ae8343425b7988a29e10caa494bfb25de1a6"},{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"14ca85c04f6300a3cf4ca82a7ce731e30fec30da","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"4b6e3d00_8adff649","in_reply_to":"a2b0e60f_c5429072","updated":"2022-05-02 08:21:48.000000000","message":"No wait, tox_extra_args is not set in base job. Something else is going on. Why is Zuul using keystone_role instead of identity_role in jobs?","commit_id":"fd23ae8343425b7988a29e10caa494bfb25de1a6"}],"ci/roles/keystone_role/defaults/main.yml":[{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"e0b8d36b16789524c5113b011913dbc48041acf0","unresolved":true,"context_lines":[{"line_number":4,"context_line":"  - domain_id"},{"line_number":5,"context_line":"  - id"},{"line_number":6,"context_line":"  - links"},{"line_number":7,"context_line":"  - location"},{"line_number":8,"context_line":"  - name"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"a6b3ed14_0e5d256c","line":7,"updated":"2022-04-27 08:25:07.000000000","message":"location is not part of the fields because of computed\u003dFalse. For me, valid entries were:\n\n  [\u0027description\u0027, \u0027domain_id\u0027, \u0027id\u0027, \u0027links\u0027, \u0027name\u0027]","commit_id":"f744145147f1bec077c24ef265d33bdf552b2078"},{"author":{"_account_id":34208,"name":"Rafael Castillo","email":"rcastill@redhat.com","username":"rcastill"},"change_message_id":"6da08cccd7d9d5d023336cf46a1d0a55e969a006","unresolved":false,"context_lines":[{"line_number":4,"context_line":"  - domain_id"},{"line_number":5,"context_line":"  - id"},{"line_number":6,"context_line":"  - links"},{"line_number":7,"context_line":"  - location"},{"line_number":8,"context_line":"  - name"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"59b1277e_60a4eab2","line":7,"in_reply_to":"a6b3ed14_0e5d256c","updated":"2022-04-27 15:37:29.000000000","message":"Done","commit_id":"f744145147f1bec077c24ef265d33bdf552b2078"}],"plugins/modules/identity_role.py":[{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"e0b8d36b16789524c5113b011913dbc48041acf0","unresolved":true,"context_lines":[{"line_number":48,"context_line":"    description: Dictionary describing the role."},{"line_number":49,"context_line":"    returned: On success when I(state) is \u0027present\u0027."},{"line_number":50,"context_line":"    type: complex"},{"line_number":51,"context_line":"    contains:"},{"line_number":52,"context_line":"        id:"},{"line_number":53,"context_line":"            description: Unique role ID."},{"line_number":54,"context_line":"            type: str"}],"source_content_type":"text/x-python","patch_set":2,"id":"1ef67041_46e4ad5a","line":51,"updated":"2022-04-27 08:25:07.000000000","message":"Please add other values as well, i.e. description, domain_id and links.","commit_id":"f744145147f1bec077c24ef265d33bdf552b2078"},{"author":{"_account_id":34208,"name":"Rafael Castillo","email":"rcastill@redhat.com","username":"rcastill"},"change_message_id":"6da08cccd7d9d5d023336cf46a1d0a55e969a006","unresolved":false,"context_lines":[{"line_number":48,"context_line":"    description: Dictionary describing the role."},{"line_number":49,"context_line":"    returned: On success when I(state) is \u0027present\u0027."},{"line_number":50,"context_line":"    type: complex"},{"line_number":51,"context_line":"    contains:"},{"line_number":52,"context_line":"        id:"},{"line_number":53,"context_line":"            description: Unique role ID."},{"line_number":54,"context_line":"            type: str"}],"source_content_type":"text/x-python","patch_set":2,"id":"f6cfc984_9cadbc15","line":51,"in_reply_to":"1ef67041_46e4ad5a","updated":"2022-04-27 15:37:29.000000000","message":"Done","commit_id":"f744145147f1bec077c24ef265d33bdf552b2078"},{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"c233940635371efe14bb91b645ac03713e4e6289","unresolved":true,"context_lines":[{"line_number":108,"context_line":"            role \u003d role.to_dict(computed\u003dFalse)"},{"line_number":109,"context_line":"            self.exit_json(changed\u003dchanged, role\u003drole)"},{"line_number":110,"context_line":"        elif state \u003d\u003d \u0027absent\u0027 and role is not None:"},{"line_number":111,"context_line":"            self.conn.delete_role(name)"},{"line_number":112,"context_line":"            changed \u003d True"},{"line_number":113,"context_line":"        self.exit_json(changed\u003dchanged)"},{"line_number":114,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"29b840cf_03f0ad70","line":111,"updated":"2022-04-28 10:07:03.000000000","message":"Please use\n  self.conn.identity.delete_role(role)\nor\n  self.conn.identity.delete_role(role[\u0027id\u0027])\nbecause we already have the role. Or maybe you had a good reason for sticking to cloud layer?","commit_id":"d1da77c7e526034ef3c32d22cf69ce049c070886"},{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"14ca85c04f6300a3cf4ca82a7ce731e30fec30da","unresolved":false,"context_lines":[{"line_number":108,"context_line":"            role \u003d role.to_dict(computed\u003dFalse)"},{"line_number":109,"context_line":"            self.exit_json(changed\u003dchanged, role\u003drole)"},{"line_number":110,"context_line":"        elif state \u003d\u003d \u0027absent\u0027 and role is not None:"},{"line_number":111,"context_line":"            self.conn.delete_role(name)"},{"line_number":112,"context_line":"            changed \u003d True"},{"line_number":113,"context_line":"        self.exit_json(changed\u003dchanged)"},{"line_number":114,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"711a9227_7230c61c","line":111,"in_reply_to":"29b840cf_03f0ad70","updated":"2022-05-02 08:21:48.000000000","message":"Done","commit_id":"d1da77c7e526034ef3c32d22cf69ce049c070886"},{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"faf858fb954ba40ab5fc583292465298813e1725","unresolved":false,"context_lines":[{"line_number":47,"context_line":"role:"},{"line_number":48,"context_line":"    description: Dictionary describing the role."},{"line_number":49,"context_line":"    returned: On success when I(state) is \u0027present\u0027."},{"line_number":50,"context_line":"    type: list"},{"line_number":51,"context_line":"    elements: dict"},{"line_number":52,"context_line":"    contains:"},{"line_number":53,"context_line":"        description:"}],"source_content_type":"text/x-python","patch_set":6,"id":"ef4e2d8b_959572c2","line":50,"updated":"2022-05-04 08:43:19.000000000","message":"Its not a list but a dict. Fixed in follow up patch here: https://review.opendev.org/c/openstack/ansible-collections-openstack/+/840423","commit_id":"cc1b5ecae8563b28960cbd1fb706ca09ca701162"}]}