events.action: opened
events.author: zhangxiaoyu-zidif
events.text: <!--  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
  ```
events.title: Return nil when err is nil
events.type: issue
events.action: created
events.author: cblecker
events.text: This is effectively a noop. Why is this needed?
events.type: comment
events.action: created
events.author: zhangxiaoyu-zidif
events.text: @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.
events.type: comment
events.action: created
events.author: cblecker
events.text: @k8s-bot ok to test
events.type: comment
events.action: created
events.author: cblecker
events.text: /lgtm
events.type: comment
events.action: created
events.author: wojtek-t
events.text: /approve
events.type: comment
data_with_bots.bot: [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"]} -->
data_with_bots.meta: This message directly preceded comment from bot *k8s-merge-robot* by author *cblecker*:
data_with_bots.text: /lgtm
data_with_bots.bot: [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":[]} -->
data_with_bots.meta: This message directly preceded comment from bot *k8s-merge-robot* by author *wojtek-t*:
data_with_bots.text: /approve
data_without_bots.meta: This message directly preceded comment from bot *k8s-merge-robot* by author *cblecker*:
data_without_bots.text: /lgtm
data_without_bots.meta: This message directly preceded comment from bot *k8s-merge-robot* by author *wojtek-t*:
data_without_bots.text: /approve