feat: add new registration-service config param for ui canary deployement

[mfrancisc]

This PR adds new UICanaryDeploymentWeight in the toolchainconfig fields.

As a side effect it also updates the controller-runtime version in the CRDs.

Related to:
api: codeready-toolchain/api#476

Jira: https://issues.redhat.com/browse/SANDBOX-1144

[alexeykazakov]

As a side effect it also updates the controller-runtime version in the CRDs.

But why? Why would codeready-toolchain/api#476 trigger such controller-runtime update?

[mfrancisc]

But why? Why would codeready-toolchain/api#476 trigger such controller-runtime update?

Have we maybe updated the controller-runtime without updating the version in the CRDS ?

[mfrancisc]

I see the controller-gen update here : https://github.com/codeready-toolchain/host-operator/pull/1160/files , but I don't see the update to the CRDs, so I think we haven't ran make generate in api after the upgrade . Does it make sense ? cc @rsoaresd .

[alexeykazakov]

@mfrancisc I just tried to generated a host PR for your API change: #1174

[mfrancisc]

This one looks stuck on the e2e since hours now 😕

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[alexeykazakov]

OK. I see that after Rafaela merged the 4.18 updates your PR is clean now too. Feel free to merge this PR or #1174 (which are now identical). Whichever is green first :)

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: alexeykazakov, fbm3307, MatousJobanek, mfrancisc

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 [MatousJobanek,alexeykazakov,fbm3307,mfrancisc]

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

[mfrancisc]

yeah, let's merge the one that is ready first. This one was stuck with pending e2e tests for several hours today.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.53%. Comparing base (c137181) to head (d3cd019).
Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #1172   +/-   ##
=======================================
  Coverage   83.53%   83.53%           
=======================================
  Files          82       82           
  Lines        7927     7927           
=======================================
  Hits         6622     6622           
  Misses       1104     1104           
  Partials      201      201           

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.