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

Optimize resource sharing with concurrent reconciles #643

Draft
wants to merge 25 commits into
base: main
Choose a base branch
from
Draft

Optimize resource sharing with concurrent reconciles #643

wants to merge 25 commits into from

Conversation

kabicin
Copy link
Member

@kabicin kabicin commented Oct 24, 2024 •
edited
Loading

What this PR does / why we need it?:

  • Optimizes resource sharing (LTPA/Password Encryption keys) by caching the DecisionTree into a global *sync.Map.
  • Enables MaxConcurrentReconciles by making the operator config map and resource sharing CR instances threadsafe.

Does this PR introduce a user-facing change?

  • User guide
  • CHANGELOG.md

Which issue(s) this PR fixes:

Fixes #

@kabicin
Check for outdated app image reference in LTPA Job
95539a0
@kabicin
Create cache.go
7a15f27
@kabicin
Pull RCO max concurrent reconcile changes
4019039
@kabicin
Pull RCO concurrent reconcile fix
991a466
@kabicin
Add DecisionTree cache logic + temp flag
4a63aba
@kabicin
Set reconcilers to block for ResourceSharing mutex
02622ce
@kabicin
Operator config map to retry reconcile when failed to get
646c84e
@kabicin
Add experimental CR config
7d800a6
@kabicin
Create concurrent reconcile func
0ca4251
@kabicin
Concurrently reconcile semeru cloud compiler
d20bbbd
@kabicin
Update experimental.go
4e9dbb4
@kabicin
Concurrently reconcile deployment/statefulset
e5bea14
@kabicin
Concurrently reconcile routes/ingress
1bf4760
@kabicin
Limit use of instance mutex
14f8ec2
@kabicin
Merge branch 'main' into ltpa-opti
1ffab6a
@kabicin
Set default value for passwordEncryptionMetadata
3219346
@kabicin
Merge branch 'ltpa-opti' of https://github.com/OpenLiberty/open-liber…
3afb72b 
…ty-operator into ltpa-opti
@kabicin
Update experimental.go
554a2cb
@kabicin
Add namespace lock
42001b2
@kabicin
Unlock password encryption mutex
41d90a5
@kabicin
Add debug stmt
64e7f64
@kabicin
Set passwordEncryptionMetadata to not nil
71d9131
@kabicin
remove namespace mutex
5ddb916
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.
1 participant
@kabicin