Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cpuunits default to 1024 instead of 100 on cgroup2 machines #1589

Open
joshmcorreia opened this issue Oct 11, 2024 · 0 comments
Open

cpuunits default to 1024 instead of 100 on cgroup2 machines #1589

joshmcorreia opened this issue Oct 11, 2024 · 0 comments
Labels
🐛 bug Something isn't working size/S
Milestone

Comments

@joshmcorreia
Copy link
Contributor

Describe the bug
cpuunits default to 1024 regardless of cgroups version. According to the Proxmox wiki, it should default to 100 on machines that support cgroup2

It is a relative weight which defaults to 100 (or 1024 if the host uses legacy cgroup v1).

Since my cluster is on a machine with cgroup2, Proxmox defaults to 100 on all of my other VMs and then the VMs created with this provider end up with 1024.

To Reproduce
Steps to reproduce the behavior:

  1. Create a resource "proxmox_virtual_environment_vm"
  2. Run "terraform apply"
  3. See that it gives 1024 cpuunits instead of 100

Please also provide a minimal Terraform configuration that reproduces the issue:

resource "proxmox_virtual_environment_vm" "joshtest" {
  name      = "joshtest"
  node_name = "proxmoxjoshtest"
}

Expected behavior
I expect the default to be set depending on the cgroup version supported by the Proxmox server.

Screenshots
N/A

Additional context
Add any other context about the problem here.

  • Single or clustered Proxmox: Single
  • Proxmox version: 8.2.7
  • Provider version (ideally it should be the latest version): v0.66.1
  • Terraform/OpenTofu version:
$ tofu version
OpenTofu v1.8.3
  • OS (where you run Terraform/OpenTofu from):
$ cat /etc/os-release
PRETTY_NAME="Ubuntu 22.04.5 LTS"
NAME="Ubuntu"
VERSION_ID="22.04"
VERSION="22.04.5 LTS (Jammy Jellyfish)"
VERSION_CODENAME=jammy
...
  • Debug logs (TF_LOG=DEBUG terraform apply): N/A
@joshmcorreia joshmcorreia added the 🐛 bug Something isn't working label Oct 11, 2024
@bpg bpg added this to the v1.0 milestone Oct 12, 2024
@bpg bpg moved this to ☑️ Todo in terraform-provider-proxmox Dec 8, 2024
@bpg bpg added the size/S label Dec 8, 2024
@bpg bpg moved this from ☑️ Todo to ⛔️ Blocked in terraform-provider-proxmox Dec 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working size/S
Projects
Status: ⛔️ Blocked
Development

No branches or pull requests

2 participants