Skip to content

Refactor/extract shared k8s module#47

Merged
gCyrille merged 3 commits intomainfrom
refactor/extract-shared-k8s-module
Mar 12, 2026
Merged

Refactor/extract shared k8s module#47
gCyrille merged 3 commits intomainfrom
refactor/extract-shared-k8s-module

Conversation

@gCyrille
Copy link
Copy Markdown
Collaborator

Summary

This PR extracts shared XO/Kubernetes logic into the new xenorchestra-k8s-common module and upgrades dependencies to address stdlib vulnerabilities.

Changes

Refactor: delegate shared XO/k8s logic to xenorchestra-k8s-common

Removes duplicated code that has been consolidated into the shared xenorchestra-k8s-common library:

  • Deleted pkg/provider/provider.go and tests — ProviderName, GetProviderID now come from xok8s
  • Deleted pkg/xenorchestra/client.go and tests — XoClient and cloud config logic moved to xok8s
  • Deleted pkg/xenorchestra/cloud_config.go and tests — cloud config parsing delegated to xok8s
  • Deleted pkg/xenorchestra/xo_labels.go — XO label constants now from xok8s
  • Updated pkg/xenorchestra/instances.go, cloud.go, and node label sync helpers to use the shared module

Build: dependency upgrades

  • Upgraded Xen Orchestra Golang SDK to v1.13.0
  • Upgraded Go toolchain to 1.25.8 to resolve stdlib security vulnerabilities
  • Pinned github.com/vatesfr/xenorchestra-k8s-common to v0.1.0
  • Upgraded k8s.io/* dependencies

Testing

  • Existing unit tests updated
  • No behavioral changes — this is a pure structural refactor

Related

@gCyrille gCyrille force-pushed the refactor/extract-shared-k8s-module branch from 8a7abe9 to 97330d2 Compare March 10, 2026 16:29
@gCyrille gCyrille merged commit ea0419c into main Mar 12, 2026
2 checks passed
@gCyrille gCyrille deleted the refactor/extract-shared-k8s-module branch March 12, 2026 09:02
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