Skip to content

Conversation

@Schtil
Copy link

@Schtil Schtil commented Nov 9, 2025

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=ru

Description of changes:

Обновлено описание поля resources.disk_size и поля disk_size_autoscaling.disk_size_limit в ресурсе yandex_mdb_redis_cluster_v2.

Ранее в документации было указано, что значение задаётся в байтах.
На практике API ожидает целое число в гигабайтах — при передаче значения в байтах
создание кластера завершается ошибками валидации или переполнением int64
(disk size must be a positive integer value (-9223372036854775808)).

При этом в примерах все задается корректно - в гигабайтах :)

resources = {
    resource_preset_id = "hm1.nano"
    disk_size          = 16
  }

Никто уже не вернёт мне пару часов жизни, потраченных на поиски причины,
почему «20 ГБ» внезапно оказались отрицательным числом.

Copilot AI review requested due to automatic review settings November 9, 2025 18:58
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the documentation for disk size units in the Terraform Redis cluster v2 resource, changing the unit specification from "bytes" to "gigabytes" for clarity.

  • Updated disk_size parameter documentation to specify gigabytes instead of bytes
  • Updated disk_size_limit parameter documentation to specify gigabytes instead of bytes

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@yfm-team
Copy link

yfm-team commented Nov 9, 2025

Все готово!
Commit: ff6cc96
Проверьте результат: RU.
Check the result: EN.

@yfm-team
Copy link

yfm-team commented Nov 9, 2025

Все готово!
Commit: dd5ce7f
Проверьте результат: RU.
Check the result: EN.

@yfm-team
Copy link

yfm-team commented Nov 9, 2025

Все готово!
Commit: bb25d85
Проверьте результат: RU.
Check the result: EN.

@vnkiryushkin vnkiryushkin added managed-redis Yandex Managed Service for Redis terraform labels Nov 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

managed-redis Yandex Managed Service for Redis terraform

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants