[internal] add e2e tests to CI by NikolayDemchuk · Pull Request #196 · deckhouse/sds-node-configurator

NikolayDemchuk · 2026-02-17T06:30:37Z

Description

Add E2E smoke tests for sds-node-configurator using storage-e2e framework. Tests cover BlockDevice discovery and LVMVolumeGroup creation.

Why do we need it, and what problem does it solve?

These tests validate core functionality on real clusters, triggered via e2e-smoke-test PR label.

What is the expected result?

Adding e2e-smoke-test label to PR triggers E2E job. Results posted as PR comment with pass/fail status and link to logs.

Checklist

The code is covered by unit tests.
e2e tests passed.
Documentation updated according to the changes.
Changes were tested in the Kubernetes cluster manually.

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

…ging initialization. Update go.mod to include storage-e2e module.

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

github-actions · 2026-03-06T12:16:58Z

E2E Smoke Tests Results ✅

Module: sds-node-configurator
Framework: storage-e2e
Namespace: e2e-sds-node-configurator-pr196-22763042077
Status: Tests passed

View Details

Workflow: Build and push for dev
Run ID: 22763042077
Commit: 83b64a1
PR: [internal] add e2e tests to CI #196
Triggered by: e2e-smoke-test label
Exit Code: 0

Tests were executed via storage-e2e (BlockDevice discovery, LVMVolumeGroup).

Artifacts: Test logs are available in workflow artifacts.

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

github-actions · 2026-03-06T12:58:19Z

E2E Smoke Tests Results ✅

Module: sds-node-configurator
Framework: storage-e2e
Namespace: e2e-sds-node-configurator-pr196-22764407083
Status: Tests passed

View Details

Workflow: Build and push for dev
Run ID: 22764407083
Commit: c141404
PR: [internal] add e2e tests to CI #196
Triggered by: e2e-smoke-test label
Exit Code: 0

Tests were executed via storage-e2e (BlockDevice discovery, LVMVolumeGroup).

Artifacts: Test logs are available in workflow artifacts.

github-actions · 2026-03-06T12:59:45Z

E2E Smoke Tests Results ✅

Module: sds-node-configurator
Framework: storage-e2e
Namespace: e2e-sds-node-configurator-pr196-22764407178
Status: Tests passed

View Details

Workflow: Build and checks
Run ID: 22764407178
Commit: c141404
PR: [internal] add e2e tests to CI #196
Triggered by: e2e-smoke-test label
Exit Code: 0

Tests were executed via storage-e2e (BlockDevice discovery, LVMVolumeGroup).

Artifacts: Test logs are available in workflow artifacts.

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

github-actions · 2026-03-06T13:35:37Z

E2E Smoke Tests Results ✅

Module: sds-node-configurator
Framework: storage-e2e
Namespace: e2e-sds-node-configurator-pr196-22765679745
Status: Tests passed

View Details

Workflow: Build and push for dev
Run ID: 22765679745
Commit: 9f9d622
PR: [internal] add e2e tests to CI #196
Triggered by: e2e-smoke-test label
Exit Code: 0

Tests were executed via storage-e2e (BlockDevice discovery, LVMVolumeGroup).

Artifacts: Test logs are available in workflow artifacts.

github-actions · 2026-03-06T13:35:44Z

E2E Smoke Tests Results ✅

Module: sds-node-configurator
Framework: storage-e2e
Namespace: e2e-sds-node-configurator-pr196-22765679782
Status: Tests passed

View Details

Workflow: Build and checks
Run ID: 22765679782
Commit: 9f9d622
PR: [internal] add e2e tests to CI #196
Triggered by: e2e-smoke-test label
Exit Code: 0

Tests were executed via storage-e2e (BlockDevice discovery, LVMVolumeGroup).

Artifacts: Test logs are available in workflow artifacts.

github-actions · 2026-03-12T13:51:10Z

E2E Smoke Tests Results ❌

Module: sds-node-configurator
Framework: storage-e2e
Namespace: undefined
Status: Tests failed

View Details

Workflow: Build and checks
Run ID: 23005223636
Commit: e0de21b
PR: [internal] add e2e tests to CI #196
Triggered by: e2e-smoke-test label
Exit Code: 1

Tests were executed via storage-e2e (BlockDevice discovery, LVMVolumeGroup).

Artifacts: Test logs are available in workflow artifacts.

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

github-actions · 2026-03-12T15:04:32Z

E2E Smoke Tests Results ✅

