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

cherry-pick yurthub code of pool-coordinator-dev #1156

Merged
merged 20 commits into from
Jan 18, 2023
Merged
Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
16a399e
add yurthub leader election and coordinator framework (#1035)
rambohe-ch Nov 10, 2022
3e69468
pool-coordinator implementation of yurthub (#1073)
Congrool Dec 7, 2022
0e6b9d6
generate coordinator client and pass through SubjectAccessReview acco…
Congrool Jan 4, 2023
7ae8fbb
Fix (#1125)
LaurenceLiZhixin Jan 4, 2023
2bff621
set isPoolCacheSynced as true if lease updated (#1131)
Congrool Jan 10, 2023
39029b3
bugfix: do not send pool-scoped resource list/watch request to pool-c…
Congrool Jan 11, 2023
0a8659c
Fix: pool-coordinator (#1126)
LaurenceLiZhixin Jan 11, 2023
2c1d57b
Bugfix of pool-coordinator and enable unit test to pass (#1137)
Congrool Jan 11, 2023
948ce9f
bugfix: return fake getter if poolcoordinator is not enabled (#1138)
Congrool Jan 12, 2023
ce76fda
mocked etcd client should not use copied locks (#1139)
Congrool Jan 12, 2023
512cec6
add unit test for poolcoordinator cert manager (#1140)
Congrool Jan 12, 2023
edf7ccd
Fix (#1141)
LaurenceLiZhixin Jan 12, 2023
68b3de9
fix coordinator sync problem and update ready condition (#1142)
Congrool Jan 12, 2023
157ac45
Fix: dns bug for endpointslices v1beta1 version (#1144)
LaurenceLiZhixin Jan 13, 2023
7886dbc
Fix: set inElecting to false when handling follow hub, and add metric…
LaurenceLiZhixin Jan 13, 2023
5781203
fix coredns cannot get /apis/discovery.k8s.io/v1 if cluster does not …
Congrool Jan 16, 2023
735b40a
Fix: add PoolScopeResource validation and dynamic configuration suppo…
LaurenceLiZhixin Jan 17, 2023
e84c4f6
bugfix: ReplaceComponentList should not delete resources of other gvr…
Congrool Jan 18, 2023
9bc4d1c
fix go.mod
Congrool Jan 17, 2023
634409d
fix unit test of yurthub options
Congrool Jan 17, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix (#1125)
LaurenceLiZhixin authored and Congrool committed Jan 17, 2023
commit 7ae8fbb9df078325c61af194e75d8fd0373c37cc
3 changes: 1 addition & 2 deletions cmd/yurthub/app/start.go
Original file line number Diff line number Diff line change
@@ -234,7 +234,6 @@ func coordinatorRun(ctx context.Context,
var coordinatorHealthChecker healthchecker.HealthChecker
var coordinatorTransportMgr transport.Interface
var coordinator poolcoordinator.Coordinator
var coordinatorCertManager *coordinatorcertmgr.CertManager
var returnErr error

readyCh := make(chan struct{})
@@ -247,7 +246,7 @@ func coordinatorRun(ctx context.Context,
return
}

coorTransportMgr, err := poolCoordinatorTransportMgrGetter(cfg.HeartbeatTimeoutSeconds, cfg.CoordinatorServerURL, coordinatorCertManager, ctx.Done())
coorTransportMgr, err := poolCoordinatorTransportMgrGetter(cfg.HeartbeatTimeoutSeconds, cfg.CoordinatorServerURL, coorCertManager, ctx.Done())
if err != nil {
returnErr = fmt.Errorf("failed to create coordinator transport manager, %v", err)
return