Description of problem: If the user is in a group and someone tries to approve their membership again the can_apply_for and can_approve checks should s be skipped. Version-Release number of selected component (if applicable): 5.0 How reproducible: ??? Steps to Reproduce: 1. ?? 2. 3. Actual results: Approval is denied because can_apply_for checks if the user is already a member. Expected results: Just ignore the change request if the user is already a member. Additional info: This could be related to inheritance. e.g. If a user applies for A and B and A grants membership to B, if A is processed first then B could possibly fail this way.
A more reliable way for this to trigger is for the user to open two bugs, one with asking for A and another bug asking for B. If the bug asking for A is processed first then the bug asking for B will fail.
Verified on latest QE environment, Result: PASS the comment will appear when user is already a user for that group, and bug can close itself without problem. and API will not open up another workflow bug when that user already in that group.
This change has been staged on partner bugzilla. Feel free to test it there and let us know if you have any issues. https://partner-bugzilla.redhat.com
This change is now live. If there are any issues, do not reopen this bug. Instead, you should create a new bug and reference this bug.