Skip to content

fix: align vsphere-vm with conventions, bump TF module to v2.12.0#52

Merged
patrick-hermann-sva merged 3 commits intomainfrom
fix/vsphere-vm-conventions
Apr 9, 2026
Merged

fix: align vsphere-vm with conventions, bump TF module to v2.12.0#52
patrick-hermann-sva merged 3 commits intomainfrom
fix/vsphere-vm-conventions

Conversation

@patrick-hermann-sva
Copy link
Copy Markdown
Contributor

Summary

  • Fix filename typo: defintion.yamldefinition.yaml
  • Move composition from apis/ to compositions/vsphere-vm.yaml
  • Rename example claim.yamlvsphere-vm.yaml
  • Bump TF module from v1.7.5-2.7.0 to v2.12.0-1.0.0 (released 2026-04-07)
  • Fix crossplane.yaml version to >=2.13.0
  • Remove unused function-go-templating from functions.yaml
  • Update README render command paths

Notes

The new TF module version adds validation constraints on vm_memory, vm_num_cpus, and vm_disk_size. The composition passes these as strings which OpenTofu auto-converts, but the allowed values are now restricted (see release notes).

Test plan

  • Run crossplane render with updated paths
  • Validate composition against cluster with new module ref
  • Test VM provisioning with updated module

🤖 Generated with Claude Code

patrick-hermann-sva and others added 3 commits April 9, 2026 09:01
- Fix filename typo: defintion.yaml → definition.yaml
- Move composition to compositions/vsphere-vm.yaml
- Rename example claim.yaml → vsphere-vm.yaml
- Update TF module from v1.7.5-2.7.0 to v2.12.0-1.0.0
- Fix crossplane version to >=2.13.0
- Remove unused function-go-templating from functions.yaml
- Update README render command paths

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Match the installed Function resource name on cluster
(crossplane-contrib-function-patch-and-transform).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Document ClusterProviderConfig, tfvars secret (incl. SOPS), RBAC, and
DeploymentRuntimeConfig setup. Update poll interval to 30s.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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