Bug 1813706

Summary: Limit name length for PV BackingStore
Product: [Red Hat Storage] Red Hat OpenShift Container Storage Reporter: aberner
Component: Multi-Cloud Object GatewayAssignee: Romy Ayalon <rayalon>
Status: CLOSED ERRATA QA Contact: aberner
Severity: high Docs Contact:
Priority: medium    
Version: 4.3CC: ebenahar, etamir, madam, nbecker, ocs-bugs
Target Milestone: ---Keywords: AutomationBackLog
Target Release: OCS 4.4.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 4.4.0-410 Doc Type: No Doc Update
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-06-04 12:54:37 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:
Attachments:
Description Flags
error message none

Description aberner 2020-03-15 16:17:29 UTC
Created attachment 1670343 [details]
error message

Description of problem (please be detailed as possible and provide log
snippests):
When creating PV based Backing Store the character limit for Backing Store name is not low enough and creation fails due to kubernetes requirement that pod name should not exceed 64 characters.
The only place i found this error is in the Stateful Set event log in ocs.


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)?
No

Is there any workaround available to the best of your knowledge?
No

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

Can this issue reproducible?
yes

Can this issue reproduce from the UI?
yes

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

Steps to Reproduce:
1. Create a PV Backing Store in MTG with a very long name
2. Go to oc Stateful Set create by noobaa and watch event log



Actual results:
Creation of backing store fails

Expected results:
Lower character limit on Backing Store

Additional info:
Attached error message below

Comment 2 Romy Ayalon 2020-03-24 18:50:52 UTC
Hi Amit,
How exactly did you create the PV Backing Store? if it was through CLI please add the command.
when you said this issue can reproduced from the UI, did you mean Noobaa UI?

Comment 3 aberner 2020-03-25 07:39:09 UTC
Hi Rom,
You can create a pv backingstore in what ever way you wish and the problem would reproduce. 
The issue is created when NooBaa creates the Statefulset and adds noobaa-noobaa to it as well as noobaa-noobaa-(number) to the pods.

Comment 4 Michael Adam 2020-03-25 23:29:00 UTC
Not sure this should be on MODIFIED:

It is not qa_acke-ed, so not approved for 4.4.0.

Comment 5 Nimrod Becker 2020-03-26 08:07:18 UTC
@Michael, 
Talked about it with Raz yesterday, I think we just missed the flag .

Comment 9 aberner 2020-05-13 15:10:06 UTC
Noobaa UI and CLI verified as fixed.

Bug still occurs when creating pv backing store with the OCS UI and with YAML file as well.

Comment 10 Nimrod Becker 2020-05-13 15:16:43 UTC
You cannot limit on a yaml edit ...
Regarding the UI, its a different component and different issue.
I think we should create a new BZ for the UI and move this one according to its state.

Comment 13 errata-xmlrpc 2020-06-04 12:54:37 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, 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/RHBA-2020:2393