opened
zhangxiaoyu-zidif
<!--  Thanks for sending a pull request!  Here are some tips for you:
1. If this is your first time, read our contributor guidelines https://github.com/kubernetes/community/blob/master/contributors/devel/pull-requests.md#the-pr-submit-process and developer guide https://github.com/kubernetes/community/blob/master/contributors/devel/development.md#development-guide
2. If you want *faster* PR reviews, read how: https://github.com/kubernetes/community/blob/master/contributors/devel/pull-requests.md#best-practices-for-faster-reviews
3. Follow the instructions for writing a release note: https://github.com/kubernetes/community/blob/master/contributors/devel/pull-requests.md#write-release-notes-if-needed
-->

**What this PR does / why we need it**:
Return nil directly when err is nil

**Special notes for your reviewer**:

**Release note**:
<!--  Steps to write your release note:
1. Use the release-note-* labels to set the release note state (if you have access)
2. Enter your extended release note in the below block; leaving it blank means using the PR title as the release note. If no release note is required, just write `NONE`.
-->
```release-note
NONE
```
Return nil when err is nil
issue
created
cblecker
This is effectively a noop. Why is this needed?
comment
created
zhangxiaoyu-zidif
@cblecker  Sir， please refer to https://github.com/kubernetes/kubernetes/pull/29866, https://github.com/kubernetes/kubernetes/pull/45162. That may be a format. Thanks.
comment
created
cblecker
@k8s-bot ok to test
comment
created
cblecker
/lgtm
comment
created
wojtek-t
/approve
comment
[APPROVALNOTIFIER] This PR is **NOT APPROVED**

This pull-request has been approved by: *<a href="https://github.com/kubernetes/kubernetes/pull/46561#issuecomment-304702943" title="LGTM">cblecker</a>*, *<a href="https://github.com/kubernetes/kubernetes/pull/46561#" title="Author self-approved">zhangxiaoyu-zidif</a>*
We suggest the following additional approver: **ixdy**

Assign the PR to them by writing `/assign @ixdy` in a comment when ready.

<details open>
Needs approval from an approver in each of these OWNERS Files:

- **[hack/OWNERS](https://github.com/kubernetes/kubernetes/blob/master/hack/OWNERS)**

You can indicate your approval by writing `/approve` in a comment
You can cancel your approval by writing `/approve cancel` in a comment
</details>
<!-- META={"approvers":["ixdy"]} -->
This message directly preceded comment from bot *k8s-merge-robot* by author *cblecker*:
/lgtm
[APPROVALNOTIFIER] This PR is **APPROVED**

This pull-request has been approved by: *<a href="https://github.com/kubernetes/kubernetes/pull/46561#issuecomment-304702943" title="LGTM">cblecker</a>*, *<a href="https://github.com/kubernetes/kubernetes/pull/46561#issuecomment-304815527" title="Approved">wojtek-t</a>*, *<a href="https://github.com/kubernetes/kubernetes/pull/46561#" title="Author self-approved">zhangxiaoyu-zidif</a>*

<details >
Needs approval from an approver in each of these OWNERS Files:

- ~~[hack/OWNERS](https://github.com/kubernetes/kubernetes/blob/master/hack/OWNERS)~~ [wojtek-t]

You can indicate your approval by writing `/approve` in a comment
You can cancel your approval by writing `/approve cancel` in a comment
</details>
<!-- META={"approvers":[]} -->
This message directly preceded comment from bot *k8s-merge-robot* by author *wojtek-t*:
/approve
This message directly preceded comment from bot *k8s-merge-robot* by author *cblecker*:
/lgtm
This message directly preceded comment from bot *k8s-merge-robot* by author *wojtek-t*:
/approve