Module: sds-node-configurator
Framework: storage-e2e
Namespace: e2e-sds-node-configurator-pr196-23007008138
Status: Tests passed

View Details

Workflow: Build and checks
Run ID: 23007008138
Commit: 0823ebc
PR: [internal] add e2e tests to CI #196
Triggered by: e2e-smoke-test label
Exit Code: 0

Tests were executed via storage-e2e (BlockDevice discovery, LVMVolumeGroup).

Artifacts: Test logs are available in workflow artifacts.

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

github-actions · 2026-03-12T15:35:08Z

E2E Smoke Tests Results ❌

Module: sds-node-configurator
Framework: storage-e2e
Namespace: undefined
Status: Tests failed

View Details

Workflow: Build and checks
Run ID: 23010080820
Commit: e2a7c99
PR: [internal] add e2e tests to CI #196
Triggered by: e2e-smoke-test label
Exit Code: 1

Tests were executed via storage-e2e (BlockDevice discovery, LVMVolumeGroup).

Artifacts: Test logs are available in workflow artifacts.

github-actions · 2026-03-13T09:53:43Z

E2E Smoke Tests Results ✅

Module: sds-node-configurator
Framework: storage-e2e
Namespace: e2e-sds-node-configurator-pr196-23010080820
Status: Tests passed

View Details

Workflow: Build and checks
Run ID: 23010080820
Commit: e2a7c99
PR: [internal] add e2e tests to CI #196
Triggered by: e2e-smoke-test label
Exit Code: 0

Tests were executed via storage-e2e (BlockDevice discovery, LVMVolumeGroup).

Artifacts: Test logs are available in workflow artifacts.

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

github-actions · 2026-03-16T09:56:20Z

E2E Smoke Tests Results ✅

Module: sds-node-configurator
Framework: storage-e2e
Namespace: e2e-sds-node-configurator-pr196-23136544110
Status: Tests passed

View Details

Workflow: Build and checks
Run ID: 23136544110
Commit: faea341
PR: [internal] add e2e tests to CI #196
Triggered by: e2e-smoke-test label
Exit Code: 0

Tests were executed via storage-e2e (BlockDevice discovery, LVMVolumeGroup).

Artifacts: Test logs are available in workflow artifacts.

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

* added E2E test Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> --------- Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

* [agent] Use GetLV instead of cache after LVM mutations to avoid delays After lvcreate and lvextend, the in-memory cache contains stale data until the scanner runs. This caused unnecessary 5s requeues and blocking busy-wait loops. - LLV create: replace getLVActualSize with commands.GetLV after lvcreate - LLV resize: replace getLVActualSize with commands.GetLV after lvextend - LLV extender: replace FindLV busy-wait loop with GetLV after lvextend - LLVS snapshot: use GetLV after CreateThinLogicalVolumeSnapshot instead of requeueing for cache discovery Signed-off-by: Aleksandr Zimin <alexandr.zimin@flant.com> * [agent] Fix BlockDeviceFilter crash on In/NotIn with empty values BlockDeviceFilter resources with In/NotIn matchExpressions and nil/empty values caused metav1.LabelSelectorAsSelector to fail, breaking the entire block device reconciliation loop. Add sanitizeLabelSelector() that drops such vacuous expressions before parsing. Add tests for nil values, empty values, and mixed cases. Signed-off-by: Aleksandr Zimin <alexandr.zimin@flant.com> * tests Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> * test added Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> * fix Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> * [internal] add e2e tests to CI (#196) * added E2E test Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> --------- Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> * [agent] Use GetLV instead of cache after LVM mutations to avoid delays After lvcreate and lvextend, the in-memory cache contains stale data until the scanner runs. This caused unnecessary 5s requeues and blocking busy-wait loops. - LLV create: replace getLVActualSize with commands.GetLV after lvcreate - LLV resize: replace getLVActualSize with commands.GetLV after lvextend - LLV extender: replace FindLV busy-wait loop with GetLV after lvextend - LLVS snapshot: use GetLV after CreateThinLogicalVolumeSnapshot instead of requeueing for cache discovery Signed-off-by: Aleksandr Zimin <alexandr.zimin@flant.com> * tests Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> * fix Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> * fixed duplicate consts and such Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> * fix Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> * added docs Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> * fix Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> * fix Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> * fix Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> * fix Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> * fix Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> * fix Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> * fix Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> * fix Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> * fix Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> * fix Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> * fix: remove accidental merge conflict markers in .gitignore and reconciler Made-with: Cursor * fix Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> * fix duplicates Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> * fix Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> * fix Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> * fix Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> * fix Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> * fix Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> --------- Signed-off-by: Aleksandr Zimin <alexandr.zimin@flant.com> Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> Co-authored-by: Aleksandr Zimin <alexandr.zimin@flant.com>

NikolayDemchuk added 20 commits February 6, 2026 00:17

added E2E test

b65dad2

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

fixes

b79d1ee

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

fix

78a0310

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

fix

c387861

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

fix

f2d9ade

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

fix

064ac02

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

fix

5ec9980

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

fix

bf84c5e

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

fix

a59624d

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

ЕУЫЕШТП

b8695f9

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

fix

2a2004c

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

fix

15c066b

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

fix

ed65a14

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

fix

ca735b7

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

fix

145d8f2

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

fix

076b5b8

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

fix

0cfe682

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

Enhance block device discovery tests with DKP cluster support and log…

78f48b0

…ging initialization. Update go.mod to include storage-e2e module.

added CI

aa8df22

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

Update build_dev.yml

09777d0

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

NikolayDemchuk added the e2e-smoke-test label Mar 6, 2026

NikolayDemchuk self-assigned this Mar 6, 2026

fix go version

2a65416

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

fix

beab985

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

fix

208b560

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

cache fixes

22b2f1d

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

fix

8b7fb91

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

fix

6ffacf2

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

github-actions Bot removed the e2e-smoke-test label Mar 16, 2026

fix

9682b6e

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

duckhawk reviewed Mar 23, 2026

View reviewed changes

NikolayDemchuk added 6 commits March 25, 2026 17:06

fix

c5af4b9

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

fix

bd1b698

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

fix

4518c73

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

fix

b767b38

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

fix

740d432

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

fix

629fd11

Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

NikolayDemchuk added the enhancement New feature or request label Mar 25, 2026

NikolayDemchuk changed the title ~~Ndemchuk e2e tests~~ [internal] add e2e tests to CI Mar 25, 2026

NikolayDemchuk marked this pull request as ready for review March 25, 2026 11:25

duckhawk self-requested a review March 25, 2026 11:26

duckhawk approved these changes Mar 25, 2026

View reviewed changes

NikolayDemchuk merged commit a87764d into main Mar 25, 2026
27 of 30 checks passed

NikolayDemchuk added a commit that referenced this pull request Mar 26, 2026

[internal] add e2e tests to CI (#196)

3f0d5a0

* added E2E test Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> --------- Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

NikolayDemchuk added a commit that referenced this pull request Mar 27, 2026

[internal] add e2e tests to CI (#196)

5d24bd8

* added E2E test Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> --------- Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

NikolayDemchuk added a commit that referenced this pull request Apr 3, 2026

[internal] add e2e tests to CI (#196)

69abdf7

* added E2E test Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> --------- Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

NikolayDemchuk added a commit that referenced this pull request Apr 12, 2026

[internal] add e2e tests to CI (#196)

c648481

* added E2E test Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> --------- Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

NikolayDemchuk added a commit that referenced this pull request Apr 13, 2026

[internal] add e2e tests to CI (#196)

b88d070

* added E2E test Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com> --------- Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>

NikolayDemchuk deleted the ndemchuk-e2e-tests branch April 13, 2026 16:05

Conversation

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

Uh oh!

Description

Why do we need it, and what problem does it solve?

What is the expected result?

Checklist

Uh oh!

github-actions Bot commented Mar 6, 2026

E2E Smoke Tests Results ✅

Uh oh!

github-actions Bot commented Mar 6, 2026

E2E Smoke Tests Results ✅

Uh oh!

github-actions Bot commented Mar 6, 2026

E2E Smoke Tests Results ✅

Uh oh!

github-actions Bot commented Mar 6, 2026

E2E Smoke Tests Results ✅

Uh oh!

github-actions Bot commented Mar 6, 2026

E2E Smoke Tests Results ✅

Uh oh!

github-actions Bot commented Mar 12, 2026

E2E Smoke Tests Results ❌

Uh oh!

github-actions Bot commented Mar 12, 2026

E2E Smoke Tests Results ✅

Uh oh!

github-actions Bot commented Mar 12, 2026

E2E Smoke Tests Results ❌

Uh oh!

github-actions Bot commented Mar 13, 2026

E2E Smoke Tests Results ✅

Uh oh!

github-actions Bot commented Mar 16, 2026

E2E Smoke Tests Results ✅

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

NikolayDemchuk commented Feb 17, 2026 •

edited

Loading