Bug 1850089
| Summary: | OBC CRD is outdated and leads to missing columns in get queries | ||
|---|---|---|---|
| Product: | [Red Hat Storage] Red Hat OpenShift Container Storage | Reporter: | Ben Eli <belimele> |
| Component: | ocs-operator | Assignee: | Jose A. Rivera <jarrpa> |
| Status: | CLOSED ERRATA | QA Contact: | Elad <ebenahar> |
| Severity: | medium | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | 4.5 | CC: | dzaken, ebenahar, etamir, jalbo, jarrpa, jthottan, madam, muagarwa, nbecker, ocs-bugs, sostapov |
| Target Milestone: | --- | Flags: | dzaken:
needinfo-
|
| Target Release: | OCS 4.7.0 | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | 4.7.0-235.ci | Doc Type: | No Doc Update |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2021-05-19 09:14:56 UTC | Type: | Bug |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
|
Description
Ben Eli
2020-06-23 14:13:10 UTC
Adding some info I got from Vu Dinh when I tried to figure out what's going on: So it looks like the issue here is with v1beta using JSONPath but for v1 it is renamed to jsonPath. So during the conversion, the entire additionalPrinterColumns section is dropped somehow as you can it is missing in the CRD yaml that you query from apiserver. I think you need to go ahead and update your CRDs to v1 format. v1beta1 CRD is deprecated already so it is best to update CRD for 4.5+ I believe the OBC CRD definitions are in the NooBaa Operator, so moving this to the MCG component. Nimrod, can you take a look and see if this is something we need for OCS 4.5? What about deploy/olm-catalog/ocs-operator/manifests/objectbucket.crd.yaml deploy/olm-catalog/ocs-operator/manifests/objectbucketclaim.crd.yaml They are not just copied from the noobaa repo right? I might be missing something here, but if we take the CRDs defined in noobaa how are we not conflicting with the ones in rook? Isn't it that neither rook nor noobaa-operator brings them when in OCS and they are within the ocs-operator? We get the OB and OBC CRD manifests by running: podman run <NOOBAA_IMAGE> crd yaml So we're running a subcommand of the main entrypoint of the NooBaa image. We then use this output to generate the files indicated above. Thus, it falls on the NooBaa Operator to update the CRD generation, I believe somewhere around here: https://github.com/noobaa/noobaa-operator/blob/master/pkg/crd/crd.go Moving back to MCG component. There is a lack of updates here, but I was reading in an email, that this might require updates in lib-bucket-provisioner and rook. Because of medium severity, i'm moving it out of 4.5.0 Please move back if there's disagreement. @Jose Do we want to update lib-bucket-provisioner ? Not sure its worth it as its deprecated and will be replaced with COSI in the not so far future... As mentioned before we use CRD definitions of v1beta1 which is deprecated in Kubernetes 1.19. this is the case for all OCS CRDs and not only OBC. in OCP-4.6 there is an automatic conversion of the CRDs from v1beta1 to v1 and this issue doesn't occur. I think we still need to update all CRDs to v1 and not just rely on the conversion. this is not only a change in noobaa and should be synced between all OCS components. it also requires more work than just updating the YAMLs, since some fields have changed (e.g. spec.version was changed to spec.versions) because of the automatic conversion in OCP4.6 I am not sure how critical it is for OCS 4.6. following the discussion in ocs operators status meeting (http://post-office.corp.redhat.com/archives/rhocs-eng/2020-September/msg00213.html) pushing to ocs 4.7 @ben this true for all OCS CRDs. let's make sure we have similar BZs for other components (rook, OCS-operator) Hi Danny,
> in OCP-4.6 there is an automatic conversion of the CRDs from v1beta1 to v1 and this issue doesn't occur.
> I think we still need to update all CRDs to v1 and not just rely on the conversion.
If this is the case, don't we need the fix in OCS 4.6 and even backported to 4.5.z?
@elad, since in 4.6 the CRD is converted automatically, this issue will not happen. I think it makes it less critical to fix right now. updating the CRD is something that we better do regardless of the automatic conversion. since it requires some work from all components we decided to push it to 4.7 as for 4.5.z, I am not sure it is such a critical issue that requires us to handle it now The topic was discussed via email, and it was decided to create two Jira stories, one for each component, and attach them to this epic - https://issues.redhat.com/browse/KNIP-1491 Yes, just note that the epic is targeted to 4.7 Since the epic is in FF, moving to on qe and to the ocs-op (according to the epic) Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory (Moderate: Red Hat OpenShift Container Storage 4.7.0 security, bug fix, and enhancement update), and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHSA-2021:2041 The needinfo request[s] on this closed bug have been removed as they have been unresolved for 500 days |