Skip to content

KAFKA-19316: added share_group_command_test.py system tests #19774

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.

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

Open
wants to merge 2 commits into
base: trunk
Choose a base branch
from

Conversation

chirag-wadhwa5
Copy link
Contributor

This PR include system tests in the file share_group_command_test.py. These tests test the functionality of kafka-share-groups.sh tool

@github-actions github-actions bot added triage PRs from the community tests Test fixes (including flaky tests) labels May 20, 2025
@AndrewJSchofield AndrewJSchofield added KIP-932 Queues for Kafka ci-approved and removed triage PRs from the community labels May 20, 2025
@AndrewJSchofield AndrewJSchofield self-requested a review May 20, 2025 22:20
if group:
if describe_members:
def has_expected_share_group_member():
output = self.kafka.describe_share_group_members(group=group,node=kafka_node,command_config=command_config_file)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: some formatting issues - no space after ,

def has_expected_share_group_member():
output = self.kafka.describe_share_group_members(group=group,node=kafka_node,command_config=command_config_file)
return len(output) == 1 and all("test-share-group" in line for line in output)
wait_until(has_expected_share_group_member,timeout_sec=10, err_msg="Timed out waiting to describe members of the share group.")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same here

Copy link
Collaborator

@smjn smjn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good overall,
some nit comments

@chirag-wadhwa5 chirag-wadhwa5 requested a review from smjn May 21, 2025 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-approved KIP-932 Queues for Kafka tests Test fixes (including flaky tests)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants