Bug 1637613

Summary:	Unable to add gluster storage domain in RHV-M without the comment
Product:	[Red Hat Storage] Red Hat Gluster Storage	Reporter:	bipin <bshetty>
Component:	rhhi	Assignee:	Sahina Bose <sabose>
Status:	CLOSED CURRENTRELEASE	QA Contact:	SATHEESARAN <sasundar>
Severity:	high	Docs Contact:
Priority:	high
Version:	rhhi-1.1	CC:	bugs, godas, lsvaty, rhs-bugs, sabose, sankarshan, sasundar
Target Milestone:	---	Keywords:	Regression
Target Release:	RHHI-V 1.5
Hardware:	x86_64
OS:	Linux
Whiteboard:
Fixed In Version:	ovirt-engine-4.2.7.3	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:	1637605	Environment:
Last Closed:	2019-05-20 04:54:51 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	Gluster	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:	1633232, 1637605
Bug Blocks:	1520836

Description bipin 2018-10-09 15:00:46 UTC

+++ This bug was initially created as a clone of Bug #1637605 +++

Description of problem:
======================
While adding the gluster storage domain in RHV-M , it throws error. This was seen after creating additional gluster volumes and creating storage domains for those volumes.


Version-Release number of selected component (if applicable):
===========================================================
rhv-release-4.2.7-4-001


How reproducible:
================
2/2


Steps to Reproduce:
==================
1.Create additional gluster volumes via cockpit Day 2
2.While adding the new gluster storage domain it is unsuccessful.


Actual results:
==============
The storage domains cannot be added


Expected results:
================
The storage domains should be successfully added


Additional info:
===============

--- Additional comment from bipin on 2018-10-09 10:56:14 EDT ---

Please reassign the component if not accurate.

Snippet from the log:
====================
2018-10-09 17:43:00,360+05 WARN  [org.ovirt.engine.core.bll.storage.domain.AddGlusterFsStorageDomainCommand] (default task-53) [b5ddffb2-d42b-434e-a722-2eb80ae91877] Validation of action 'AddGlusterFsStorageDomain' failed for user admin@internal-authz. Reasons: VAR__TYPE__STORAGE__DOMAIN,VAR__ACTION__ADD,may not be null,$message {javax.validation.constraints.NotNull.message},$payload [Ljava.lang.Class;@199300f9,$groups [Ljava.lang.Class;@3e83543c,ACTION_TYPE_FAILED_ATTRIBUTE_PATH,$path privateStorageDomain.comment,$validatedValue null

--- Additional comment from bipin on 2018-10-09 10:58 EDT ---

Comment 1 Gobinda Das 2018-10-10 04:52:23 UTC

Bipin, I think it's expecting Comment from UI. If you will provide Comment then it should work. I am not sure why it's mandatory even though it's not marked as mandatory from UI.

Comment 2 SATHEESARAN 2018-10-10 08:23:00 UTC

(In reply to Gobinda Das from comment #1)
> Bipin, I think it's expecting Comment from UI. If you will provide Comment
> then it should work. I am not sure why it's mandatory even though it's not
> marked as mandatory from UI.

Correct, with comment in place, creation of storage domain worked good

Is this the intended behaviour ?

Comment 3 Sahina Bose 2018-10-10 08:24:32 UTC

(In reply to SATHEESARAN from comment #2)
> (In reply to Gobinda Das from comment #1)
> > Bipin, I think it's expecting Comment from UI. If you will provide Comment
> > then it should work. I am not sure why it's mandatory even though it's not
> > marked as mandatory from UI.
> 
> Correct, with comment in place, creation of storage domain worked good
> 
> Is this the intended behaviour ?

No - this is not a mandatory field..seems like a regression in the latest version. Gobinda can you check?

Comment 9 SATHEESARAN 2018-10-18 18:16:49 UTC

The dependent ovirt bug is verified, moving this bug ON_QA

Comment 10 SATHEESARAN 2018-10-18 18:17:29 UTC

Tested with ovirt-engine-4.2.7.3 and the storage domains could be created without the comment