Repository: knative-sandbox/eventing-kafka-broker
Pull request #2975 (2023-02-21 13:46:00)

[main] Upgrade to latest dependencies
Cron -knative-prow-robot

/cc knative-sandbox/eventing-writers
/assign knative-sandbox/eventing-writers

Produced by: knative-sandbox/knobots/actions/update-deps

knative-prow[bot]: @knative-automation: GitHub didn't allow me to request PR reviews from the following users: knative-sandbox/eventing-writers.

Note that only [knative-sandbox members](https://github.com/orgs/knative-sandbox/people) and repo collaborators can review this PR, and authors cannot review their own PRs.

<details>

In response to [this](https://github.com/knative-sandbox/eventing-kafka-broker/pull/2975):

>Cron -knative-prow-robot
>
>/cc knative-sandbox/eventing-writers
>/assign knative-sandbox/eventing-writers
>
>Produced by: knative-sandbox/knobots/actions/update-deps


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.
</details>
knative-prow[bot]: [APPROVALNOTIFIER] This PR is **NOT APPROVED**

This pull-request has been approved by: *<a href="https://github.com/knative-sandbox/eventing-kafka-broker/pull/2975#" title="Author self-approved">knative-automation</a>*
**Once this PR has been reviewed and has the lgtm label**, please assign lionelvillard for approval. For more information see [the Kubernetes Code Review Process](https://git.k8s.io/community/contributors/guide/owners.md#the-code-review-process).

The full list of commands accepted by this bot can be found [here](https://go.k8s.io/bot-commands?repo=knative-sandbox%2Feventing-kafka-broker).

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

- **[OWNERS](https://github.com/knative-sandbox/eventing-kafka-broker/blob/main/OWNERS)**

Approvers can indicate their approval by writing `/approve` in a comment
Approvers can cancel approval by writing `/approve cancel` in a comment
</details>
<!-- META={"approvers":["lionelvillard"]} -->
knative-prow[bot]: @knative-automation: The following test **failed**, say `/retest` to rerun all failed tests or `/retest-required` to rerun all mandatory failed tests:

Test name | Commit | Details | Required | Rerun command
--- | --- | --- | --- | ---
upgrade-tests_eventing-kafka-broker_main | f3bf83b93296e54429fa1da530f367d89f31e4b5 | [link](https://prow.knative.dev/view/gs/knative-prow/pr-logs/pull/knative-sandbox_eventing-kafka-broker/2975/upgrade-tests_eventing-kafka-broker_main/1628028276293242880) | true | `/test upgrade-tests`

[Your PR dashboard](https://prow.knative.dev/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 -->
knative-prow[bot]: [APPROVALNOTIFIER] This PR is **APPROVED**

This pull-request has been approved by: *<a href="https://github.com/knative-sandbox/eventing-kafka-broker/pull/2975#" title="Author self-approved">knative-automation</a>*, *<a href="https://github.com/knative-sandbox/eventing-kafka-broker/pull/2975#pullrequestreview-1307527097" title="Approved">pierDipi</a>*

The full list of commands accepted by this bot can be found [here](https://go.k8s.io/bot-commands?repo=knative-sandbox%2Feventing-kafka-broker).

The pull request process is described [here](https://git.k8s.io/community/contributors/guide/owners.md#the-code-review-process)

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

- ~~[OWNERS](https://github.com/knative-sandbox/eventing-kafka-broker/blob/main/OWNERS)~~ [pierDipi]

Approvers can indicate their approval by writing `/approve` in a comment
Approvers can cancel approval by writing `/approve cancel` in a comment
</details>
<!-- META={"approvers":[]} -->

Patch:
diff --git a/go.mod b/go.mod
index 88e596b9f..e91cd699e 100644
--- a/go.mod
+++ b/go.mod
@@ -42,7 +42,7 @@ require (
 	knative.dev/eventing v0.36.5
 	knative.dev/hack v0.0.0-20230217131352-f2f3107be156
 	knative.dev/pkg v0.0.0-20230220111620-f306c139ba1c
-	knative.dev/reconciler-test v0.0.0-20230216102253-e707ddfb3379
+	knative.dev/reconciler-test v0.0.0-20230221093927-d6796614f8e8
 	sigs.k8s.io/controller-runtime v0.12.3
 )
 
diff --git a/go.sum b/go.sum
index c174eecc1..68a501676 100644
--- a/go.sum
+++ b/go.sum
@@ -1520,8 +1520,8 @@ knative.dev/hack v0.0.0-20230217131352-f2f3107be156 h1:l4/KJeKC7Nrm9WUF3AhAnvoj0
 knative.dev/hack v0.0.0-20230217131352-f2f3107be156/go.mod h1:yk2OjGDsbEnQjfxdm0/HJKS2WqTLEFg/N6nUs6Rqx3Q=
 knative.dev/pkg v0.0.0-20230220111620-f306c139ba1c h1:OHZp+0ih0gun1S8QoSedIs7aW/GjD9qbRYB4QgLciC0=
 knative.dev/pkg v0.0.0-20230220111620-f306c139ba1c/go.mod h1:FLoALs9HfjVPT7sjrpvNLqO+mx+RzJZwUJ2b0UsLLus=
-knative.dev/reconciler-test v0.0.0-20230216102253-e707ddfb3379 h1:JaBb6JAxpcE7rgPtONvnyR1AYp/21FtYFh/89iwE2cY=
-knative.dev/reconciler-test v0.0.0-20230216102253-e707ddfb3379/go.mod h1:yk6YxV6oxVfD+ETK1NqAUvDfeTn/XbXoJ33LCNQwny8=
+knative.dev/reconciler-test v0.0.0-20230221093927-d6796614f8e8 h1:ouBap6mT+XP4S0mAZISe2Cu1GlMH/iIfc+yQZCyYqRc=
+knative.dev/reconciler-test v0.0.0-20230221093927-d6796614f8e8/go.mod h1:yk6YxV6oxVfD+ETK1NqAUvDfeTn/XbXoJ33LCNQwny8=
 pgregory.net/rapid v0.3.3 h1:jCjBsY4ln4Atz78QoBWxUEvAHaFyNDQg9+WU62aCn1U=
 pgregory.net/rapid v0.3.3/go.mod h1:UYpPVyjFHzYBGHIxLFoupi8vwk6rXNzRY9OMvVxFIOU=
 rsc.io/binaryregexp v0.2.0/go.mod h1:qTv7/COck+e2FymRvadv62gMdZztPaShugOCi3I+8D8=
diff --git a/vendor/modules.txt b/vendor/modules.txt
index ef81523bb..ca5390552 100644
--- a/vendor/modules.txt
+++ b/vendor/modules.txt
@@ -1484,7 +1484,7 @@ knative.dev/pkg/webhook/json
 knative.dev/pkg/webhook/resourcesemantics
 knative.dev/pkg/webhook/resourcesemantics/defaulting
 knative.dev/pkg/webhook/resourcesemantics/validation
-# knative.dev/reconciler-test v0.0.0-20230216102253-e707ddfb3379
+# knative.dev/reconciler-test v0.0.0-20230221093927-d6796614f8e8
 ## explicit; go 1.18
 knative.dev/reconciler-test/cmd/eventshub
 knative.dev/reconciler-test/pkg/environment


Test patch:
diff --git a/vendor/knative.dev/reconciler-test/pkg/eventshub/103-pod.yaml b/vendor/knative.dev/reconciler-test/pkg/eventshub/103-pod.yaml
index e683cc0ca..b49e07681 100644
--- a/vendor/knative.dev/reconciler-test/pkg/eventshub/103-pod.yaml
+++ b/vendor/knative.dev/reconciler-test/pkg/eventshub/103-pod.yaml
@@ -27,7 +27,7 @@ metadata:
   {{ end }}
 spec:
   serviceAccountName: "{{ .namespace }}"
-  restartPolicy: "Never"
+  restartPolicy: "OnFailure"
   {{ if .podSecurityContext }}
   securityContext:
     runAsNonRoot: {{ .podSecurityContext.runAsNonRoot }}
diff --git a/vendor/knative.dev/reconciler-test/pkg/k8s/wait.go b/vendor/knative.dev/reconciler-test/pkg/k8s/wait.go
index 94249a68b..fd072a980 100644
--- a/vendor/knative.dev/reconciler-test/pkg/k8s/wait.go
+++ b/vendor/knative.dev/reconciler-test/pkg/k8s/wait.go
@@ -325,6 +325,8 @@ func WaitForServiceReady(ctx context.Context, t feature.T, name string, readines
 	curl := fmt.Sprintf("curl --max-time 2 "+
 		"--trace-ascii %% --trace-time "+
 		"--retry 6 --retry-connrefused %s", sinkURI)
+	maybeQuitIstio := fmt.Sprintf("(curl -fsI -X POST http://localhost:15020/quitquitquit || echo no-istio)")
+	curl = fmt.Sprintf("%s && %s", curl, maybeQuitIstio)
 	var one int32 = 1
 	job := &batchv1.Job{
 		ObjectMeta: metav1.ObjectMeta{Name: jobName, Namespace: ns},
@@ -343,6 +345,14 @@ func WaitForServiceReady(ctx context.Context, t feature.T, name string, readines
 			},
 		},
 	}
+
+	if cfg := environment.GetIstioConfig(ctx); cfg.Enabled {
+		job.Spec.Template.Annotations = map[string]string{
+			"sidecar.istio.io/inject":                "true",
+			"sidecar.istio.io/rewriteAppHTTPProbers": "true",
+		}
+	}
+
 	created, err := jobs.Create(ctx, job, metav1.CreateOptions{})
 	if err != nil {
 		return fmt.Errorf("%w: %v", ErrWaitingForServiceReady, err)