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.

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

Kraft #74

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

Kraft #74

wants to merge 50 commits into from

Conversation

dobrerazvan
Copy link

Description

Please provide a meaningful description of what this change will do, or is for. Bonus points for including links to
related issues, other PRs, or technical references.

Note that by not including a description, you are asking reviewers to do extra work to understand the context of this
change, which may lead to your PR taking much longer to review, or result in it not being reviewed at all.

Type of Change

  • Bug Fix
  • New Feature
  • Breaking Change
  • Refactor
  • Documentation
  • Other (please describe)

Checklist

  • I have read the contributing guidelines
  • Existing issues have been referenced (where applicable)
  • I have verified this change is not present in other open pull requests
  • Functionality is documented
  • All code style checks pass
  • New code contribution is covered by automated tests
  • All new and existing tests pass

panyuenlau and others added 30 commits July 25, 2023 13:39
@panyuenlau
Add koperator/api changes for KRaft support
dfc0824
@panyuenlau
Merge branch 'master' into kraft-api
722fd96
@panyuenlau
Merge branch 'master' into kraft-api
30427bf
@panyuenlau
Move processRoles under brokerConfig
54af705
@panyuenlau
Update comments for ZK-relevant configurations in kafkacluster_types.go
dbf3200
@panyuenlau
Rebase origin/kraft-api
219d998
@panyuenlau
Support running Kafka cluster in KRaft mode
314ea41
@panyuenlau
brokerRoles -> processRoles to match upstream Kafka naming
7613f06
@panyuenlau
Update pod start-up process so pods can be restarted during rolling u…
aca2396 
…pgrade triggered by controller addition/removal
@panyuenlau
Rebase from origin/kraft-api
c5a4819
@panyuenlau
Update exsiting integration tests and func signatures
5d1b777
@panyuenlau
Fix broker configurations; add unit tests for broker configurations u…
aa4f5f5 
…nder KRaft mode
@panyuenlau
Remove unnecessary method from koperator/api
7431c93
@panyuenlau
Extend integration tests to cover KRaft mode
2e3be06
@panyuenlau
make lint-fix
78e63ee
@panyuenlau
Update static KafkaCluster yamls; add check for kraft mode before set…
3b5aff9 
…ting ClusterID in status
@panyuenlau
Rebase from origin/koperator-api
f9aedac
@panyuenlau
Use util functions that got moved to the koperator/api module
5804835
@panyuenlau
Remove unineteded changes during rebase
ca16422
@panyuenlau
Do not take active controller identity into consideration when reorde…
afd567b 
…r the brokers
@panyuenlau
Update implementation to accomomdate the latest KafkaCluster API change
bbc0307
@panyuenlau
Make comments about ZK-relevant configurations more clear
455ef3f
@ctrlaltluc @panyuenlau
Add ConcurrentBrokerRestartCountPerRack to RollingUpgradeConfig (banz…
6b3d616 
…aicloud#1002)

* Add ConcurrentBrokerRestartCountPerRack to RollingUpgradeConfig
@panyuenlau
Small refactoring
0a90251
@panyuenlau
Merge branch 'master' into kraft
63e15fa
@panyuenlau
Exclude controller-only nodes from all CC operations
aa859a1
@panyuenlau
Add processRoles label key for Kafka pods in KRaft mode; export consts
28a4a75
@amuraru
Allow concurrent broker restarts from same AZ (broker rack) (#62)
514fa07
@ctrlaltluc @amuraru
Fix flaky test by deleting nodeports explicitly (#67)
16a9fc2
@ctrlaltluc @amuraru
Allow dashes when parsing broker rack (#68)
cdfb6b9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@dobrerazvan @panyuenlau @ctrlaltluc @amuraru @cristianpetrache