Skip to content

🐛 fix: resolve Build and Deploy KC workflow failure#18015

Closed
kubestellar-hive[bot] wants to merge 1 commit into
mainfrom
scanner/fix-18009
Closed

🐛 fix: resolve Build and Deploy KC workflow failure#18015
kubestellar-hive[bot] wants to merge 1 commit into
mainfrom
scanner/fix-18009

Conversation

@kubestellar-hive

Copy link
Copy Markdown
Contributor

Fixes #18009

Summary

Fixed compilation errors in the Build and Deploy KC workflow caused by undefined symbols in the gitops package.

Changes

  • Created pkg/api/handlers/gitops/helpers.go with missing helper functions:
    • isDemoMode() - checks demo mode header
    • errNoClusterAccess() - returns cluster access error
    • handleK8sError() - handles K8s errors
    • writeSSEEvent() - writes SSE events
    • streamDemoSSE() - streams demo SSE data
  • Added maxK8sNameLen constant (253) to pkg/api/handlers/validation.go

Root Cause

The operators.go file in the gitops package was calling functions that were not available in its package scope. Following the established pattern from other sub-packages (workloads, missions, benchmarks), each sub-package maintains its own copy of common helper functions.

Testing

CI build will validate the fix.

Add missing helper functions to gitops package and maxK8sNameLen constant
to validation.go to fix undefined symbol compilation errors.

- Created pkg/api/handlers/gitops/helpers.go with isDemoMode, errNoClusterAccess,
  handleK8sError, writeSSEEvent, and streamDemoSSE functions
- Added maxK8sNameLen constant (253) to pkg/api/handlers/validation.go

These functions were being called by operators.go but were not available in
the gitops package. Following the established pattern from other sub-packages
(workloads, missions, benchmarks), each sub-package maintains its own copy
of common helper functions.

Fixes #18009

Signed-off-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Scanner Bot <scanner@kubestellar.io>
Copilot AI review requested due to automatic review settings June 12, 2026 10:07

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@kubestellar-prow kubestellar-prow Bot added the dco-signoff: yes Indicates the PR's author has signed the DCO. label Jun 12, 2026
@kubestellar-prow

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign mikespreitzer for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@netlify

netlify Bot commented Jun 12, 2026

Copy link
Copy Markdown

Deploy Preview for kubestellarconsole canceled.

Name Link
🔨 Latest commit 1f50ddc
🔍 Latest deploy log https://app.netlify.com/projects/kubestellarconsole/deploys/6a2bda6483c8d7000774197f

@github-actions github-actions Bot added the ai-generated Pull request generated by AI label Jun 12, 2026
@github-actions

Copy link
Copy Markdown
Contributor

👋 Hey @kubestellar-hive[bot] — thanks for opening this PR!

🤖 This project is developed exclusively using AI coding assistants.

Please do not attempt to code anything for this project manually.
All contributions should be authored using an AI coding tool such as:

This ensures consistency in code style, architecture patterns, test coverage,
and commit quality across the entire codebase.


This is an automated message.

@kubestellar-prow kubestellar-prow Bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jun 12, 2026
@kubestellar-hive

Copy link
Copy Markdown
Contributor Author

Closing: merge conflicts. Will open a fresh fix.

@kubestellar-hive kubestellar-hive Bot closed this Jun 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-generated Pull request generated by AI dco-signoff: yes Indicates the PR's author has signed the DCO. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. tier/2-standard

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Workflow failure: Build and Deploy KC

1 participant