Skip to content
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

KIP 848: Extend DescribeConfigs and IncrementalAlterConfigs to support GROUP Config #2366

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

PratRanj07
Copy link
Contributor

@PratRanj07 PratRanj07 commented Nov 26, 2024

This PR intends to add support for Group Resource type for IncrementalAlterConfigs API as specified in KIP 848 and extended the DescribeConfigs support till API version 3.
Additional changed the integration test to check for Group Resource Type.
Librdkafka PR: confluentinc/librdkafka#4939

I have tested it locally and it is working fine
dotnet test --filter "FullyQualifiedName=Confluent.Kafka.IntegrationTests.Tests.AdminClient_IncrementalAlterConfigs"

Passed! - Failed: 0, Passed: 1, Skipped: 0, Total: 1, Duration: < 1 ms - Confluent.Kafka.IntegrationTests.dll (net6.0)

Passed! - Failed: 0, Passed: 1, Skipped: 0, Total: 1, Duration: < 1 ms - Confluent.Kafka.IntegrationTests.dll (net8.0)

@PratRanj07 PratRanj07 requested review from a team as code owners November 26, 2024 05:42
@confluent-cla-assistant
Copy link

🎉 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.

@PratRanj07 PratRanj07 changed the title Add integration test for config apis group support Extend DescribeConfigs and IncrementalAlterConfigs to support GROUP Config Dec 19, 2024
@PratRanj07 PratRanj07 changed the title Extend DescribeConfigs and IncrementalAlterConfigs to support GROUP Config KIP 848: Extend DescribeConfigs and IncrementalAlterConfigs to support GROUP Config Dec 19, 2024
@airlock-confluentinc airlock-confluentinc bot force-pushed the dev_check_group_support branch from 49afa24 to 24c1c2b Compare December 26, 2024 10:06
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.

1 participant