Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,7 @@

# Generated test configuration files
testkit_v2/data/config.yml
testkit_v2/data/resources.yml
testkit_v2/data/resources.yml

# E2E test temporary files
e2e-tests/temp/
57 changes: 57 additions & 0 deletions e2e-tests/TODO.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,57 @@
# TODOs, ideas and questions

## Переиспользование кластера

1. Мы подключились к базовому кластеру (далее, гипервизор) и нужно создавать виртуалки. Мы по умолчанию считаем, что виртуализация там включена (Да) и падаем, если нет; либо мы включаем ее сами и падаем если не включилась (Нет). Да/нет?
2. Про идемпотентность тестового кластера.

2.1. Есть у нас описанный конфигом тестовый кластер. Мы подключились к ДКП на гипервизоре и видим, что виртуалка (одна из) вроде бы уже есть и вроде даже такая, как надо - неймспейс, проц, память, диск, имя, образ, даже клауд-инит конфиг. НО подключиться к машине проверить ОС и ядро не можем - authentication error. В этом случае мы:

2.1.1. Если это единственный мастер (в конфиге тестового кластера), сносим все виртуалки и переставляем их заново с развертыванием нового кластера с нуля (Да), или падаем (Нет). Да/нет?

2.1.2. Если это воркер или не единственный мастер, при условии нормального подключения хотя бы к одному мастеру и его соответствия конфигу, мы удаляем эту ноду в кластере, удаляем виртуалку, переставляем ее с нуля и добавляем в кластер в той роли, в которой она была (Да), или падаем (Нет). Да/нет? Тут могут быть сложности в виде невыгоняемых подов с ноды - как быть в этом случае?

2.2. Аналогично 2.1. но нода не соответствует конфигу.

2.2.1. Если это единственный мастер, то убиваем все и переставляем весь кластер целиком с нуля (Да), либо падаем (Нет). Да/нет?

2.2.2. Если это воркер или не единственный мастер, при условии нормального подключения хотя бы к одному мастеру и его соответствия конфигу, мы удаляем эту ноду в кластере, удаляем виртуалку, переставляем ее с нуля и добавляем в кластер в той роли, в которой она была (Да), или падаем (Нет). Да/нет? Тут могут быть сложности в виде невыгоняемых подов с ноды - как быть в этом случае?
> 2.3. Что делать если одна или несколько виртуалок выключены? Включаем и пробуем подключиться (Да), падаем (Нет). Да/нет?


alexandr.zimin
привет.

1. Да - падаем, если нет виртуализации(сами не включаем)

2. пункт - сделать флаг, который будет менять поведение, Пока там 2 переключалки сделать:
alwaysUseExisting (или еще варианты: ignoreTestConfig) - вообще не смотрим на требования, запускаем тесты на любом указанном кластере. если нет доступа - падаем и пишем об этом
alwaysRecreate - если в ns есть уже хоть одна виртуалка - падаем и пишем об этом. С этим режимом можно запускать только в чистом ns
Потом можно еще будет добавить useExistingWithConfigCheck - но пока его НЕ нужно реализовывать, чтобы время не терять

наверное alwaysRecreate стоит переименовать на alwaysCreate

ну и по умолчанию alwaysCreate

хотя даже давай обязательным флагом сделаем

просто падаем, если не задан

## Новая репа

Нужна ли? И в гитхабе ли? - сделать новую репу в фоксе, перетащить туда существующий код, убить в гитхаюе и пересоздать.

## Как ревьюить и к кому с этим идти?

Не ревьюить.






alwaysUseExisting - что проверять? Включен, доступен ДКП, включены модули? - да.
alwaysCreate - если виртуалки такие уже есть в НС, то падаем, если виртуалок нет - создаем кластер.

Нужно все просто. Если есть какие-то непонятные кейсы, падать с ошибкой, потом обсудим и пофиксим, есличо.

51 changes: 15 additions & 36 deletions e2e-tests/go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -5,65 +5,44 @@ go 1.24.2
toolchain go1.24.3

