)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"9796908aebe2333603f62e3eab214c2c93b375b6","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"TODO:"},{"line_number":20,"context_line":"* unit tests"},{"line_number":21,"context_line":"* functional test for pre-provisioned credentials ??? if i will figure"},{"line_number":22,"context_line":"  out how to do this."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"Change-Id: I4add0d24548163254f0e3051b38391dfe9948c28"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9fdfeff1_87250cae","line":22,"range":{"start_line":21,"start_character":2,"end_line":22,"end_character":21},"updated":"2019-02-05 05:41:44.000000000","message":"Tempest has plenty of jobs that set up the gate for pre-provisioned credentials: https://github.com/openstack/tempest/blob/139b453d00d4dbdd08a57beaea15be7dda682f28/.zuul.yaml#L443\n\nThat flag will execute: https://docs.openstack.org/tempest/latest/account_generator.html\n\nI\u0027ve entertained adding something like this to Patrole before, perhaps as an experimental job to start out, or non-voting, assuming it works out better than I anticipate. I don\u0027t remember why I didn\u0027t get around to it.","commit_id":"0504791e9e7cbc387aa3a017b859345caf0db790"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"29ccd28db166da11670984e170c306ee7b3a8114","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7b660cd0_b714d0cd","updated":"2023-03-11 02:12:08.000000000","message":"no activity for long time, and as this repo is almost inactive I will propose it to retire soon but if you would like to maintain it, ping us on #openstack-qa IRC OFTC channel","commit_id":"0504791e9e7cbc387aa3a017b859345caf0db790"}],"patrole_tempest_plugin/rbac_utils.py":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"dd3b520b783b9efdbf2045a6d93a24fc1cffb220","unresolved":false,"context_lines":[{"line_number":258,"context_line":"        cls._all_needed_role_ids \u003d [role_mapping[role_id]"},{"line_number":259,"context_line":"                                    for role_id in cls.all_needed_roles]"},{"line_number":260,"context_line":""},{"line_number":261,"context_line":"        # Memorizing original user roles on project to restore them if used"},{"line_number":262,"context_line":"        # pre-provisioned credentials"},{"line_number":263,"context_line":"        if CONF.auth.test_accounts_file:"},{"line_number":264,"context_line":"            cls._user_role_ids_on_project \u003d cls._list_user_roles_on_project()"},{"line_number":265,"context_line":""},{"line_number":266,"context_line":"            # Using addClassResourceCleanup to restore user roles on a project"},{"line_number":267,"context_line":"            # allows to clear all resources created in RBAC tests, then restore"},{"line_number":268,"context_line":"            # roles, and then clear all resources that can be created at any"},{"line_number":269,"context_line":"            # earlier stages."},{"line_number":270,"context_line":"            cls.addClassResourceCleanup(cls._reset_roles,"},{"line_number":271,"context_line":"                                        cls._user_role_ids_on_project)"},{"line_number":272,"context_line":""},{"line_number":273,"context_line":"    @classmethod"},{"line_number":274,"context_line":"    def _get_role_inferences_mapping(cls):"}],"source_content_type":"text/x-python","patch_set":3,"id":"9fdfeff1_442082b1","line":271,"range":{"start_line":261,"start_character":0,"end_line":271,"end_character":70},"updated":"2019-02-05 05:35:39.000000000","message":"This appears to be fixing a separate issue than the one described in the commit message. I agree that better support for pre-provisioned credentials should be added but feels like too much too add to one patch set.","commit_id":"0504791e9e7cbc387aa3a017b859345caf0db790"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"4ce0735f0bc4cf91f91e064b89d21e2cf77964f5","unresolved":false,"context_lines":[{"line_number":258,"context_line":"        cls._all_needed_role_ids \u003d [role_mapping[role_id]"},{"line_number":259,"context_line":"                                    for role_id in cls.all_needed_roles]"},{"line_number":260,"context_line":""},{"line_number":261,"context_line":"        # Memorizing original user roles on project to restore them if used"},{"line_number":262,"context_line":"        # pre-provisioned credentials"},{"line_number":263,"context_line":"        if CONF.auth.test_accounts_file:"},{"line_number":264,"context_line":"            cls._user_role_ids_on_project \u003d cls._list_user_roles_on_project()"},{"line_number":265,"context_line":""},{"line_number":266,"context_line":"            # Using addClassResourceCleanup to restore user roles on a project"},{"line_number":267,"context_line":"            # allows to clear all resources created in RBAC tests, then restore"},{"line_number":268,"context_line":"            # roles, and then clear all resources that can be created at any"},{"line_number":269,"context_line":"            # earlier stages."},{"line_number":270,"context_line":"            cls.addClassResourceCleanup(cls._reset_roles,"},{"line_number":271,"context_line":"                                        cls._user_role_ids_on_project)"},{"line_number":272,"context_line":""},{"line_number":273,"context_line":"    @classmethod"},{"line_number":274,"context_line":"    def _get_role_inferences_mapping(cls):"}],"source_content_type":"text/x-python","patch_set":3,"id":"9fdfeff1_870c2c3c","line":271,"range":{"start_line":261,"start_character":0,"end_line":271,"end_character":70},"in_reply_to":"9fdfeff1_442082b1","updated":"2019-02-05 05:36:32.000000000","message":"I meant to say this is fixing more than the first issue described. Anyway, point is this is 2 fixes in 1.","commit_id":"0504791e9e7cbc387aa3a017b859345caf0db790"}]}