)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8367,"name":"Arx Cruz","email":"arxcruz@redhat.com","username":"arxcruz"},"change_message_id":"a6707ae7d71fad6605cc7a065b0899e799fcd639","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"517df8d0_43fe304f","updated":"2022-08-03 13:50:54.000000000","message":"-W for now because it depends on changes on federation_idp_info to passes since it changes the same set of tasks ","commit_id":"f9f8530d7a41f9b17a7c47d791cbe57015c58086"}],"plugins/modules/federation_idp.py":[{"author":{"_account_id":34208,"name":"Rafael Castillo","email":"rcastill@redhat.com","username":"rcastill"},"change_message_id":"c3b182da41a876416dd56f2e31ec5677964f16ab","unresolved":true,"context_lines":[{"line_number":74,"context_line":"RETURN \u003d \u0027\u0027\u0027"},{"line_number":75,"context_line":"identity_provider:"},{"line_number":76,"context_line":"    description: Dictionary describing the identity providers"},{"line_number":77,"context_line":"    returned: success"},{"line_number":78,"context_line":"    type: list"},{"line_number":79,"context_line":"    elements: dict"},{"line_number":80,"context_line":"    contains:"}],"source_content_type":"text/x-python","patch_set":5,"id":"e678f7e6_cfb4fb4b","line":77,"updated":"2022-08-09 23:55:17.000000000","message":"only when state is present","commit_id":"45bb73a8869baa156bae6f878c9f2d59798cf63f"},{"author":{"_account_id":8367,"name":"Arx Cruz","email":"arxcruz@redhat.com","username":"arxcruz"},"change_message_id":"08fb6e209160bc0632b67c82ae3639eea9c4c6fe","unresolved":false,"context_lines":[{"line_number":74,"context_line":"RETURN \u003d \u0027\u0027\u0027"},{"line_number":75,"context_line":"identity_provider:"},{"line_number":76,"context_line":"    description: Dictionary describing the identity providers"},{"line_number":77,"context_line":"    returned: success"},{"line_number":78,"context_line":"    type: list"},{"line_number":79,"context_line":"    elements: dict"},{"line_number":80,"context_line":"    contains:"}],"source_content_type":"text/x-python","patch_set":5,"id":"e4d710a2_517d12cf","line":77,"in_reply_to":"e678f7e6_cfb4fb4b","updated":"2022-08-16 14:03:21.000000000","message":"Done","commit_id":"45bb73a8869baa156bae6f878c9f2d59798cf63f"},{"author":{"_account_id":34208,"name":"Rafael Castillo","email":"rcastill@redhat.com","username":"rcastill"},"change_message_id":"c3b182da41a876416dd56f2e31ec5677964f16ab","unresolved":true,"context_lines":[{"line_number":75,"context_line":"identity_provider:"},{"line_number":76,"context_line":"    description: Dictionary describing the identity providers"},{"line_number":77,"context_line":"    returned: success"},{"line_number":78,"context_line":"    type: list"},{"line_number":79,"context_line":"    elements: dict"},{"line_number":80,"context_line":"    contains:"},{"line_number":81,"context_line":"        description:"}],"source_content_type":"text/x-python","patch_set":5,"id":"d996683c_8b34085c","line":78,"updated":"2022-08-09 23:55:17.000000000","message":"Not a list, just a single dict","commit_id":"45bb73a8869baa156bae6f878c9f2d59798cf63f"},{"author":{"_account_id":8367,"name":"Arx Cruz","email":"arxcruz@redhat.com","username":"arxcruz"},"change_message_id":"08fb6e209160bc0632b67c82ae3639eea9c4c6fe","unresolved":false,"context_lines":[{"line_number":75,"context_line":"identity_provider:"},{"line_number":76,"context_line":"    description: Dictionary describing the identity providers"},{"line_number":77,"context_line":"    returned: success"},{"line_number":78,"context_line":"    type: list"},{"line_number":79,"context_line":"    elements: dict"},{"line_number":80,"context_line":"    contains:"},{"line_number":81,"context_line":"        description:"}],"source_content_type":"text/x-python","patch_set":5,"id":"b121501c_c78fe948","line":78,"in_reply_to":"d996683c_8b34085c","updated":"2022-08-16 14:03:21.000000000","message":"Done","commit_id":"45bb73a8869baa156bae6f878c9f2d59798cf63f"},{"author":{"_account_id":34208,"name":"Rafael Castillo","email":"rcastill@redhat.com","username":"rcastill"},"change_message_id":"c3b182da41a876416dd56f2e31ec5677964f16ab","unresolved":true,"context_lines":[{"line_number":91,"context_line":"            type: str"},{"line_number":92,"context_line":"            sample: \"test-idp\""},{"line_number":93,"context_line":"        is_enabled:"},{"line_number":94,"context_line":"            description: Indicates wether the identity provider is enabled"},{"line_number":95,"context_line":"            type: bool"},{"line_number":96,"context_line":"        name:"},{"line_number":97,"context_line":"            description: Name of the identity provider, equals its ID."}],"source_content_type":"text/x-python","patch_set":5,"id":"7650f586_9884fff3","line":94,"updated":"2022-08-09 23:55:17.000000000","message":"s/wether/whether","commit_id":"45bb73a8869baa156bae6f878c9f2d59798cf63f"},{"author":{"_account_id":8367,"name":"Arx Cruz","email":"arxcruz@redhat.com","username":"arxcruz"},"change_message_id":"08fb6e209160bc0632b67c82ae3639eea9c4c6fe","unresolved":false,"context_lines":[{"line_number":91,"context_line":"            type: str"},{"line_number":92,"context_line":"            sample: \"test-idp\""},{"line_number":93,"context_line":"        is_enabled:"},{"line_number":94,"context_line":"            description: Indicates wether the identity provider is enabled"},{"line_number":95,"context_line":"            type: bool"},{"line_number":96,"context_line":"        name:"},{"line_number":97,"context_line":"            description: Name of the identity provider, equals its ID."}],"source_content_type":"text/x-python","patch_set":5,"id":"d2aa3558_d69d67fa","line":94,"in_reply_to":"7650f586_9884fff3","updated":"2022-08-16 14:03:21.000000000","message":"Done","commit_id":"45bb73a8869baa156bae6f878c9f2d59798cf63f"},{"author":{"_account_id":34208,"name":"Rafael Castillo","email":"rcastill@redhat.com","username":"rcastill"},"change_message_id":"c3b182da41a876416dd56f2e31ec5677964f16ab","unresolved":false,"context_lines":[{"line_number":217,"context_line":"                    self.fail_json(msg\u003d\u0027A domain_id must be passed when creating\u0027"},{"line_number":218,"context_line":"                                   \u0027 an identity provider\u0027)"},{"line_number":219,"context_line":"                (changed, idp) \u003d self.create_identity_provider(name)"},{"line_number":220,"context_line":"                self.exit_json(changed\u003dchanged, identity_provider\u003didp)"},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"            (changed, new_idp) \u003d self.update_identity_provider(idp)"},{"line_number":223,"context_line":"            self.exit_json(changed\u003dchanged, identity_provider\u003dnew_idp)"}],"source_content_type":"text/x-python","patch_set":5,"id":"eb691345_d9b9c38e","line":220,"updated":"2022-08-09 23:55:17.000000000","message":"Suggestion: we could save some repeated code by putting the calls to to_dict in this outer layer.","commit_id":"45bb73a8869baa156bae6f878c9f2d59798cf63f"}]}