Skip to content

Conversation

diafour
Copy link
Member

@diafour diafour commented Sep 18, 2025

Description

  • Ignore failed to watch, fail to elect, fail to create vmip errors in virtualization-controller log.
  • Use Eventually to test IP in cilium agents.
  • Fix handling kubectl errors in SaveResourcesForTest.

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

  • More stable e2e tests, ignore some false-positives.
  • See errors occurred in AfterEach handler.

What is the expected result?

  • Reduce number of failed e2e runs.
  • Upload artifact with resources

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.

Note: tested in e2e runs for #1432 :
https://github.com/deckhouse/virtualization/actions/runs/17849395047/job/50754944907
https://github.com/deckhouse/virtualization/actions/runs/17837555108/job/50718609496

Changelog entries

section: ci
type: chore
summary: Fix creating artifact for failed e2e test, ignore more false-positive errors.

Isteb4k
Isteb4k previously approved these changes Sep 18, 2025
@diafour diafour force-pushed the chore/e2e/stability-fixes-upload-artifacts-fix branch from b31c1da to 0a83e98 Compare September 18, 2025 15:54
@diafour diafour changed the title chore(e2e): fixes for more stability chore(ci): e2e fixes for more stable tests Sep 18, 2025
@diafour diafour enabled auto-merge (squash) September 18, 2025 15:58
- Ignore failed to watch, fail to elect, fail to create vmip errors in virtualization-controller log.
- Use Eventually to test IP in cilium agents.
- Fix handling kubectl errors in SaveResourcesForTest.
- Wait for snapshots readiness before checking for unfrozen filesystems.

Signed-off-by: Ivan Mikheykin <[email protected]>
@diafour diafour force-pushed the chore/e2e/stability-fixes-upload-artifacts-fix branch from 0a83e98 to e48ee09 Compare September 19, 2025 06:59
@diafour diafour merged commit 10906cb into main Sep 19, 2025
26 of 27 checks passed
@diafour diafour deleted the chore/e2e/stability-fixes-upload-artifacts-fix branch September 19, 2025 08:00
diafour added a commit that referenced this pull request Sep 19, 2025
e2e artifacts are created in runner temp dir after PR #1470

Signed-off-by: Ivan Mikheykin <[email protected]>
diafour added a commit that referenced this pull request Sep 19, 2025
e2e artifacts are created in runner temp dir after PR #1470

Signed-off-by: Ivan Mikheykin <[email protected]>
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.

2 participants