Bug 1634375

Summary: The installer failed to show the iscsi disk if there is no acl create on the target
Product: [Fedora] Fedora Reporter: lnie <lnie>
Component: anacondaAssignee: Radek Vykydal <rvykydal>
Status: CLOSED NOTABUG QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 29CC: anaconda-maint-list, jonathan, kellin, lnie, rvykydal, vanmeeuwen+fedora, vponcova, wwoods
Target Milestone: ---Keywords: Reopened
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-10-09 04:45:30 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
anaconda.log
none
storage.log
none
syslog
none
screencast
none
anaconda.log
none
storage.log
none
syslog none

Description lnie 2018-09-30 10:30:38 UTC
Created attachment 1488508 [details]
anaconda.log

Description of problem:

As shown in the attached screencast,the installer can login,and show the disks in lsblk lists but failed to show in the GUI,so the user can't use the iscsi disks



/iscsi/iqn.20...e:target/tpg1> ls
o- tpg1 ........................................................................................................ [gen-acls, no-auth]
  o- acls ................................................................................................................ [ACLs: 0]
  o- luns ................................................................................................................ [LUNs: 2]
  | o- lun0 ................................................................ [fileio/testfile2 (/tmp/fileio.img) (default_tg_pt_gp)]
  | o- lun1 ........................................................... [fileio/disk2 (/home/lnie/diskiscsi.img) (default_tg_pt_gp)]
  o- portals .......................................................................................................... [Portals: 1]
    o- 0.0.0.0:3260 ........................................................................................................... [OK]
/iscsi/iqn.20...e:target/tpg1> get attribute generate_node_acls
generate_node_acls=1 
/iscsi/iqn.20...e:target/tpg1> get attribute authentication
authentication=0 


Version-Release number of selected component (if applicable):
Fedora-Workstation-netinst-x86_64-29-20180923.n.0.iso

How reproducible:
always

Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 lnie 2018-09-30 10:31:06 UTC
Created attachment 1488509 [details]
storage.log

Comment 2 lnie 2018-09-30 10:31:29 UTC
Created attachment 1488510 [details]
syslog

Comment 3 lnie 2018-09-30 10:31:57 UTC
Created attachment 1488511 [details]
screencast

Comment 4 Radek Vykydal 2018-10-05 13:57:22 UTC
I think Anaconda is not showing the disk because it is read-only so it can't be used for installation.

To make it writeable I think that you'd need to
set attribute demo_mode_write_protect=0
on the target.

If it is not the case feel free to reopen the bug.

Comment 5 lnie 2018-10-07 08:41:42 UTC
The installer still failed to show the iscsi disk after set attribute demo_mode_write_protect=0 on the target

Comment 6 lnie 2018-10-07 08:42:11 UTC
Created attachment 1491298 [details]
anaconda.log

Comment 7 lnie 2018-10-07 08:42:50 UTC
Created attachment 1491299 [details]
storage.log

Comment 8 lnie 2018-10-07 08:43:18 UTC
Created attachment 1491300 [details]
syslog

Comment 9 Radek Vykydal 2018-10-08 07:57:20 UTC
(In reply to lnie from comment #5)
> The installer still failed to show the iscsi disk after set attribute
> demo_mode_write_protect=0 on the target

When trying with the option it may require also modifying the initiator name in the GUI (eg just delete one character) as the acl configuration for the target can be cached on the server. Or create a target completely from the scratch.
If even this does not make it working, please attach output of

cat /sys/devices/platform/host2/session1/target2:0:0/2:0:0:0/block/sda/ro

(run after the login into target)
.. it should be "0" if the targetcli is configured in read-write for demo mode.

and list of tpg1 attributes from targetcli, for my working configuration it is:


iscsi/iqn.20...ample:t1/tpg1> get attribute 
ATTRIBUTE CONFIG GROUP
======================
authentication=0
----------------
If set to 1, enforce authentication for this TPG.

cache_dynamic_acls=1
--------------------
If set to 1 in demo mode, cache dynamically generated ACLs.

default_cmdsn_depth=64
----------------------
Default CmdSN (Command Sequence Number) depth.

default_erl=0
-------------
Default Error Recovery Level.

demo_mode_discovery=1
---------------------
If set to 1 in demo mode, enable discovery.

demo_mode_write_protect=0
-------------------------
If set to 1 in demo mode, prevent writes to LUNs.

fabric_prot_type=0
------------------
Fabric DIF protection type.

generate_node_acls=1
--------------------
If set to 1, allow all initiators to login (i.e. demo mode).

login_keys_workaround=1
-----------------------


login_timeout=15
----------------
Login timeout value in seconds.

netif_timeout=2
---------------
NIC failure timeout in seconds.

prod_mode_write_protect=0
-------------------------
If set to 1, prevent writes to LUNs.

t10_pi=0
--------
If set to 1, enable T10 Protection Information.

tpg_enabled_sendtargets=1
-------------------------
If set to 1, the SendTargets discovery response advertises the TPG only if the TPG is enabled.

Comment 10 lnie 2018-10-09 04:45:30 UTC
(In reply to Radek Vykydal from comment #9)
> (In reply to lnie from comment #5)
> > The installer still failed to show the iscsi disk after set attribute
> > demo_mode_write_protect=0 on the target
> 
> When trying with the option it may require also modifying the initiator name
> in the GUI (eg just delete one character) as the acl configuration for the
> target can be cached on the server. Or create a target completely from the
> scratch.
 Works here now,thanks for your information.