Skip to content

Conversation

@arms11
Copy link
Contributor

@arms11 arms11 commented Feb 14, 2021

Closes #5
The values are added based on production monitoring average base on during peak/off-peak hours.
Please note: While I have not been able to test this in minikube due to unrelated reasons, below is the resource utilization of the 3 nodes cluster with t3.large (2 CPUs/8GB Memory) spec.

NAME                             CPU(cores)   CPU%   MEMORY(bytes)   MEMORY%
ip-192-168-137-84.ec2.internal   428m         22%    2587Mi          36%
ip-192-168-171-99.ec2.internal   279m         14%    3048Mi          42%
ip-192-168-99-86.ec2.internal    237m         12%    2640Mi          37%

@pull-request-size pull-request-size bot added the size/M PR that changes 30-99 lines. Good size to review. label Feb 14, 2021
@arms11
Copy link
Contributor Author

arms11 commented Feb 14, 2021

@armab - Please review/guide. Thanks.

Copy link
Member

@arm4b arm4b left a comment

Choose a reason for hiding this comment

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

Thanks for sharing the data here based on using the StackStorm-HA cluster in production 👍

Please also add a Changelog record and we should be good to go!

@arm4b arm4b added the enhancement New feature or request label Feb 15, 2021
@arms11 arms11 force-pushed the resource-requests-spec-for-st2-pods branch from e67476b to 954ee4b Compare February 15, 2021 20:39
@arms11 arms11 marked this pull request as ready for review February 15, 2021 20:56
@arms11
Copy link
Contributor Author

arms11 commented Feb 15, 2021

Added Changelog entry and opened the pull request. Thanks.

Copy link
Member

@arm4b arm4b left a comment

Choose a reason for hiding this comment

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

That worked well! 👍

Thanks for sharing the st2 pod usage data from the prod environment and codifying it here.
Tha'll help with proper resource allocation and scheduling in the wild.

@arm4b arm4b merged commit 097d4f7 into StackStorm:master Feb 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request size/M PR that changes 30-99 lines. Good size to review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Find resource Requests for all K8s pods/deployments

2 participants