ci: nested SDS E2E on skeleton#1693
Draft
yachmenevas wants to merge 14 commits intochore/ci/e2e-matrix-skeletonfrom
Draft
ci: nested SDS E2E on skeleton#1693yachmenevas wants to merge 14 commits intochore/ci/e2e-matrix-skeletonfrom
yachmenevas wants to merge 14 commits intochore/ci/e2e-matrix-skeletonfrom
Conversation
575af76 to
3665f4e
Compare
Isteb4k
requested changes
Nov 13, 2025
14c8ee7 to
20ca878
Compare
…ort; restore jump-host; stabilize SDS bring-up (MPO + explicit ModuleConfigs + CRD‑compatible LVG + safe default SC patch) Signed-off-by: Anton Yachmenev <anton.yachmenev@flant.com>
aaf1812 to
b10fc10
Compare
Isteb4k
requested changes
Nov 14, 2025
Isteb4k
requested changes
Nov 14, 2025
Isteb4k
requested changes
Nov 14, 2025
…scripts; HTTPS-only ingress; param cleanup wait\n\n- remove report scripts and local tests task\n- drop unused values (storageProfiles, infra/virtualDisks/security, extra features)\n- add build_parent_kubeconfig.sh + inject_registry_cfg.sh and use them in workflow\n- call task nested:storage:sds directly from workflow; remove wrapper\n- trim logs (no lsblk, no set -x); keep concise status\n- ingress: drop 80/wildcard-http; keep HTTPS passthrough only\n- cleanup: param prefix + wait for namespace deletion Signed-off-by: Anton Yachmenev <anton.yachmenev@flant.com>
- use DEV_REGISTRY_DOCKER_CFG only; inject into values; login via docker cfg - fix GitHub Actions 'if' expressions (no secrets.* in if; use env) - add Apache-2.0 headers to helper scripts (dmtlint) - format ci/dvp-e2e/Taskfile.yaml with repo Prettier (prettier) - trim push triggers to ci-e2e-nested-sds only; remove noisy comments - keep setup/prepare/cleanup only (tests/report moved to next PR) Signed-off-by: Anton Yachmenev <anton.yachmenev@flant.com>
bfec2ca to
22b75f5
Compare
…oncurrency + unified job concurrency (head_ref||ref_name) to prevent duplicate runs\n- use DEV_REGISTRY_DOCKER_CFG only; inject into values; login via docker cfg\n- fix Actions 'if' expressions (no secrets.* in if; use env)\n- add Apache-2.0 headers to helper scripts (dmtlint)\n- format ci/dvp-e2e/Taskfile.yaml with repo Prettier (prettier)\n- trim push triggers to ci-e2e-nested-sds only; remove noisy comments Signed-off-by: Anton Yachmenev <anton.yachmenev@flant.com>
1abf3fb to
66d4150
Compare
…laced by infra:attach-storage-disks-hotplug) Signed-off-by: Anton Yachmenev <anton.yachmenev@flant.com>
… and drop deckhouse.kubernetesVersion from values Signed-off-by: Anton Yachmenev <anton.yachmenev@flant.com>
c471757 to
092aa3f
Compare
…espaces) and call it from workflow; prefix stays configurable Signed-off-by: Anton Yachmenev <anton.yachmenev@flant.com>
…e (parent:kubeconfig, values:inject-registry); workflow calls tasks; keep params minimal Signed-off-by: Anton Yachmenev <anton.yachmenev@flant.com>
…(caused unmatched 'done'); ensure Task installed before calling tasks in cleanup Signed-off-by: Anton Yachmenev <anton.yachmenev@flant.com>
730c3d1 to
c318912
Compare
c318912 to
ff007a7
Compare
Isteb4k
requested changes
Nov 19, 2025
.github/workflows/e2e-matrix.yml
Outdated
| PARENT_STORAGE_CLASS: ${{ env.PARENT_STORAGE_CLASS }} | ||
| IMAGE_STORAGE_CLASS: ${{ env.IMAGE_STORAGE_CLASS }} | ||
| ATTACH_DISK_SIZE: ${{ env.ATTACH_DISK_SIZE }} | ||
| DATA_DISK_COUNT: ${{ env.DATA_DISK_COUNT }} |
Contributor
There was a problem hiding this comment.
Can we use matrix.data_disk_count here, instead of env.DATA_DISK_COUNT?
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Add a minimal nested SDS E2E on top of the skeleton.
Preserves triggers, dependencies, and ordering; replaces Ceph-only parts with SDS enablement (source: deckhouse) and nested replicated storage setup.
Why do we need it, and what problem does it solve?
Validate triggers/dependencies and SDS profile flow inside nested clusters before introducing full tests/notifications.
What is the expected result?
Workflow runs on schedule/dispatch/PR, boots a nested cluster, enables SDS modules, creates LVMVolumeGroups + ReplicatedStoragePool + ReplicatedStorageClass, and proceeds to E2E gates.
Checklist
Changelog entries
section: ci
type: chore
summary: "Add nested SDS E2E on top of skeleton (deckhouse source)."