Skip to content

BUGFIX: Zxporter cmd fix#302

Draft
mutantkeyboard wants to merge 2 commits intomainfrom
zxporter_cmd_fix
Draft

BUGFIX: Zxporter cmd fix#302
mutantkeyboard wants to merge 2 commits intomainfrom
zxporter_cmd_fix

Conversation

@mutantkeyboard
Copy link
Contributor

@mutantkeyboard mutantkeyboard commented Feb 26, 2026

[Title]

📚 Description of Changes

Removed the double code init from file.

  • 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.

Move HealthManager creation and health server startup to before
ctrl.NewManager() so K8s probes are answered immediately, preventing
connection refused and 503 errors during slow initialization.
@gitar-bot
Copy link

gitar-bot bot commented Feb 26, 2026

Important

Upgrade your plan to unlock code review, CI analysis, custom rules, and more.

@mutantkeyboard mutantkeyboard marked this pull request as draft February 26, 2026 11:09
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