Skip to content

No sidecar for domain-dispatcher job#3708

Closed
sleepycat wants to merge 1 commit into
masterfrom
dont-inject-dispatcher-job
Closed

No sidecar for domain-dispatcher job#3708
sleepycat wants to merge 1 commit into
masterfrom
dont-inject-dispatcher-job

Conversation

@sleepycat

Copy link
Copy Markdown
Contributor

This commit adds an injection to the domain-dispatcher job that prevents the
istio sidecar from being injected.

With sidecars injected, the jobs are not properly marked as completed, even
though they are.

$ kubectl get jobs -n scanners domain-dispatcher-27551520
NAME                         COMPLETIONS   DURATION   AGE
domain-dispatcher-27551520   0/1           20h        20h

This commit adds an injection to the domain-dispatcher job that prevents the
istio sidecar from being injected.

With sidecars injected, the jobs are not properly marked as completed, even
though they are.
```bash
$ kubectl get jobs -n scanners domain-dispatcher-27551520
NAME                         COMPLETIONS   DURATION   AGE
domain-dispatcher-27551520   0/1           20h        20h
```
@FestiveKyle
FestiveKyle self-requested a review May 26, 2022 17:39
@sleepycat

Copy link
Copy Markdown
Contributor Author

Closed in favour of #3741

@sleepycat sleepycat closed this May 26, 2022
@lcampbell2
lcampbell2 deleted the dont-inject-dispatcher-job branch May 26, 2023 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants