Skip to content

add lightning model status check endpoint - #172

Merged
AngeloB-AIS merged 1 commit into
mainfrom
add-lightning-model-status-check-endpoint
Oct 2, 2026
Merged

AngeloB-AIS merged 1 commit into
mainfrom
add-lightning-model-status-check-endpoint

Conversation

@AngeloB-AIS

@AngeloB-AIS AngeloB-AIS commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • New Features
    • Added ingress routing for the Lightning model status check endpoint.
  • Updates
    • Updated the Multiwoven chart and application versions to 0.114.0.

@coderabbitai

coderabbitai Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Essentials

Run ID: 1beb2766-4f42-4f9f-83c2-ae9caae4fe9c

📥 Commits

Reviewing files that changed from the base of the PR and between 775f844 and a435b02.

📒 Files selected for processing (3)
  • .gitignore
  • charts/multiwoven/Chart.yaml
  • charts/multiwoven/templates/multiwoven-ingress.yaml

Included review availability: This review used your included allowance. 2 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.


📝 Walkthrough

Walkthrough

The Multiwoven Helm chart adds a Lightning ingress path for /model_status_check and updates its chart and app versions to 0.114.0. The .gitignore file also adds an ignore rule for aisquared chart documentation.

Changes

Multiwoven chart update

Layer / File(s) Summary
Lightning ingress and chart version
charts/multiwoven/templates/multiwoven-ingress.yaml, charts/multiwoven/Chart.yaml
The Lightning ingress adds a /model_status_check path with Prefix matching and the configured service port. The chart version and appVersion change to 0.114.0.

Chart documentation ignore rule

Layer / File(s) Summary
Chart documentation ignore rule
.gitignore
The existing ignore rule for charts/multiwoven/docs/ remains. A rule for charts/aisquared/docs/ is added.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Feature

Suggested reviewers: hassimnaseef

Merge Risk: ⚪ Minimal · up to a435b

The chart adds the Lightning status-check route and updates its version metadata without an identified behavior regression. It appears ready to merge subject to normal chart validation.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the main change: adding the Lightning model status check endpoint. This matches the ingress update for /model_status_check.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


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

@AngeloB-AIS
AngeloB-AIS merged commit 81981af into main Oct 2, 2026
3 checks passed
AngeloB-AIS added a commit that referenced this pull request Oct 2, 2026
Brings in #172 (lightning /model_status_check ingress route). The route
merged into the renamed charts/aisquared/templates/ingress.yaml. The
modify/delete conflict on charts/multiwoven/Chart.yaml is resolved by
keeping the deletion; main's change there was only the 0.113.0 -> 0.114.0
bump, and charts/aisquared/Chart.yaml is already 0.114.0.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.

1 participant