Skip to content

Add resource limits to Smart Gateway containers#199

Open
danpawlik wants to merge 2 commits into
infrawatch:masterfrom
danpawlik:add-resource-limit
Open

Add resource limits to Smart Gateway containers#199
danpawlik wants to merge 2 commits into
infrawatch:masterfrom
danpawlik:add-resource-limit

Conversation

@danpawlik

@danpawlik danpawlik commented Jul 15, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Set CPU/memory requests and limits on all three containers (oauth-proxy, bridge, sg-core) in the generated Smart Gateway deployment to prevent unbounded resource consumption.

Related-To: OSPRH-32349

@danpawlik
Add resource limits to Smart Gateway containers
2a4aaa3 
Set CPU/memory requests and limits on all three containers
(oauth-proxy, bridge, sg-core) in the generated Smart Gateway
deployment to prevent unbounded resource consumption.

Signed-off-by: Daniel Pawlik <dpawlik@redhat.com>
@danpawlik
danpawlik requested a review from a team July 15, 2026 05:39
vkmc
vkmc reviewed Jul 15, 2026
View reviewed changes

@vkmc vkmc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, but for this one we should test we can update the memory limits on the smartgateway pods (if required)

@danpawlik
danpawlik requested a review from a team July 15, 2026 08:30
@danpawlik danpawlik mentioned this pull request Jul 16, 2026
Merged
@centosinfra-prod-github-app

centosinfra-prod-github-app Bot commented Jul 17, 2026

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/a3a7bb2b432345988762be3b64cda555

✔️ stf-crc-ocp_418-local_build SUCCESS in 50m 13s
✔️ stf-crc-ocp_420-local_build SUCCESS in 51m 05s
✔️ stf-crc-ocp_418-local_build-index_deploy SUCCESS in 58m 13s
✔️ stf-crc-ocp_420-local_build-index_deploy SUCCESS in 59m 49s
✔️ stf-crc-ocp_418-nightly_bundles-index_deploy SUCCESS in 45m 44s
✔️ stf-crc-ocp_420-nightly_bundles-index_deploy SUCCESS in 46m 23s
❌ stf-crc-ocp_418-catalog_deploy POST_FAILURE in 44m 04s
✔️ stf-crc-ocp_420-catalog_deploy SUCCESS in 41m 43s

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

Reviewers

@vkmc vkmc vkmc left review comments

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@danpawlik @vkmc