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

feat: InformerManager and FederatedInformerManager #141

Merged
merged 25 commits into from
Jul 13, 2023
Merged

feat: InformerManager and FederatedInformerManager #141

merged 25 commits into from
Jul 13, 2023

Conversation

limhawjia
Copy link
Collaborator

No description provided.

SOF3
SOF3 reviewed Jul 10, 2023
View reviewed changes
pkg/util/tools/bijection.go Outdated Show resolved Hide resolved
SOF3
SOF3 reviewed Jul 10, 2023
View reviewed changes
pkg/util/tools/bijection.go Outdated Show resolved Hide resolved
SOF3
SOF3 reviewed Jul 10, 2023
View reviewed changes
pkg/util/tools/bijection.go Outdated Show resolved Hide resolved
@limhawjia limhawjia force-pushed the feat-federated-informer-manager-2 branch from 032cc43 to 14faa29 Compare July 12, 2023 04:25
@limhawjia
feat: InformerManager and FederatedInformerManager
8855579
@limhawjia
implement informermanager and bootstrap unit tests
2913975
@limhawjia
start federatedinformer implementation
fde4b01
@limhawjia
add bijection map to handle source type changes
a522520
@limhawjia
finish implementation of federatedinformermanager
fa0020d
@limhawjia
change api of EventHandlerGenerators
fc7cfbc
@limhawjia
fix bug
a917640
@limhawjia
finish informermanager tests
0e9f2ba
@limhawjia
refactor unit tests for informermanager
eedebb5
@limhawjia
fix logger keys, use ratelimiting and other fixes
d505f2c
@limhawjia
propagate start contexts
d533029
@limhawjia
refactor bijection package
507bd3f
@limhawjia
improve countingResourceEventHandler
ab92ccd
@limhawjia
finish federatedinformermanager tests
3e4945d
@limhawjia
fix typos
9ce179b
@limhawjia
fix lint errors
65f8ff1
@limhawjia
fix lint errors and goroutine leak in tests
8b2db19
@limhawjia
reset rate limiter on succesful worker reconciles
50591a5
@limhawjia
fix race
f7edcf6
@limhawjia
fix race
ff75595
@limhawjia
uncomment files
b01676d
@limhawjia
uncomment files
0d2fb00
@limhawjia
add utils to inject klog values in context
89e78e4
@limhawjia
reset rate limiter on no reenqueue
a457bd4
@limhawjia limhawjia force-pushed the feat-federated-informer-manager-2 branch from e045fc6 to a457bd4 Compare July 13, 2023 06:10
@gary-lgy gary-lgy merged commit f9f2637 into kubewharf:refactor/unified-federated-type Jul 13, 2023
2 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SOF3 SOF3 SOF3 left review comments

@gary-lgy gary-lgy gary-lgy approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@limhawjia @SOF3 @gary-lgy