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 #1344

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

Conversation

PratRanj07
Copy link
Contributor

@PratRanj07 PratRanj07 commented Nov 21, 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

@PratRanj07 PratRanj07 requested review from a team as code owners November 21, 2024 07:02
@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 to check for added group support in config apis 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
@sonarqube-confluent
Copy link

Passed

Analysis Details

2 Issues

  • Bug 0 Bugs
  • Vulnerability 0 Vulnerabilities
  • Code Smell 2 Code Smells

Coverage and Duplications

  • Coverage No coverage information (58.90% Estimated after merge)
  • Duplications No duplication information (0.00% Estimated after merge)

Project ID: confluent-kafka-go

View in SonarQube

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