Skip to content

chore: Added additional field to existing proto #289

Merged
mutantkeyboard merged 1 commit into
mainfrom
add_additional_fields
Feb 24, 2026
Merged

chore: Added additional field to existing proto #289
mutantkeyboard merged 1 commit into
mainfrom
add_additional_fields

Conversation

@mutantkeyboard
Copy link
Copy Markdown
Contributor

@mutantkeyboard mutantkeyboard commented Feb 20, 2026

[Title]

📚 Description of Changes

Provide an overview of your changes and why they’re needed. Link to any related issues (e.g., "Fixes #123"). If your PR fixes a bug, resolves a feature request, or updates documentation, please explain how.

  • What Changed:
    (Describe the modifications, additions, or removals.)

  • Why This Change:
    (Explain the problem this PR addresses or the improvement it provides.)

  • Affected Components:
    (Which component does this change affect? - put x for all components)

  • Compose

  • K8s

  • Other (please specify)

❓ Motivation and Context

Why is this change required? What problem does it solve?

  • Context:
    (Provide background information or link to related discussions/issues.)

  • Relevant Tasks/Issues:
    (e.g., Fixes: #GitHub Issue)

🔍 Types of Changes

Indicate which type of changes your code introduces (check all that apply):

  • BUGFIX: Non-breaking fix for an issue.
  • NEW FEATURE: Non-breaking addition of functionality.
  • BREAKING CHANGE: Fix or feature that causes existing functionality to not work as expected.
  • ENHANCEMENT: Improvement to existing functionality.
  • CHORE: Changes that do not affect production (e.g., documentation, build tooling, CI).

🔬 QA / Verification Steps

Describe the steps a reviewer should take to verify your changes:

  1. (Step one: e.g., "Run make test to verify all tests pass.")
  2. (Step two: e.g., "Deploy to a Kind cluster with make create-kind && make deploy.")
  3. (Additional steps as needed.)

✅ Global Checklist

Please check all boxes that apply:

  • I have read and followed the CONTRIBUTING guidelines.
  • My code follows the code style of this project.
  • I have updated the documentation as needed.
  • I have added tests that cover my changes.
  • All new and existing tests have passed locally.
  • I have run this code in a local environment to verify functionality.
  • I have considered the security implications of this change.

Summary by Gitar

  • Enhanced operator health reporting:
    • Added commit field to ReportHealthRequest and OperatorHealthStatus proto messages for tracking operator build version information in health heartbeats

This will update automatically on new commits.

@gitar-bot
Copy link
Copy Markdown

gitar-bot Bot commented Feb 20, 2026

Code Review ✅ Approved

Clean, additive change that plumbs a Git commit hash through the health reporting proto and all call sites. All callers updated, tests cover the new field, and proto field addition is backward-compatible.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@mutantkeyboard mutantkeyboard enabled auto-merge (squash) February 20, 2026 15:59
@mutantkeyboard mutantkeyboard changed the title Added additional field chore: Added additional field to existing proto Feb 20, 2026
@mutantkeyboard mutantkeyboard merged commit 6aadb05 into main Feb 24, 2026
26 checks passed
@mutantkeyboard mutantkeyboard deleted the add_additional_fields branch February 24, 2026 12:10
Parthiba-Hazra pushed a commit that referenced this pull request May 5, 2026
- added the additional commit field to existing proto
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