fix: Use cached and non-cached clients in backup/restore controllers

[mmorhun]

Signed-off-by: Mykola Morhun [email protected]

What does this PR do?

Uses two k8s clients in backup and restore controllers: caching client and non-caching client.

Screenshot/screencast of this PR

N/A

What issues does this PR fix or reference?

A step towards eclipse-che/che#20647

How to test this PR?

1. Run chectl server:deploy --installer=operator --platform=openshift
2. Run chectl server:backup
3. Run chectl server:restore --snapshot-id=latest
4. Check Che is ready and works

PR Checklist

As the author of this Pull Request I made sure that:

• The Eclipse Contributor Agreement is valid
• Code produced is complete
• Code builds without errors
• Tests are covering the bugfix
• Custom resource definition file is up to date
• OLM bundles are up to date
• The repository devfile is up to date and works
• Sections What issues does this PR fix or reference and How to test this PR completed
• Relevant user documentation updated
• Relevant contributing documentation updated
• CI/CD changes implemented, documented and communicated

Reviewers

Reviewers, please comment how you tested the PR when approving it.

[openshift-ci]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: AndrienkoAleksandr, mmorhun, tolusha

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

Needs approval from an approver in each of these files:

• OWNERS

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

[l0rd]

@mmorhun @tolusha shouldn't we decrease the operator memory limit now that this PR has been merged? If not how can we justify the need for 5GB?
Update: sorry I just saw your comment that it's just one step forward the solution...