Bug 1370101 - RefreshLunsSizeCommand doesn't update the new LUNs' sizes in the DB
Summary: RefreshLunsSizeCommand doesn't update the new LUNs' sizes in the DB
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: BLL.Storage
Version: 4.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ovirt-4.0.4
: 4.0.4
Assignee: Idan Shaby
QA Contact: Kevin Alon Goldblatt
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-08-25 10:28 UTC by Idan Shaby
Modified: 2016-09-26 12:42 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Previously, after resizing a lun, checking the storage domain's size gave us the new value, but checking the lun's size gave us the old one. This bug was fixed and now the new size is immediately updated, so checking it right after the resize brings the updated value.
Clone Of:
Environment:
Last Closed: 2016-09-26 12:42:23 UTC
oVirt Team: Storage
Embargoed:
amureini: ovirt-4.0.z?
rule-engine: planning_ack?
amureini: devel_ack+
rule-engine: testing_ack+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 62837 0 master MERGED core: update LUNs in DB on RefreshLunsSizeCommand 2016-08-28 11:45:06 UTC
oVirt gerrit 62910 0 ovirt-engine-4.0 MERGED core: update LUNs in DB on RefreshLunsSizeCommand 2016-08-29 09:24:32 UTC

Description Idan Shaby 2016-08-25 10:28:07 UTC
Description of problem:
When resizing a LUN, RefreshLunsSizeCommand updates the storage domain's new size but doesn't update the storage domain's LUNs' new sizes in the DB.

Version-Release number of selected component (if applicable):
a58290fe513ffc1c879e984ed746667319ede8ba

How reproducible:
100%

Steps to Reproduce:
1. Create an iSCSI storage domain.
2. Resize the underlying lun(s) of that domain in the storage server.
3. Edit the storage domain in the web admin, go to "LUNs > Targets" and resize the lun(s).

Actual results:
The resize works - the PVs are resized and the storage domain's size is updated.
However, the LUNs in the DB are not updated so their sizes are not up to date.

Expected results:
The LUNs in the DB should be updated and contain their new sizes.

Comment 1 Allon Mureinik 2016-08-31 15:58:16 UTC
Idan - please add some doctext on this issue.

Comment 2 Kevin Alon Goldblatt 2016-09-13 16:57:21 UTC
(In reply to Idan Shaby from comment #0)
> Description of problem:
> When resizing a LUN, RefreshLunsSizeCommand updates the storage domain's new
> size but doesn't update the storage domain's LUNs' new sizes in the DB.
> 
> Version-Release number of selected component (if applicable):
> a58290fe513ffc1c879e984ed746667319ede8ba
> 
> How reproducible:
> 100%
> 
> Steps to Reproduce:
> 1. Create an iSCSI storage domain.
> 2. Resize the underlying lun(s) of that domain in the storage server.
> 3. Edit the storage domain in the web admin, go to "LUNs > Targets" and
> resize the lun(s).


At this point according to the above scenario the additional size added to the LUN is displayed as a text button in the "LUNs > Targets" tab >>>> HOWEVER nothing happens when pressing this button and no resizing is performed. Please clarify whether the LUN resize should be performed by pressing the additional size added with the text "+10 GB"




> 
> Actual results:
> The resize works - the PVs are resized and the storage domain's size is
> updated.
> However, the LUNs in the DB are not updated so their sizes are not up to
> date.
> 
> Expected results:
> The LUNs in the DB should be updated and contain their new sizes.

Comment 3 Idan Shaby 2016-09-14 05:44:10 UTC
(In reply to Kevin Alon Goldblatt from comment #2)
> (In reply to Idan Shaby from comment #0)
> > Description of problem:
> > When resizing a LUN, RefreshLunsSizeCommand updates the storage domain's new
> > size but doesn't update the storage domain's LUNs' new sizes in the DB.
> > 
> > Version-Release number of selected component (if applicable):
> > a58290fe513ffc1c879e984ed746667319ede8ba
> > 
> > How reproducible:
> > 100%
> > 
> > Steps to Reproduce:
> > 1. Create an iSCSI storage domain.
> > 2. Resize the underlying lun(s) of that domain in the storage server.
> > 3. Edit the storage domain in the web admin, go to "LUNs > Targets" and
> > resize the lun(s).
> 
> 
> At this point according to the above scenario the additional size added to
> the LUN is displayed as a text button in the "LUNs > Targets" tab >>>>
> HOWEVER nothing happens when pressing this button and no resizing is
> performed. Please clarify whether the LUN resize should be performed by
> pressing the additional size added with the text "+10 GB"
You should press that button and then click on the "OK" button.
> 
> 
> 
> 
> > 
> > Actual results:
> > The resize works - the PVs are resized and the storage domain's size is
> > updated.
> > However, the LUNs in the DB are not updated so their sizes are not up to
> > date.
> > 
> > Expected results:
> > The LUNs in the DB should be updated and contain their new sizes.

Comment 4 Kevin Alon Goldblatt 2016-09-14 14:48:30 UTC
Tested with the following code:
----------------------------------------
rhevm-4.0.4.2-0.1.el7ev.noarch
vdsm-4.18.13-1.el7ev.x86_64

Tested with the following scenario:

Steps to Reproduce:
1. Create an iSCSI storage domain.
2. Resize the underlying lun(s) of that domain in the storage server.
3. Edit the storage domain in the web admin, go to "LUNs > Targets" and resize the lun(s) >>>>> The resize operation is successful and the DB is correctly updated.

Actual results:
The resize works - the PVs are resized and the storage domain's size is updated.
The LUNs in the DB ARE NOW updated so their sizes ARE up to date.

Expected results:
The LUNs in the DB should be updated and contain their new sizes.


Moving to VERIFIED!


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