apache / apache/cloudstack

Custom IOPS not applied to root disk when deploying a VM from a template

Đang mở
#14,103 0 bình luận 0 reaction 0 người được giao Được @xujiantop-crypto nhận Xem trên GitHub
bug
Ngôn ngữ chính
Java
Star
3.1k
Fork
1.4k
Merge trung bình
6 ngày 19 giờ
Pull request đã merge (30 ngày)
32

Mô tả

### problem

### Description

When a user deploys a VM from a template with a Custom IOPS offering for the ROOT disk, the min IOPS and max IOPS entered in Deploy Instance are stored in VM details but not copied to the ROOT volume.
volumes.min_iops and volumes.max_iops stay empty. Vendor storage that reads IOPS from the volumes gets null for min IOPS and max IOPS instead of the values entered at deploy.

### Root Cause
The UI sends custom IOPS as details[0].minIops and details[0].maxIops. CloudStack stores them in vm_instance_details with those same names (minIops / maxIops).

When the ROOT volume is created, CloudOrchestrator.createVirtualMachine() reads IOPS using the API constants MIN_IOPS and MAX_IOPS. Those constants are miniops and maxiops.

The names do not match, so the lookup finds nothing. DiskOfferingInfo is left with no IOPS, and VolumeOrchestrator creates the ROOT volume with empty min_iops / max_iops.

It is a regression from Apache CloudStack PR: https://github.com/apache/cloudstack/pull/8800 (April 2024).

Image

### Observations:

vm_instance_details has minIops / maxIops

Image

ROOT volume min_iops / max_iops are NULL

Image

### versions

CloudStack: 4.20 and later, including 4.23.

### The steps to reproduce the bug

1. Go to Instances → Add Instance.
2. Select a template (do not use ISO).
3. In Compute offering, set Override root disk offering to true and select the Custom IOPS disk offering for the ROOT disk.
4. Enter min IOPS and max IOPS (for example, 100 and 1000).
5. Complete the wizard and deploy the VM.
6. After the VM is created, check the ROOT volume and VM details

### What to do about it?

### Impact

- The VM deploys with no error even though min and max IOPS are not applied on the ROOT volume.
- Expected IOPS are not served because custom min/max IOPS are not set on the ROOT volume, so there is no IOPS guarantee.

### Expected Behaviour
Custom IOPS from deploy-VM details must be applied to the root volume 

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Hướng nghiên cứu

Bắt đầu từ CloudOrchestrator.createVirtualMachine() và theo dõi cách các giá trị vm_instance_details được đọc vào DiskOfferingInfo. So sánh tên chi tiết minIops/maxIops với các hằng số API MIN_IOPS và MAX_IOPS, sau đó xác minh rằng volume ROOT nhận được cả hai giá trị trong quá trình deployment. Xác nhận fix bằng pull request được liên kết và một deployment test sử dụng IOPS tùy chỉnh.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
java
Lĩnh vực
backend, infrastructure
Loại issue
Lỗi
Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Đặc tả rõ ràng
Mức phù hợp với người mới
25/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.