Skip to content

Optimize resource sharing with concurrent reconciles #643

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
Draft
wants to merge 37 commits into
base: main
Choose a base branch
from
Draft

Optimize resource sharing with concurrent reconciles #643

wants to merge 37 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 added 29 commits November 26, 2024 11:12
@kabicin
Check for outdated app image reference in LTPA Job
7e7ca2a
@kabicin
Create cache.go
ad90a5e
@kabicin
Pull RCO max concurrent reconcile changes
c58f65d
@kabicin
Add DecisionTree cache logic + temp flag
ecceeb2
@kabicin
Set reconcilers to block for ResourceSharing mutex
6837156
@kabicin
Operator config map to retry reconcile when failed to get
fd4beb9
@kabicin
Add experimental CR config
4a270d9
@kabicin
Create concurrent reconcile func
43b93ee
@kabicin
Concurrently reconcile semeru cloud compiler
df17b93
@kabicin
Update experimental.go
9eac1cc
@kabicin
Concurrently reconcile deployment/statefulset
d15b0b2
@kabicin
Concurrently reconcile routes/ingress
35c7ab3
@kabicin
Limit use of instance mutex
5bd67ae
@kabicin
Set default value for passwordEncryptionMetadata
b0014d9
@kabicin
Update experimental.go
fe0872f
@kabicin
Add namespace lock
53b14bd
@kabicin
Unlock password encryption mutex
8413bf7
@kabicin
Add debug stmt
eb7193b
@kabicin
Set passwordEncryptionMetadata to not nil
1dec781
@kabicin
remove namespace mutex
ab0a27e
@kabicin
Move long tasks to begin concurrent reconcile
5d45323
@kabicin
set 4 max concurrent reconciles
1847706
@kabicin
Update openlibertyapplication_controller.go
ca96620
@kabicin
Update openlibertyapplication_controller.go
5bf5422
@kabicin
Add manageErroingInstances
18579e4
@kabicin
Provide detailed error message and cleanup
d17142d
@kabicin
Perform service certificate check with LTPA in concurrent reconcile
f4dc31d
@kabicin
Update openlibertyapplication_controller.go
e91adc4
@kabicin
Create instances sync.Map
ce75d6a
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