Skip to content

fix(opencost): patch ui probe timeouts - #2555

Merged
devantler merged 1 commit into
mainfrom
claude/opencost-ui-probe-timeouts-e88c
Jul 10, 2026
Merged

devantler merged 1 commit into
mainfrom
claude/opencost-ui-probe-timeouts-e88c

Conversation

@devantler

Copy link
Copy Markdown
Contributor

🤖 Generated by the Daily AI Assistant

Summary

Validation

  • git diff --check
  • python3 scripts/validate-embedded-json.py
  • kubectl kustomize k8s/clusters/local/
  • kubectl kustomize k8s/clusters/prod/
  • ksail workload validate and ksail --config ksail.prod.yaml workload validate were also run against this same patch before chore(deps): update helm release opencost to v2.5.26 #2554 merged; both reached bases/infrastructure/opencost validated, then failed on the known CloudNativePG/kubeconform render-stream corruption tracked by platform#2273 / ksail#5362.

@coderabbitai

coderabbitai Bot commented Jul 10, 2026 •

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The OpenCost HelmRelease post-renderer now targets the opencost-ui container and sets its liveness and readiness probe timeoutSeconds values to 5.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly matches the main change: patching OpenCost UI probe timeouts.
Description check ✅ Passed The description directly summarizes the probe timeout patch and related validation work.

Comment @coderabbitai help to get the list of available commands.

@devantler
devantler marked this pull request as ready for review July 10, 2026 06:14
@devantler
devantler added this pull request to the merge queue Jul 10, 2026
Merged via the queue into main with commit e9c0d5d Jul 10, 2026
16 checks passed
@devantler
devantler deleted the claude/opencost-ui-probe-timeouts-e88c branch July 10, 2026 06:18
@github-project-automation github-project-automation Bot moved this from 🫴 Ready to ✅ Done in 🌊 Project Board Jul 10, 2026
@botantler-1

botantler-1 Bot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.109.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

1 participant