devfile / devworkspace-operator

Apache License 2.0
67 stars 55 forks source link

Update cpu and memory limit for devworkspace-controller #1298

Closed dkwon17 closed 3 months ago

dkwon17 commented 3 months ago

What does this PR do?

This PR updates the CPU and memory limit of the devworkspace-controller container of the devworkspace-controller-manager-* pod from 1000m to 3000m and 1Gi to 3Gi respectively.

What issues does this PR fix or reference?

Part of https://issues.redhat.com/browse/CRW-6889 Fix https://github.com/devfile/devworkspace-operator/issues/1299

Is it tested? How?

To test, run:

DWO_IMG_REPO=quay.io/<username>/<repo>
DWO_IMG_TAG=<tag>
export DWO_BUNDLE_IMG=${DWO_IMG_REPO}/devworkspace-operator-bundle:${DWO_IMG_TAG}
export DWO_INDEX_IMG=${DWO_IMG_REPO}/devworkspace-operator-index:${DWO_IMG_TAG}
export DOCKER=podman 
make generate_olm_bundle_yaml build_bundle_and_index register_catalogsource

And install the operator from OperatorHub.

Verify that the resulting devworkspace-controller-manager-* pod in the openshift-operators namespace has the new CPU and memory limit for the devworkspace-controller container.

PR Checklist

openshift-ci[bot] commented 3 months ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: AObuchow, dkwon17

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/devfile/devworkspace-operator/blob/main/OWNERS)~~ [AObuchow,dkwon17] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
AObuchow commented 3 months ago

@dkwon17 I've opened https://github.com/devfile/devworkspace-operator/issues/1299 -- this PR is now good to merge :) Great work as always.

dkwon17 commented 3 months ago

Thank you @AObuchow , merging