Skip to content

feat(vector): add startupProbe option to pod template#550

Merged
pront merged 2 commits intovectordotdev:developfrom
afrancis101:feature/add-pod-startup-probe
Apr 6, 2026
Merged

feat(vector): add startupProbe option to pod template#550
pront merged 2 commits intovectordotdev:developfrom
afrancis101:feature/add-pod-startup-probe

Conversation

@afrancis101
Copy link
Copy Markdown
Contributor

Description
Allows Vector to be configured with a startupProbe in the same way that it can be set for livenessProbes and readinessProbes

Testing steps

- With the default values.yaml, the startupProbe settings do not show within the Pod Spec. Can be seen when running `helm template`
- When setting the startupProbe values, the settings show within the Pod Spec when running `helm template`.

@afrancis101 afrancis101 changed the title feat: add startupProbe option to pod template feat(vector): add startupProbe option to pod template Mar 30, 2026
@afrancis101 afrancis101 force-pushed the feature/add-pod-startup-probe branch from 525ad2f to a6287e6 Compare April 3, 2026 11:00
@pront
Copy link
Copy Markdown
Member

pront commented Apr 6, 2026

@codex review

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Breezy!

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@afrancis101
Copy link
Copy Markdown
Contributor Author

@pront have merged in from develop to sort the merge conflict

@pront pront enabled auto-merge (squash) April 6, 2026 20:35
@pront pront merged commit e3ce248 into vectordotdev:develop Apr 6, 2026
15 checks passed
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