Skip to content

Conversation

@anandrkskd
Copy link
Contributor

What type of PR is this?
/kind enhancement

What does this PR do / why we need it:
This PR adds the momory optimization changes (from PR) used on upstream argocd-operator to reduce the memory usage by caching only necessary config-maps and secret.

Have you updated the necessary documentation?

  • Documentation update is required by this PR.
  • Documentation has been updated.

Which issue(s) this PR fixes:

Fixes #?

Test acceptance criteria:

  • Unit Test
  • E2E Test

How to test changes / Special notes to the reviewer:

@openshift-ci
Copy link

openshift-ci bot commented Sep 22, 2025

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@anandrkskd
Copy link
Contributor Author

/retest

@svghadi
Copy link
Member

svghadi commented Nov 10, 2025

/lgtm

@anandrkskd
Copy link
Contributor Author

image memory consumption with 100 secret and 100 configmaps, it remained under 200M.

@svghadi svghadi changed the title Downstream memory optimization featDownstream memory optimization Nov 11, 2025
@svghadi svghadi changed the title featDownstream memory optimization feat: Downstream memory optimization Nov 11, 2025
@svghadi
Copy link
Member

svghadi commented Nov 11, 2025

/approve

@openshift-ci
Copy link

openshift-ci bot commented Nov 11, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: svghadi

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

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot openshift-merge-bot bot merged commit 913ab87 into redhat-developer:master Nov 11, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants