Skip to content

test(module): add test for discovery nodes hook #1154

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

test(module): add test for discovery nodes hook #1154

wants to merge 4 commits into from

Conversation

fl64
Copy link
Member

@fl64 fl64 commented Jun 11, 2025

Description

add test for discovery nodes hook

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

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

section: module
type: chore
summary: add test for discovery nodes hook
impact_level: low

@fl64 fl64 self-assigned this Jun 11, 2025
@fl64 fl64 changed the title test(module): test(module): add test for discovery nodes hook Jun 11, 2025
@fl64 fl64 force-pushed the test/module/add-test-for-discovery-nodes-hook branch 2 times, most recently from d9c3f7a to 8138101 Compare June 12, 2025 08:29
@fl64 fl64 added this to the v0.20.0 milestone Jun 12, 2025
@fl64 fl64 force-pushed the test/module/add-test-for-discovery-nodes-hook branch from 8138101 to bc07edd Compare June 12, 2025 08:30
@fl64
fix tests
612fc1c 
Signed-off-by: Pavel Tishkov <[email protected]>
@fl64 fl64 force-pushed the test/module/add-test-for-discovery-nodes-hook branch from 6fcadb9 to 612fc1c Compare June 13, 2025 14:10
@fl64
refactor
6f21a2b 
Signed-off-by: Pavel Tishkov <[email protected]>
@fl64 fl64 force-pushed the test/module/add-test-for-discovery-nodes-hook branch from da9287b to 6f21a2b Compare June 15, 2025 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@universal-itengineer universal-itengineer Awaiting requested review from universal-itengineer universal-itengineer will be requested when the pull request is marked ready for review universal-itengineer is a code owner

@nevermarine nevermarine Awaiting requested review from nevermarine nevermarine will be requested when the pull request is marked ready for review nevermarine is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@fl64 fl64

Labels
None yet
Projects
None yet
Milestone
v0.20.0
Development

Successfully merging this pull request may close these issues.
1 participant
@fl64