require (
github.com/deckhouse/csi-nfs/api v0.0.0-20250529065000-055714087eef
github.com/deckhouse/sds-node-configurator/api v0.0.0-20241211082010-e61684e9db31
github.com/deckhouse/sds-replicated-volume/api v0.0.0-20241206095502-b096856636cd
github.com/deckhouse/virtualization/api v0.15.0
github.com/go-logr/logr v1.4.2
github.com/onsi/ginkgo/v2 v2.21.0
github.com/onsi/gomega v1.35.1
k8s.io/api v0.32.1
k8s.io/apiextensions-apiserver v0.31.0
k8s.io/apimachinery v0.32.1
github.com/pkg/sftp v1.13.10
golang.org/x/crypto v0.46.0
golang.org/x/term v0.38.0
gopkg.in/yaml.v3 v3.0.1
k8s.io/client-go v0.32.1
k8s.io/utils v0.0.0-20241210054802-24370beab758
sigs.k8s.io/controller-runtime v0.19.4
)

require (
github.com/davecgh/go-spew v1.1.2-0.20180830191138-d8f796af33cc // indirect
github.com/emicklei/go-restful/v3 v3.11.0 // indirect
github.com/evanphx/json-patch/v5 v5.9.0 // indirect
github.com/fxamacker/cbor/v2 v2.7.0 // indirect
github.com/go-openapi/jsonpointer v0.21.0 // indirect
github.com/go-openapi/jsonreference v0.20.2 // indirect
github.com/go-openapi/swag v0.23.0 // indirect
github.com/go-logr/logr v1.4.2 // indirect
github.com/go-task/slim-sprig/v3 v3.0.0 // indirect
github.com/gogo/protobuf v1.3.2 // indirect
github.com/golang/protobuf v1.5.4 // indirect
github.com/google/gnostic-models v0.6.8 // indirect
github.com/google/go-cmp v0.6.0 // indirect
github.com/google/gofuzz v1.2.0 // indirect
github.com/google/pprof v0.0.0-20241029153458-d1b30febd7db // indirect
github.com/google/uuid v1.6.0 // indirect
github.com/josharian/intern v1.0.0 // indirect
github.com/json-iterator/go v1.1.12 // indirect
github.com/mailru/easyjson v0.7.7 // indirect
github.com/kr/fs v0.1.0 // indirect
github.com/kr/pretty v0.3.1 // indirect
github.com/modern-go/concurrent v0.0.0-20180306012644-bacd9c7ef1dd // indirect
github.com/modern-go/reflect2 v1.0.2 // indirect
github.com/munnerz/goautoneg v0.0.0-20191010083416-a7dc8b61c822 // indirect
github.com/openshift/api v0.0.0-20230503133300-8bbcb7ca7183 // indirect
github.com/openshift/custom-resource-status v1.1.2 // indirect
github.com/pborman/uuid v1.2.1 // indirect
github.com/pkg/errors v0.9.1 // indirect
github.com/rogpeppe/go-internal v1.12.0 // indirect
github.com/spf13/pflag v1.0.5 // indirect
github.com/x448/float16 v0.8.4 // indirect
golang.org/x/net v0.38.0 // indirect
golang.org/x/net v0.47.0 // indirect
golang.org/x/oauth2 v0.23.0 // indirect
golang.org/x/sys v0.31.0 // indirect
golang.org/x/term v0.30.0 // indirect
golang.org/x/text v0.23.0 // indirect
golang.org/x/sys v0.39.0 // indirect
golang.org/x/text v0.32.0 // indirect
golang.org/x/time v0.7.0 // indirect
golang.org/x/tools v0.26.0 // indirect
google.golang.org/protobuf v1.35.1 // indirect
gopkg.in/evanphx/json-patch.v4 v4.12.0 // indirect
golang.org/x/tools v0.39.0 // indirect
gopkg.in/check.v1 v1.0.0-20201130134442-10cb98267c6c // indirect
gopkg.in/inf.v0 v0.9.1 // indirect
gopkg.in/yaml.v3 v3.0.1 // indirect
k8s.io/apimachinery v0.32.1 // indirect
k8s.io/klog/v2 v2.130.1 // indirect
k8s.io/kube-openapi v0.0.0-20241105132330-32ad38e42d3f // indirect
kubevirt.io/api v1.0.0 // indirect
kubevirt.io/containerized-data-importer-api v1.57.0-alpha1 // indirect
kubevirt.io/controller-lifecycle-operator-sdk/api v0.0.0-20220329064328-f3cc58c6ed90 // indirect
k8s.io/utils v0.0.0-20241210054802-24370beab758 // indirect
sigs.k8s.io/json v0.0.0-20241014173422-cfa47c3a1cc8 // indirect
sigs.k8s.io/structured-merge-diff/v4 v4.5.0 // indirect
sigs.k8s.io/yaml v1.4.0 // indirect
Expand Down
Loading