events.action: opened
events.author: hougangliu
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://git.k8s.io/community/contributors/guide/pull-requests.md#the-pull-request-submit-process and developer guide https://git.k8s.io/community/contributors/devel/development.md#development-guide
  2. If you want *faster* PR reviews, read how: https://git.k8s.io/community/contributors/guide/pull-requests.md#best-practices-for-faster-reviews
  3. Follow the instructions for writing a release note: https://git.k8s.io/community/contributors/guide/release-notes.md
  4. If the PR is unfinished, see how to mark it: https://git.k8s.io/community/contributors/guide/pull-requests.md#marking-unfinished-pull-requests
  5. If this PR changes image versions, please title this PR "Bump <image name> from x.x.x to y.y.y."
  -->
  
  **What this PR does / why we need it**:
  
  **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 #
  
  **Special notes for your reviewer**:
  
  1. Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.
  
  **Release note**:
  <!--  Write your release note:
  1. Enter your extended release note in the below block. If the PR requires additional action from users switching to the new release, include the string "action required".
  2. If no release note is required, just write "NONE".
  -->
  ```release-note
  
  ```
events.title: Change web failPolicy to fail instead of default ingore
events.type: issue
events.action: created
events.author: hougangliu
events.text: Based on https://kubernetes.io/docs/reference/access-authn-authz/extensible-admission-controllers/#failure-policy, katib webhook failure policy should be `Fail` instead of default `Ignore` value
  /cc @gaocegege @johnugeorge
events.type: comment
events.action: created
events.author: johnugeorge
events.text: /retest
events.type: comment
events.action: created
events.author: hougangliu
events.text: /test kubeflow-katib-presubmit
events.type: comment
events.action: created
events.author: johnugeorge
events.text: /retest
events.type: comment
data_with_bots.bot: @hougangliu: The following test **failed**, say `/retest` to rerun them all:
  
  Test name | Commit | Details | Rerun command
  --- | --- | --- | ---
  kubeflow-katib-presubmit | 8b5a605792c6a1342af75122a7fec609c034328a | [link](https://prow.k8s.io/view/gcs/kubernetes-jenkins/pr-logs/pull/kubeflow_katib/933/kubeflow-katib-presubmit/1198895129566384128/) | `/test kubeflow-katib-presubmit`
  
  [Full PR test history](https://prow.k8s.io/pr-history?org=kubeflow&repo=katib&pr=933). [Your PR dashboard](https://gubernator.k8s.io/pr/hougangliu). Please help us cut down on flakes by [linking to](https://git.k8s.io/community/contributors/devel/sig-testing/flaky-tests.md#filing-issues-for-flaky-tests) an [open issue](https://github.com/kubeflow/katib/issues?q=is:issue+is:open) when you hit one in your PR.
  
  <details>
  
  Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
  </details>
  <!-- test report -->
data_with_bots.meta: This message directly preceded comment from bot *k8s-ci-robot* by author *johnugeorge*:
data_with_bots.text: /retest
data_with_bots.bot: @hougangliu: The following test **failed**, say `/retest` to rerun them all:
  
  Test name | Commit | Details | Rerun command
  --- | --- | --- | ---
  kubeflow-katib-presubmit | 8b5a605792c6a1342af75122a7fec609c034328a | [link](https://prow.k8s.io/view/gcs/kubernetes-jenkins/pr-logs/pull/kubeflow_katib/933/kubeflow-katib-presubmit/1198919793076342784/) | `/test kubeflow-katib-presubmit`
  
  [Full PR test history](https://prow.k8s.io/pr-history?org=kubeflow&repo=katib&pr=933). [Your PR dashboard](https://gubernator.k8s.io/pr/hougangliu). Please help us cut down on flakes by [linking to](https://git.k8s.io/community/contributors/devel/sig-testing/flaky-tests.md#filing-issues-for-flaky-tests) an [open issue](https://github.com/kubeflow/katib/issues?q=is:issue+is:open) when you hit one in your PR.
  
  <details>
  
  Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
  </details>
  <!-- test report -->
data_with_bots.meta: This message directly preceded comment from bot *k8s-ci-robot* by author *hougangliu*:
data_with_bots.text: /test kubeflow-katib-presubmit
data_without_bots.meta: This message directly preceded comment from bot *k8s-ci-robot* by author *johnugeorge*:
data_without_bots.text: /retest
data_without_bots.meta: This message directly preceded comment from bot *k8s-ci-robot* by author *hougangliu*:
data_without_bots.text: /test kubeflow-katib-presubmit