feat!: add support for private subnets for dedicated servers by Disaxy · Pull Request #376 · selectel/terraform-provider-selectel

Disaxy · 2026-02-17T13:17:50Z

Добавлены атрибуты с IP-адресами созданного сервера для ресурса
dedicated_server_v1.
Добавлен аргумент для ресурса dedicated_server_v1 для возможности создания
сервера сразу с приватным vlan.
Добавлен атрибут для ресурса dedicated_server_v1, отражающий наличие
приватного VLAN на сервере.
Добавлен источник данных для работы с приватными подсетями.
Добавлены параметры для подключения приватной подсети к серверу для
ресурса dedicated_server_v1.
Добавлен ресурс для приватных подсетей.
Убраны из списка вывода data source dedicated_location_v1 локации, у которых
visibility = “admin_only”.

Jira Issue: COOPERATION-97008

TawR1024 · 2026-02-20T09:38:56Z

selectel/resource_selectel_dedicated_private_subnet_v1.go

+	}
+
+	locationID := d.Get("location_id").(string)
+	vlan := d.Get("vlan").(string)


vlan имеет ограничения, хорошо бы проверять, что номер влана не выходит за пределы [1..4095]. А лучше вообще сделать его int и в строку превращать уже ближе к формированию query

если делать интом проверку можно повесить в ресурсе

validation.IntBetween(1, 4095)

я не эксперт, но гугл подсказывает что 4095 зарезерирован. тут нужна доп верификация. возможно итоговая валидация должна быть

validation.IntBetween(1, 4094)

Да, у нас есть валидация на переданное значение. Range(1,3499)

TawR1024 · 2026-02-24T08:59:00Z

selectel/resource_selectel_dedicated_private_subnet_v1.go

