Dropping the unused parameters

[fbm3307]

Description

This is to Drop unused parameters from the embedded MUR.Status.UserAccounts.Clusters type

Checks

1. Did you run make generate target? yes

2. Did make generate change anything in other projects (host-operator, member-operator)? yes

3. In case of new CRD, did you the following? yes/no

   • make/generate.mk in this repository
   • resources/setup/roles/host.yaml in the sandbox-sre repository
   • PROJECT file: https://github.com/codeready-toolchain/host-operator/blob/master/PROJECT
   • CSV file: https://github.com/codeready-toolchain/host-operator/blob/master/config/manifests/bases/host-operator.clusterserviceversion.yaml

4. In case other projects are changed, please provides PR links.

   • host-operator: Dropping the unused parameters host-operator#925
   • toolchain e2e - Dropping the unused parameters toolchain-e2e#855

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[alexeykazakov]

Hm... I wonder why there is no changes in api/v1alpha1/zz_generated.deepcopy.go
Don't you have that file changed after running make generate?

[fbm3307]

Hm... I wonder why there is no changes in api/v1alpha1/zz_generated.deepcopy.go Don't you have that file changed after running make generate?

Maybe because the deepcopy happens on whole struct and here i am just removing the parameters. ?
There were no changes to api/v1alpha1/zz_generated.deepcopy.go when i ran make generate.

[alexeykazakov]

Yeah.. probably because these fields are strings. Not structs.

[MatousJobanek]

Yeah.. probably because these fields are strings. Not structs.

yup, I was looking at it too. The logic of the deepcopy is pretty generic and for the strings in the struct it's sufficient to just assign the whole value of the struct from the original one.