Skip to content

fix: Resolves issues with namespacing if the namespace has to be different than orka-default#83

Open
jasonjanderson wants to merge 1 commit intomacstadium:mainfrom
jasonjanderson:fix-namespacing
Open

fix: Resolves issues with namespacing if the namespace has to be different than orka-default#83
jasonjanderson wants to merge 1 commit intomacstadium:mainfrom
jasonjanderson:fix-namespacing

Conversation

@jasonjanderson
Copy link

@jasonjanderson jasonjanderson commented Feb 24, 2026

Summary

  • WaitForImage was hardcoded to use DefaultOrkaNamespace instead of the configured namespace, causing image saves to fail when the VM builder namespace differs from orka-default
  • Updated WaitForImage (interface, implementation, and mock) to accept a namespace parameter and pass it through to the underlying watcher
  • Fixed stepCreateImage cleanup and imageSaveNFS to use config.OrkaVMBuilderNamespace instead of the hardcoded DefaultOrkaNamespace

Changes

  • builder/orka/orka_client.go: Added namespace parameter to WaitForImage interface and implementation; pass namespace to waitForImage instead of DefaultOrkaNamespace
  • builder/orka/step_create_image.go: Use config.OrkaVMBuilderNamespace in both image cleanup and imageSaveNFS creation/wait calls
  • mocks/orka_client_mock.go: Updated mock to match the updated WaitForImage interface signature

@sonarqubecloud
Copy link

@jasonjanderson jasonjanderson marked this pull request as ready for review February 24, 2026 20:42
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