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

Merge kraft 2 #1

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

Merge kraft 2 #1

wants to merge 82 commits into from

Conversation

dvaseekara
Copy link
Owner

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) (adobe#62)
514fa07
@ctrlaltluc @amuraru
Fix flaky test by deleting nodeports explicitly (adobe#67)
16a9fc2
@ctrlaltluc @amuraru
Allow dashes when parsing broker rack (adobe#68)
cdfb6b9
Daniel Vaseekaran and others added 30 commits November 14, 2024 12:45
Fixed build issues
c1c0f02
Fix TestGenerateBrokerConfig
4e5f972
Added LoadBalancer for Kind E2E test cluster
0179d93
Added LoadBalancer for Kind E2E test cluster
f05f1ee
Added LoadBalancer for Kind E2E test cluster
3a8e73f
Added LoadBalancer for Kind E2E test cluster
8f15bec
Added LoadBalancer for Kind E2E test cluster
2da7c29
Added LoadBalancer for Kind E2E test cluster
a174dfb
Added LoadBalancer for Kind E2E test cluster
69634e1
Added LoadBalancer for Kind E2E test cluster
5faf114
Added LoadBalancer for Kind E2E test cluster
bd1ccb5
Added LoadBalancer for Kind E2E test cluster
deded52
pushing up latest go.sum
b428487
Merge branch 'master' of https://github.com/dvaseekara/koperator into…
1a07e63 
… merge-kraft-2
Merge branch 'merge-kraft-2' of https://github.com/dvaseekara/koperator…
7143005 
… into merge-kraft-2
Clean up Merge
38cf906
Enabling Tmate to debug e2e Test
e17bb79
Revert Cert Changes
5c5b19c
Revert "Revert Cert Changes"
8f5aa5d 
This reverts commit 5c5b19c.
Enable sslClientAuth
116c89e
trigger test
3b699c9
WIP: Fix Listener Config
615f46c
Clean up test case results - tc-1
7fc9ebf
Clean up test case results - tc-2
251679a
Updated Kraft Test Cases
0943500
Cleanup Linting Issues
d691828
Remove Tmate Debugger
c99efe2
Run Kraft CLuster E2E
8fc0f46
Add kraft e2e test
2fa3aea
Revert Test
5b721e5
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.
6 participants
@dvaseekara @panyuenlau @ctrlaltluc @amuraru @dobrerazvan @cristianpetrache