Skip to content

test: add describe nodes to dump filed resourses#2042

Merged
universal-itengineer merged 9 commits intomainfrom
test/fix/add-describe-nodes-modern-test
Mar 2, 2026
Merged

test: add describe nodes to dump filed resourses#2042
universal-itengineer merged 9 commits intomainfrom
test/fix/add-describe-nodes-modern-test

Conversation

@universal-itengineer
Copy link
Member

@universal-itengineer universal-itengineer commented Mar 2, 2026

Description

Add describe for nodes when test fails.
Also increase timeout for UntilVMAgentReady in VirtualMachineConfiguration
The error output for the error in function UntilVMAgentReady has become clearer

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

We need information about the nodes when the test fails.

Sometimes the agent doesn't have time to start within 30 seconds. To achieve this, the timeout for the UntilVMAgentReady step was increased to 60 seconds in the Virtual Machine Configuration test.

The message "vm %s is not ready" is not entirely correct, as it should be a message about the agent not being ready.

What is the expected result?

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

Signed-off-by: Nikita Korolev <nikita.korolev@flant.com>
Signed-off-by: Nikita Korolev <nikita.korolev@flant.com>
…figuration

Signed-off-by: Nikita Korolev <nikita.korolev@flant.com>
@universal-itengineer universal-itengineer added this to the v1.7.0 milestone Mar 2, 2026
Signed-off-by: Nikita Korolev <nikita.korolev@flant.com>
@universal-itengineer universal-itengineer marked this pull request as ready for review March 2, 2026 15:18
universal-itengineer and others added 3 commits March 2, 2026 19:39
Co-authored-by: Roman Sysoev <36233932+hardcoretime@users.noreply.github.com>
Signed-off-by: Nikita Korolev <141920865+universal-itengineer@users.noreply.github.com>
Co-authored-by: Roman Sysoev <36233932+hardcoretime@users.noreply.github.com>
Signed-off-by: Nikita Korolev <141920865+universal-itengineer@users.noreply.github.com>
Signed-off-by: Nikita Korolev <nikita.korolev@flant.com>
Signed-off-by: Nikita Korolev <nikita.korolev@flant.com>
@universal-itengineer universal-itengineer merged commit 32e1db0 into main Mar 2, 2026
27 of 29 checks passed
@universal-itengineer universal-itengineer deleted the test/fix/add-describe-nodes-modern-test branch March 2, 2026 17:35
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