Skip to content

Conversation

@Jonsy13
Copy link
Contributor

@Jonsy13 Jonsy13 commented Jun 8, 2021

Signed-off-by: Jonsy13 vedant.shrotria@chaosnative.com

What this PR does / why we need it:
This PR will add resources requirements for control-plane components for Portal ( Platforms used - GKE & KIND )

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #

Special notes for your reviewer:

Checklist:

  • Fixes #
  • Signed the commit for DCO to be passed
  • Labelled this PR & related issue with documentation tag

Data gathered from VPA -

// Frontend
    Container Name:  litmusportal-frontend
        Lower Bound:
            Cpu:     25m
            Memory:  262144k
        Target:
            Cpu:     25m
            Memory:  262144k
        Uncapped Target:
            Cpu:     25m
            Memory:  262144k
        Upper Bound:
            Cpu:     478m
            Memory:  499755528

// Backend
    Container Name:  graphql-server
        Lower Bound:
            Cpu:     12m
            Memory:  131072k
        Target:
            Cpu:     12m
            Memory:  131072k
        Uncapped Target:
            Cpu:     12m
            Memory:  131072k
        Upper Bound:
            Cpu:           449m
            Memory:        3828582077
    Container Name:  auth-server
        Lower Bound:
            Cpu:     12m
            Memory:  131072k
        Target:
            Cpu:     12m
            Memory:  131072k
        Uncapped Target:
            Cpu:     12m
            Memory:  131072k
        Upper Bound:
            Cpu:     451m
            Memory:  471500k

// MongoDB
    Container Name:  mongo
        Lower Bound:
            Cpu:     25m
            Memory:  262144k
        Target:
            Cpu:     25m
            Memory:  262144k
        Uncapped Target:
            Cpu:     25m
            Memory:  262144k
        Upper Bound:
            Cpu:     379m
            Memory:  6310616433

Signed-off-by: Jonsy13 <vedant.shrotria@chaosnative.com>
@Jonsy13 Jonsy13 added the documentation Improvements or additions to documentation label Jun 8, 2021
@Jonsy13 Jonsy13 self-assigned this Jun 8, 2021
Jonsy13 added 2 commits June 8, 2021 10:42
Signed-off-by: Jonsy13 <vedant.shrotria@chaosnative.com>
Signed-off-by: Jonsy13 <vedant.shrotria@chaosnative.com>

### Control Plane Resource Requirements

The Resource requests and limits provided here have been calculated using data gathered by manually using a command `kubectl top` and Vertical-Pod-Autoscaler & also through recommendation by a great utility Goldilocks.The resources are getting monitored continuously and the information below will be updated as the metrics changes.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can restructure this a bit to provide more detail on the resource req estimation process.

### Control Plane Resource Requirements

The Resource requests and limits provided here have been calculated using data gathered by manually using a command `kubectl top` and Vertical-Pod-Autoscaler & also through recommendation by a great utility Goldilocks.The resources are getting monitored continuously and the information below will be updated as the metrics changes.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we can use some buffer units as well

Jonsy13 and others added 2 commits June 11, 2021 10:10
Signed-off-by: Jonsy13 <vedant.shrotria@chaosnative.com>
@Jonsy13 Jonsy13 merged commit 6dbda90 into litmuschaos:master Jun 15, 2021
@Jonsy13
Copy link
Contributor Author

Jonsy13 commented Jun 15, 2021

Merging this PR for this release. cc: @ksatchit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants