Closed hanlins closed 2 years ago
/lgtm /approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Fei-Guo, hanlins
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/retest
What this PR does / why we need it: The current logic for adding owner reference seems to have a problem: if we can cast the error to
AlreadyOwnedError
, then it means the owner reference has already been added, and we should move on. However, currently the logic is if the owner reference is added, it will emit that error.Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged): Fixes #