bump kube-openapi version

[filariow]

Description

Update kube-openapi version and removes Fake GOPATH implementation

Checks

1. Did you run make generate target? yes

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

3. In case of new CRD, did you the following? NA

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

[mfrancisc]

Thanks for updating the openapi tool 🙏
I've tested and it works fine for me.

My folder structure is pretty standard so let's wait for others to test it as well before merging.

[rajivnathan]

Nice, worked well for me too

[fbm3307]

Worked for me

[fbm3307]

Just a Thought , Do we need to discuss/resolve the update to kube-openapi version as i see PR https://github.com/codeready-toolchain/api/pull/409/files from @ranakan19 updating it to some different version , and here it updates to some latest version? wont that be conflicting or may be just duplication of work?

[filariow]

Great catch! As kube-openapi is a controller-runtime indirect dependency, maybe it's worth to merge the other PR first and then just remove the FAKE GOPATH logic with this one.

[ranakan19]

Thanks @fbm3307! I was thinking the same, but I wonder why there are no errors with the kube-openapi version you are upgrading to (which is the latest version) with the older controller-runtime version. Looking into this further.
As for merging the PR - I cannot merge the other PR untill all the other PRs across all repos are ready - so it may take another sprint or two to get to that stage. Would it be okay to wait?

[ranakan19]

Ah i see, the version you are updating to is the latest one from controller-runtime, not from the version of the controller-runtime we are currently on or would like to update it. It also has a higher version golang version. Since there is no error in upgrading to the latest version, because it is an indirect dependency, you could update the kube-openapi to this version if you need it for something. If not, I'd rather keep them updated in sync with the rest of the dependencies.
The version of kube-openapi for controller-runtime v0.15 is https://github.com/kubernetes-sigs/controller-runtime/blob/release-0.15/go.mod#L71

[filariow]

I agree, we should definitely keep them updated in sync. Unfortunately, we'll not being able to remove FAKE GOPATH implementation until -if I'm not mistaken- [email protected].

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

[fbm3307]

@filariow probably we should put the PR to draft and pick up later or close this ( as per the above discussion)