Skip to content

[KIP-932] Add share consumer workflow tests 4/N#2284

Merged
Kaushik Raina (k-raina) merged 3 commits into
masterfrom
dev_kip-932_queues-for-kafka_add_tests_poll_4_of_N
Jun 24, 2026
Merged

[KIP-932] Add share consumer workflow tests 4/N#2284
Kaushik Raina (k-raina) merged 3 commits into
masterfrom
dev_kip-932_queues-for-kafka_add_tests_poll_4_of_N

Conversation

@k-raina

Copy link
Copy Markdown
Member

Summary

  • Add tests for ack and commit workflows

@confluent-cla-assistant

Copy link
Copy Markdown

🎉 All Contributor License Agreements have been signed. Ready to merge.
Please push an empty commit if you would like to re-run the checks to verify CLA status for all contributors.

Base automatically changed from dev_kip-932_queues-for-kafka to master June 23, 2026 08:53
@airlock-confluentinc airlock-confluentinc Bot force-pushed the dev_kip-932_queues-for-kafka_add_tests_poll_4_of_N branch from 3713734 to 68eb6fd Compare June 24, 2026 08:18
Copilot AI review requested due to automatic review settings June 24, 2026 08:18
@k-raina Kaushik Raina (k-raina) removed the request for review from Copilot June 24, 2026 08:18
Copilot AI review requested due to automatic review settings June 24, 2026 09:59
@k-raina Kaushik Raina (k-raina) removed the request for review from Copilot June 24, 2026 09:59
try:
set_group_config(kafka_cluster, group_id, 'share.isolation.level', 'read_committed')
except KafkaException as e:
pytest.skip(f'cannot set share.isolation.level on group: {e}')

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

One thing the brokers above 4.2.0 allow this and I think all the share consumer tests should run after that only! So in that case I dont think we need these pytest.skips?

Copilot AI review requested due to automatic review settings June 24, 2026 12:38
@sonarqube-confluent

Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@k-raina Kaushik Raina (k-raina) merged commit 2f9e617 into master Jun 24, 2026
3 checks passed
@k-raina Kaushik Raina (k-raina) deleted the dev_kip-932_queues-for-kafka_add_tests_poll_4_of_N branch June 24, 2026 14:27
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