Cannot configure disk with Fuel 7 gui

[Claude Durocher]

Environment

{"build_id": "301", "build_number": "301", "release_versions": {"2015.1.0-7.0": {"VERSION": {"build_id": "301", "build_number": "301", "api": "1.0", "fuel-library_sha": "5d50055aeca1dd0dc53b43825dc4c8f7780be9dd", "nailgun_sha": "4162b0c15adb425b37608c787944d1983f543aa8", "feature_groups": ["mirantis"], "fuel-nailgun-agent_sha": "d7027952870a35db8dc52f185bb1158cdd3d1ebd", "openstack_version": "2015.1.0-7.0", "fuel-agent_sha": "50e90af6e3d560e9085ff71d2950cfbcca91af67", "production": "docker", "python-fuelclient_sha": "486bde57cda1badb68f915f66c61b544108606f3", "astute_sha": "6c5b73f93e24cc781c809db9159927655ced5012", "fuel-ostf_sha": "2cd967dccd66cfc3a0abd6af9f31e5b4d150a11c", "release": "7.0", "fuelmain_sha": "a65d453215edb0284a2e4761be7a156bb5627677"}}}, "auth_required": true, "api": "1.0", "fuel-library_sha": "5d50055aeca1dd0dc53b43825dc4c8f7780be9dd", "nailgun_sha": "4162b0c15adb425b37608c787944d1983f543aa8", "feature_groups": ["mirantis"], "fuel-nailgun-agent_sha": "d7027952870a35db8dc52f185bb1158cdd3d1ebd", "openstack_version": "2015.1.0-7.0", "fuel-agent_sha": "50e90af6e3d560e9085ff71d2950cfbcca91af67", "production": "docker", "python-fuelclient_sha": "486bde57cda1badb68f915f66c61b544108606f3", "astute_sha": "6c5b73f93e24cc781c809db9159927655ced5012", "fuel-ostf_sha": "2cd967dccd66cfc3a0abd6af9f31e5b4d150a11c", "release": "7.0", "fuelmain_sha": "a65d453215edb0284a2e4761be7a156bb5627677"}

Steps to reproduce

-Install Fuel 7
-Create a new environment (Kilo on Ubuntu 14.04, KVM, Neutron with VLAN segmentation, Cinder LVM over iSCSI for volumes)
-PXE boot a server
-Choose a role for the server
-click "Configure disks": screen for configuring disks appears
-click on the disk so it becomes editable : cursor changes to hand but nothing happens so it's impossible to change the disk configuration

This has been tested with Firefox and Chromium on Ubuntu 14.04.

[Ilya Kutukov (ikutukov)]

Could you please tell me is there any errors in browser debug console:
https://developer.chrome.com/devtools/docs/console

Especially related to the requests to address like:
https://[mydomain.com]:[myport]/api/nodes/[nodeid]/disks

https://www.evernote.com/l/AP-ZzOt0sHNLXbEHXmNyohI5mzk9U-8YnzI
Here is the example for google chrome latest, it's required to open debug console before you click "Configure Disks" in order to register api call.

If there are any error response, please copy and paste it text.

[Claude Durocher (claude-d)]

Here's what I got:

Uncaught Error: Syntax error, unrecognized expression: #cciss/c0d1
t.error @ jquery.js:1496
t.tokenize @ jquery.js:2113
t.select @ jquery.js:2517
t @ jquery.js:893
it.fn.extend.find @ jquery.js:2733
it.fn.init @ jquery.js:2850
it @ jquery.js:73
t @ bootstrap.js:693(anonymous function) @ bootstrap.js:735
it.event.dispatch @ jquery.js:4670
it.event.add.g.handle @ jquery.js:4338

We're using old HP Gen 5 servers with HP raid controler : I guess Fuel has trouble with the syntax of the drives (cciss/c0d1) but I could be wrong.

[Ilya Kutukov (ikutukov)]

I assign to the GUI team as the start point. Even this is a result of the misformatted field provided by the server by API, it makes sense to handle this kind of errors on GUI side more informative or less interfering way.

[Vitaly Kramskikh (vkramskikh)]

That's UI bug. Occurs when disk names have / in them

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/254915
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=cd14a8031cf46614fe80438175213efb5e002762
Submitter: Jenkins
Branch: master

commit cd14a8031cf46614fe80438175213efb5e002762
Author: Vitaly Kramskikh <email address hidden>
Date: Tue Dec 8 22:08:12 2015 +0300

    Fix for handling disks with slash in names

    Using Bootstrap JS API instead of data-* properties
    solves the issue.

    Change-Id: I35f739d03089a20d9baebdb0d4e871484d99cd5d
    Closes-Bug: #1516747

[Aleksei Stepanov (penguinolog)]

Verified on ISO: fuel-8.0-569-2016-02-15_07-42-00.iso
Image storage changed, changes saved.
Reopen settings: settings looks like saved.