Bug 1909745 - pv-pool backing store name restriction should be at 43 characters
Summary: pv-pool backing store name restriction should be at 43 characters
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat OpenShift Container Storage
Classification: Red Hat Storage
Component: Multi-Cloud Object Gateway
Version: 4.6
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: ---
: OCS 4.7.0
Assignee: Jacky Albo
QA Contact: aberner
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2020-12-21 14:36 UTC by aberner
Modified: 2021-06-01 08:45 UTC (History)
5 users (show)

Fixed In Version: v4.7.0-236.ci
Doc Type: No Doc Update
Doc Text:
Clone Of:
Environment:
Last Closed: 2021-05-19 09:17:08 UTC
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github noobaa noobaa-core pull 6354 0 None closed Update max k8s pool name len to 43 in the UI 2021-02-15 11:23:32 UTC
Github noobaa noobaa-core pull 6356 0 None closed Backport to 5.7 2021-02-15 11:23:32 UTC
Github noobaa noobaa-operator pull 504 0 None closed Backport to 5.7 2021-02-15 11:23:32 UTC
Github noobaa noobaa-operator pull 519 0 None closed Rejecting too long pv-pool backingstore names 2021-02-15 11:23:33 UTC
Github noobaa noobaa-operator pull 520 0 None closed Backport to 5.7 2021-02-15 11:23:33 UTC
Red Hat Product Errata RHSA-2021:2041 0 None None None 2021-05-19 09:17:59 UTC

Description aberner 2020-12-21 14:36:28 UTC
Description of problem (please be detailed as possible and provide log
snippests):
Due to kubernetes pod name length limitation, the prefix automatically added by noobaa to the agent pod of the pv pool name is causing a failure in the pv-pool backing store creation

Version of all relevant components (if applicable):


Does this issue impact your ability to continue to work with the product
(please explain in detail what is the user impact)?


Is there any workaround available to the best of your knowledge?
enter a name shorter than 43 characters

Rate from 1 - 5 the complexity of the scenario you performed that caused this
bug (1 - very simple, 5 - very complex)?
1

Can this issue reproducible?
yes always

Can this issue reproduce from the UI?
yes always

If this is a regression, please provide more details to justify this:


Steps to Reproduce:
1. enter the backing store creation tool in the ui
2. create a backing store with a name longer than 43 character 

Actual results:
backing store state is Rejected: Failed connecting all pods in backing store for more than 10 minutes


Expected results:
this should not be allowed, the name of a pv-pool should be limited to 43 both from the UI, Yaml and Cli

Additional info:

***Important note***
The backing store as a resource in Noobaa UI looks ok and the pod,pvc,pv are all created and in state running and bound

Comment 5 aberner 2021-01-18 12:26:35 UTC
This procedure is not fixed from the ui or yaml scenarios perspective.
The OCS UI does not limit the resource name at all and the NooBaa Dashboard is allowing a 47 characters name limit instead of the required 43 characters.
I've opened another bug for the OCS UI but we still need to fix in the NooBaa dashboard as well.
(OCS UI bug: https://bugzilla.redhat.com/show_bug.cgi?id=1917395)
The Yaml with this issue should be rejected but instead it is created and getting stuck both in ocs and in noobaa. 

OCP: 4.7.0-0.nightly-2021-01-17-065043
OCS: 4.7.0-231.ci

Comment 7 aberner 2021-02-03 13:44:22 UTC
OCS: ocs-operator.v4.7.0-250.ci
OCP: 4.7.0-0.nightly-2021-02-02-223803

Comment 10 errata-xmlrpc 2021-05-19 09:17:08 UTC
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


Note You need to log in before you can comment on or make changes to this bug.