Skip to content

fix(ci): render infra config#2223

Merged
universal-itengineer merged 1 commit into
mainfrom
fix/ci/bootstrap-e2e-nested
Apr 16, 2026
Merged

fix(ci): render infra config#2223
universal-itengineer merged 1 commit into
mainfrom
fix/ci/bootstrap-e2e-nested

Conversation

@universal-itengineer
Copy link
Copy Markdown
Member

@universal-itengineer universal-itengineer commented Apr 16, 2026

Description

Fixes the infrastructure configuration rendering for nested e2e clusters by correcting the YAML structure and the Helm template lookup logic.

Changes:

  • Fixed YAML indentation in e2e-reusable-pipeline.yml - moved networkConfig to the correct nesting level
  • Fixed _helpers.tpl to properly extract networkConfig from the instances values using get function instead of incorrect dig call

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

The previous implementation (#2181) introduced SDN configuration for nested e2e clusters, but the YAML structure was incorrectly nested and the Helm template was unable to properly access the networkConfig values. This caused the clusterNetworkName to be empty, preventing VMs from connecting to the configured ClusterNetwork.

What is the expected result?

  1. The networkConfig block is correctly rendered in the infra values YAML
  2. The Helm template properly retrieves networkConfig.clusterNetworkName from instances
  3. VMs in nested e2e clusters can use the configured ClusterNetwork for additional network interfaces

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.

Changelog entries

section: ci
type: fix
summary: Fix infrastructure config rendering for nested e2e clusters
impact_level: low

Signed-off-by: Nikita Korolev <nikita.korolev@flant.com>
@universal-itengineer universal-itengineer force-pushed the fix/ci/bootstrap-e2e-nested branch from 6e2725a to d468cde Compare April 16, 2026 07:43
@universal-itengineer universal-itengineer marked this pull request as ready for review April 16, 2026 07:45
@nevermarine nevermarine added this to the v1.8.0 milestone Apr 16, 2026
@universal-itengineer universal-itengineer merged commit db9ac22 into main Apr 16, 2026
28 of 31 checks passed
@universal-itengineer universal-itengineer deleted the fix/ci/bootstrap-e2e-nested branch April 16, 2026 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants