Bug 1275845 - Add an already exist storage connection to a data center with a storage domain should fail
Summary: Add an already exist storage connection to a data center with a storage domai...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: BLL.Storage
Version: 3.6.0.2
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ovirt-4.1.4
: 4.1.4.2
Assignee: Allon Mureinik
QA Contact: Raz Tamir
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-10-27 22:48 UTC by Raz Tamir
Modified: 2017-07-28 14:18 UTC (History)
5 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2017-07-28 14:18:18 UTC
oVirt Team: Storage
Embargoed:
rule-engine: ovirt-4.1+


Attachments (Terms of Use)
print screen (103.89 KB, image/png)
2015-10-27 22:48 UTC, Raz Tamir
no flags Details
engine and vdsm logs (513.94 KB, application/x-bzip)
2015-10-27 22:49 UTC, Raz Tamir
no flags Details

Description Raz Tamir 2015-10-27 22:48:44 UTC
Created attachment 1087086 [details]
print screen

Description of problem:
When adding an existing storage connection again, it should fail.


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


How reproducible:


Steps to Reproduce:
1. Login to all iscsi targets
2. Add new storage domain
3. Add an already existing storage connection

Actual results:
It is possible to add the same connection twice

Expected results:
It shouldn't be allowed to add the same storage connection twice

Additional info:

Comment 1 Raz Tamir 2015-10-27 22:49:20 UTC
Created attachment 1087087 [details]
engine and vdsm logs

Comment 2 Allon Mureinik 2015-10-28 12:52:26 UTC
I don't understand the flow.
What exactly are you adding, and how are you doing this?
If it's done via REST - please provide the exact requests you're sending.
if it's done via the GUI, please provide screenshots.

Comment 3 Red Hat Bugzilla Rules Engine 2015-10-28 12:52:30 UTC
Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release.

Comment 4 Raz Tamir 2015-10-28 14:57:07 UTC
I'm trying to add a storage connection that located in DB under storage_server_connections table (attached print screen)

The POST request:
https://jenkins-vm-11.scl.lab.tlv.redhat.com/ovirt-engine/api/storageconnections body:
<storage_connection>
    <address>10.35.160.107</address>
    <type>iscsi</type>
    <port>3260</port>
    <target>iqn.1992-04.com.emc:cx.ckm00121000438.b7</target>
</storage_connection>

Comment 5 Yaniv Lavi 2017-02-23 11:26:07 UTC
Moving out all non blocker\exceptions.

Comment 6 Allon Mureinik 2017-07-24 14:03:19 UTC
When testing with the current master (commit 121657e), I get the expected 409 error when testing this scenario:


<fault>
    <detail>[Cannot add Storage Connection. Storage connection already exists.]</detail>
    <reason>Operation Failed</reason>
</fault>

It seems that the relevant code has been there for a while.
Setting to ON_QA for QA to verify on 4.1.5, but realistically this can even be verified in 4.1.4 if QE have the capacity.

Comment 7 Raz Tamir 2017-07-24 14:41:39 UTC
Thanks Allon,

Verified for both master and 4.1.4.


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