Skip to content

Conversation

@jakubno
Copy link
Member

@jakubno jakubno commented Nov 27, 2025

Note

Store builder node CPU info in env_builds and plumb machine info from orchestrator/edge to handlers while TemplateManager.GetAvailableBuildClient now returns the builder instance.

  • Database:
    • Add CPU machine info columns to public.env_builds (cpu_architecture, cpu_family, cpu_model, cpu_model_name, cpu_flags).
    • Update queries and generated models to read/write the new fields across build/snapshot/template queries.
  • API/Handlers:
    • On build start (v1/v2), fetch builder node instance and persist its machine info into env_builds; use builderNode.NodeID where needed.
    • Layer file upload updated to accept builder node instance.
  • Edge/Orchestrator:
    • Add machineInfo to edge.ClusterInstance with GetMachineInfo; populate during sync and on pool insert.
    • Introduce shared machineinfo package and replace local types in node manager.
    • Change TemplateManager.GetAvailableBuildClient to return *edge.ClusterInstance; update delete fallback and callers accordingly.

Written by Cursor Bugbot for commit 8a2bbb4. This will update automatically on new commits. Configure here.

@jakubno jakubno added the improvement Improvement for current functionality label Nov 27, 2025
@linear
Copy link

linear bot commented Nov 27, 2025

@jakubno jakubno marked this pull request as ready for review December 1, 2025 14:56
cursor[bot]

This comment was marked as resolved.

@jakubno jakubno force-pushed the save-information-about-builder-node-in-db-eng-3337 branch from 3fb2ebb to c98205d Compare December 1, 2025 17:37
@jakubno jakubno enabled auto-merge (squash) December 2, 2025 09:23
@jakubno jakubno merged commit 22a36ba into main Dec 2, 2025
27 checks passed
@jakubno jakubno deleted the save-information-about-builder-node-in-db-eng-3337 branch December 2, 2025 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improvement for current functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants