nailgun-agent doesn't find disk if there a diretory in /dev/disk/by-id

http://paste.openstack.org/show/589510/

patch proposal by <AlexAvadanii> from fuel irc chanel

668 668
669 669 def _disk_id_by_name(name)
670 670 dn = "/dev/disk/by-id"
671 basepath = Dir["#{dn}/**?"].select{|f| /\/#{name}$/.match(File.readlink(f))}
671 basepath = Dir["#{dn}/**?"].select{|f| File.symlink?(f) and /\/#{name}$/.match(File.readlink(f))}
672 672 basepath.map{|p| p.split("/")[2..-1].join("/")}
673 673 end
674 674
675 675 def _disk_path_by_name(name)
676 676 dn = "/dev/disk/by-path"
677 basepath = Dir["#{dn}/**?"].find{|f| /\/#{name}$/.match(File.readlink(f))}
677 basepath = Dir["#{dn}/**?"].find{|f| File.symlink?(f) and /\/#{name}$/.match(File.readlink(f))}
678 678 basepath.split("/")[2..-1].join("/") if basepath
679 679 end

I've tried in the versions 9, 10 and 11.

Hello there, basically I'm hitting the same problem like dr Skape, I've also got a directory in the path:

2016-11-17T14:19:09Z notice nailgun-agent: E, [2016-11-17T14:19:09.633898 #9763] ERROR -- : Error 'Invalid argument @ rb_readlink - /dev/disk/by-id/scsi-SSAMSUNG_MZ6ER400HAGL' in gathering disks metadata: ["/usr/bin/nailgun-agent:875:in `readlink'", "/usr/bin/nailgun-agent:875:in `block in _disk_id_by_name'", "/usr/bin/nailgun-agent:875:in `select'", "/usr/bin/nailgun-agent:875:in `_disk_id_by_name'", "/usr/bin/nailgun-agent:1004:in `block in physical_data_storage_devices'", "/usr/bin/nailgun-agent:968:in `each'", "/usr/bin/nailgun-agent:968:in `physical_data_storage_devices'", "/usr/bin/nailgun-agent:727:in `block (2 levels) in _detailed'", "/usr/bin/nailgun-agent:721:in `each'", "/usr/bin/nailgun-agent:721:in `block in _detailed'", "/usr/lib/ruby/2.3.0/timeout.rb:91:in `block in timeout'", "/usr/lib/ruby/2.3.0/timeout.rb:33:in `block in catch'", "/usr/lib/ruby/2.3.0/timeout.rb:33:in `catch'", "/usr/lib/ruby/2.3.0/timeout.rb:33:in `catch'", "/usr/lib/ruby/2.3.0/timeout.rb:106:in `timeout'", "/usr/bin/nailgun-agent:7

Fix proposed to branch: master
Review: https://review.openstack.org/400634

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/400641

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/400642

Reviewed: https://review.openstack.org/400634
Committed: https://git.openstack.org/cgit/openstack/fuel-nailgun-agent/commit/?id=402eb1f423a3833685fb439df5616bfdfcdee453
Submitter: Jenkins
Branch: master

commit 402eb1f423a3833685fb439df5616bfdfcdee453
Author: Vladimir Kozhukalov <email address hidden>
Date: Tue Nov 22 12:28:27 2016 +0300

    Check if /dev/disk/*/ item is a symlink

    Sometimes udev places devices symlinks into a directory
    /dev/disk/by-id/foo/device_symlink and we must be able
    to go through this directories recursively and find all
    symlinks.

    Change-Id: I0749ab94e05fdf6fd12dc89c2c788a61a128b77d
    Closes-Bug: #1642391

Reviewed: https://review.openstack.org/400641
Committed: https://git.openstack.org/cgit/openstack/fuel-nailgun-agent/commit/?id=56ef29cd220379215ee996e78d665013f14f59f7
Submitter: Jenkins
Branch: stable/mitaka

commit 56ef29cd220379215ee996e78d665013f14f59f7
Author: Vladimir Kozhukalov <email address hidden>
Date: Tue Nov 22 12:28:27 2016 +0300

    Check if /dev/disk/*/ item is a symlink

    Change-Id: I0749ab94e05fdf6fd12dc89c2c788a61a128b77d
    Closes-Bug: #1642391

Reviewed: https://review.openstack.org/400642
Committed: https://git.openstack.org/cgit/openstack/fuel-nailgun-agent/commit/?id=c2ac19a35d5d628c7e128c9b93eef99637c327e7
Submitter: Jenkins
Branch: stable/newton

commit c2ac19a35d5d628c7e128c9b93eef99637c327e7
Author: Vladimir Kozhukalov <email address hidden>
Date: Tue Nov 22 12:28:27 2016 +0300

    Check if /dev/disk/*/ item is a symlink

    Sometimes udev places devices symlinks into a directory
    /dev/disk/by-id/foo/device_symlink and we must be able
    to go through this directories recursively and find all
    symlinks.

    Change-Id: I0749ab94e05fdf6fd12dc89c2c788a61a128b77d
    Closes-Bug: #1642391

verified on 9.2 snapshot #553

This issue was fixed in the openstack/fuel-nailgun-agent 11.0.0.0rc1 release candidate.