+				ForceNew:    true,
+				Description: "VLAN TAG for the private subnet",
+			},
+			"subnet": {


Аналогично тут, стоит добавить валидацию
ex

TawR1024 · 2026-02-24T11:29:31Z

selectel/resource_selectel_dedicated_private_subnet_v1.go

+				Computed:    true,
+				Description: "ID of the created private subnet",
+			},
+			"location_id": {


Стоит добавить валидацию на то, что это uuid.
можно использовать из стандартного набора хелперов

TawR1024 · 2026-02-24T11:34:25Z

website/docs/d/dedicated_location_v1.html.markdown

 ## Attributes Reference

-* `locations` - List of the available locations:
+* `locations` - List of the available locations (excluding locations with `visibility = "admin_only"`):


нужна ли эта информация тут ? Сокрытие admin_only локаций, выглядит как наша служебная история, которую можно не показывать публично, чтобы не соблазнять всех желащих туда постучать.

TawR1024 · 2026-02-24T11:38:36Z

selectel/data_source_selectel_dedicated_private_subnet_v1.go

+							Type:     schema.TypeString,
+							Computed: true,
+						},
+						"reserved_ip": {


reserved_ip -> reserved_ips, раз это список, а не одиночная сущность

TawR1024 · 2026-02-24T11:48:14Z

selectel/networking.go

+	}
+
+	// Define private IP ranges
+	privateRanges := []struct {


Почему бы тут не использовать стандартный метод из [netip](https://cs.opensource.google/go/go/+/refs/tags/go1.26.0:src/net/netip/netip.go;l=640) ?

milkrage · 2026-02-24T18:10:24Z

selectel/resource_selectel_dedicated_private_subnet_v1.go

+
+	"github.com/hashicorp/terraform-plugin-sdk/v2/diag"
+	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
+	dedicated "github.com/selectel/dedicated-go/pkg/v2"


Обнови, пожалуйста, версию dedicated-go доv2.0.0

Обновлено в d5ea6ef

arla-arla-s · 2026-02-25T13:28:57Z

selectel/data_source_selectel_dedicated_private_subnet_v1.go

+	if err != nil {
+		return diag.FromErr(err)
+	}
+	if err := d.Set("subnets", subnetsFlatten); err != nil {


Думаю лучше использовать одинаковый подход к присваиванию ошибок и здесь тоже использовать

err = d.Set("subnets", subnetsFlatten) if err != nil { return diag.FromErr(err) }

arla-arla-s · 2026-02-25T13:40:01Z

selectel/data_source_selectel_dedicated_private_subnet_v1.go

+		if (filter.subnet == "" || filter.subnet == subnet.Subnet) && (filter.ip == "" || isIPIncluded) {
+			filteredSubnets = append(filteredSubnets, subnet)
+
+			continue


После этого continue ничего нет, его можно убрать

T-Sh · 2026-02-25T19:51:44Z

selectel/data_source_selectel_dedicated_private_subnet_v1.go

+	for _, network := range nets {
+		localSubnets, _, err := dsClient.NetworkLocalSubnets(ctx, network.UUID)
+		if err != nil {
+			continue // Continue with other networks if one fails


Точно ли корректно скипать здесь ошибку? Если необходимый ip/subnet находится в network, по которому список subnet получить не удалось, datasource по идее останется пустым, абсолютно непрозрачно по какой причине. Кажется, что ошибку надо обрабатывать и возвращать

Да, этот метод может вернуть пустой список и 200

Если список будет пустым, то append просто ничего не добавит. Тут проблема с следующем кейсе: мы делаем запрос dsClient.NetworkLocalSubnets, получаем какую-то ошибку (таймаут, например) и не выкидываем эту ошибку выше, мы просто делаем continue. Со стороны выглядит странно.

sel-bukharov · 2026-02-26T03:02:40Z

selectel/networking.go

+// IsPrivateSubnet validates if the provided CIDR belongs to private IP ranges.
+func IsPrivateSubnet(cidr string) bool {
+	return validatePrivateSubnet(cidr) == nil
+}


Эта функция нигде не используется кроме тестов

sel-bukharov · 2026-02-26T03:07:34Z

selectel/data_source_selectel_dedicated_private_subnet_v1.go

+						"ip": {
+							Type:     schema.TypeString,
+							Optional: true,
+						},


здесь можно использовать

ValidateFunc: validation.IsIPAddress

то же самое применимо для всех IP

sel-bukharov · 2026-02-26T03:14:24Z

selectel/resource_selectel_dedicated_private_subnet_v1.go

+	}
+
+	locationID := d.Get("location_id").(string)
+	vlan := d.Get("vlan").(string)


если делать интом проверку можно повесить в ресурсе

validation.IntBetween(1, 4095)

я не эксперт, но гугл подсказывает что 4095 зарезерирован. тут нужна доп верификация. возможно итоговая валидация должна быть

validation.IntBetween(1, 4094)

milkrage · 2026-02-25T17:01:39Z

selectel/dedicated.go

 }

-func getDedicatedClient(d *schema.ResourceData, meta interface{}) (*dedicated.ServiceClient, diag.Diagnostics) {
+func getDedicatedClient(d *schema.ResourceData, meta interface{}, withProjectScope bool) (*dedicated.ServiceClient, diag.Diagnostics) {


Судя по swagger спецификации, Dedicated Servers API, поддерживает для всех необходимых методов project scope. Это не так?

Все так, выделенные серверы поддерживают project scope во всех методах

Тогда необходимо убрать bool флаг из сигнатуры метода и всегда использовать клиент с project scope токеном.

milkrage · 2026-02-26T03:35:36Z

selectel/data_source_selectel_dedicated_private_subnet_v1.go

+			"project_id": {
+				Type:     schema.TypeString,
+				Required: true,
+			},
+			"filter": {
+				Type:     schema.TypeSet,
+				Optional: true,
+				MaxItems: 1,
+				Elem: &schema.Resource{
+					Schema: map[string]*schema.Schema{
+						"location_id": {
+							Type:     schema.TypeString,
+							Optional: true,
+						},
+						"ip": {
+							Type:     schema.TypeString,
+							Optional: true,
+						},
+						"subnet": {
+							Type:     schema.TypeString,
+							Optional: true,
+						},
+						"vlan": {
+							Type:     schema.TypeString,
+							Optional: true,
+						},
+					},
+				},
+			},


Хорошей практикой является валидация пользовательского ввода до отправки запроса в API.

location_id - uuid, в проекте используется метод validate.IsUUID

ip - validate.IIsIPAddress

subnet - validate.IIsCIDR

vlan - validate.IntBetween

milkrage · 2026-02-26T04:02:13Z

selectel/schema_selectel_dedicated_server_v1.go

-		dedicatedServerSchemaKeyPrivateSubnet: {
+		dedicatedServerSchemaKeyPrivateSubnetID: {
 			Type:     schema.TypeString,
 			Optional: true,
+			ForceNew: true,
+		},


Тут прям очень жестко ломается манифест (будет ошибка unsupported argument для старых манифестов), стоит ли так делать? Возможно нужно старое поле отметить Deprecated флагом и написать, что нужно использовать новые поля? При этом новые поля, это именно новые поля, а не ренейм старых.

@TawR1024, подскажи, пожалуйста, у меня правильное понимание процесса изменения схемы?

Переименования поля опасная операция. Если брать во внимание, что у этого ресурса уже есть пользователи то тут следуте пойти по пути через deprecate старого поля, добавление нового поля.

Также стоит учесть, что если у поля изменилось поведение, а именно добавили поведение ForceNew, то стоит очень хорошо протестировать поведение state в таком случае. Скорее всего, посольку при state refresh значение поля вместо null получит новое значение, то может быть ресурс придётся пересоздать, что нежелательно для серверов.

selectel/resource_selectel_dedicated_server_v1.go

milkrage · 2026-02-26T04:32:21Z

website/docs/d/dedicated_private_subnet_v1.html.markdown

+## Import
+
+You can import a subnet data source:
+
+```shell
+export OS_DOMAIN_NAME=<account_id>
+export OS_USERNAME=<username>
+export OS_PASSWORD=<password>
+export INFRA_PROJECT_ID=<selectel_project_id>
+terraform import selectel_dedicated_private_subnet_v1.subnet_ds <subnet_id>


У data source есть возможность импорта?

TawR1024 · 2026-02-27T05:05:19Z

selectel/provider.go

 	objectDedicatedServer              = "dedicated-server"
 	objectOS                           = "os"
 	objectLocation                     = "location"
+	objectNetwork                      = "network"


network для dedicated и для других услуг может быть разным.
Имеет смысл делать более специфичные имена, dedicated-network

Stage 1

1e526ba

milkrage requested review from T-Sh, TawR1024, arla-arla-s, idegree, milkrage and sel-bukharov February 17, 2026 14:36

milkrage added the dedicated dedicated servers label Feb 19, 2026

TawR1024 requested changes Feb 24, 2026

View reviewed changes

milkrage reviewed Feb 24, 2026

View reviewed changes

milkrage changed the title ~~Stage 1~~ feat: add support for local subnets Feb 25, 2026

milkrage changed the title ~~feat: add support for local subnets~~ feat(dedicated_server_v1): add support for local subnets Feb 25, 2026

arla-arla-s reviewed Feb 25, 2026

View reviewed changes

T-Sh reviewed Feb 25, 2026

View reviewed changes

sel-bukharov reviewed Feb 26, 2026

View reviewed changes

milkrage requested changes Feb 26, 2026

View reviewed changes

milkrage reviewed Feb 26, 2026

View reviewed changes

selectel/resource_selectel_dedicated_server_v1.go Show resolved Hide resolved

milkrage reviewed Feb 26, 2026

View reviewed changes

milkrage changed the title ~~feat(dedicated_server_v1): add support for local subnets~~ feat: add support for private subnets for dedicated servers Feb 26, 2026

chore: bump dedicated-go to v2.0.0

d5ea6ef

TawR1024 reviewed Feb 27, 2026

View reviewed changes

milkrage changed the title ~~feat: add support for private subnets for dedicated servers~~ feat!: add support for private subnets for dedicated servers Feb 28, 2026

Conversation

Disaxy commented Feb 17, 2026 • edited by milkrage Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

milkrage Feb 27, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

milkrage Feb 27, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

milkrage Feb 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

milkrage Feb 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

Disaxy commented Feb 17, 2026 •

edited by milkrage

Loading

milkrage Feb 27, 2026 •

edited

Loading

milkrage Feb 27, 2026 •

edited

Loading

milkrage Feb 26, 2026 •

edited

Loading

milkrage Feb 26, 2026 •

edited

Loading