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

fix: charts java memory options passed #8564

Merged
merged 4 commits into from
May 27, 2024
Merged

fix: charts java memory options passed #8564

merged 4 commits into from
May 27, 2024

Conversation

moabu
Copy link
Member

@moabu moabu commented May 22, 2024

Prepare


Description

Target issue

closes #8563

Signed-off-by: moabu <47318409+moabu@users.noreply.github.com>
@moabu moabu requested a review from iromli as a code owner May 22, 2024 12:30
@mo-auto mo-auto added comp-charts-jans kind-bug Issue or PR is a bug in existing functionality labels May 22, 2024
moabu added 2 commits May 22, 2024 15:42
Signed-off-by: moabu <47318409+moabu@users.noreply.github.com>
Signed-off-by: moabu <47318409+moabu@users.noreply.github.com>
@iromli
Copy link
Contributor

iromli commented May 23, 2024

@moabu we're lowering the memory allocation almost 50% as seen below (for example):

env:
  - name: CN_SCIM_JAVA_OPTIONS                                                                                                                                            
    value: "-XX:MaxDirectMemorySize=492m -Xmx612m"                                                                                                                        
    
resources:
  limits:                                                                                                                                                                 
    cpu: 1000m                                                                                                                                                            
    memory: 1200Mi                                                                                                                                                        
  requests:                                                                                                                                                               
    cpu: 1000m                                                                                                                                                            
    memory: 1200Mi            

@moabu moabu merged commit 4226f2a into main May 27, 2024
2 checks passed
@moabu moabu deleted the fix-chart-java-option branch May 27, 2024 11:42
yuriyz pushed a commit that referenced this pull request Nov 7, 2024
* fix: memory options

Signed-off-by: moabu <47318409+moabu@users.noreply.github.com>

* fix: use percentages

Signed-off-by: moabu <47318409+moabu@users.noreply.github.com>

* fix: use mulf for floats percentages

Signed-off-by: moabu <47318409+moabu@users.noreply.github.com>

---------

Signed-off-by: moabu <47318409+moabu@users.noreply.github.com>
Former-commit-id: 4226f2a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp-charts-jans kind-bug Issue or PR is a bug in existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix(charts): passing in calculated Java options
3 participants