Web UI restrictions doesn't work correcly for Ironic

[Sofiia Andriichenko]

Description:
Deployment of MOS ISO #225 http://paste.openstack.org/show/495123/
OpenStack Settings Configuration Error
Some restrictions didn't pass verification: ["Validation failed for attribute 'Install Ironic': Ironic requires Swift or RadosGW for Glance images."]

Steps to reproduce:
1. Got o Fuel
2. Create environment
3. Name "env1"
4. Compute "QEMU-KVM"
5. Networking setup "Neutron vlan segmentation"
6. Storage Backends:
 - Ceph RBD for volumes (Cinder)
 - Ceph RBD for ephemeral volumes (Nova)
 - Ceph RBD for images (Glance)

Expected results:
Impossible to select Ironic in Additional components tab when Ceph for Objects is not set RadosGW

Actual result:
Possible to install Ironic without RadosGw and Images in Ceph

Reproducibility:
100%

Additional information:
--

[Sergii Turivnyi (sturivnyi)]

This bug is blocker for MOS-QA Team
On SI we got Error:

2016-04-20 22:16:35,468 - ERROR http.py:110 -- Bad Request code 400 [{"message": "Some restrictions didn't pass verification: [\"Validation failed for attribute 'Install Ironic': Ironic requires Swift or RadosGW for Glance images.\"]", "errors": []}]
2016-04-20 22:16:35,470 - ERROR __init__.py:66 -- update_cluster_attributes raised: HTTPError()

[Alexei Sheplyakov (asheplyakov)]

Actually radosgw is not required for using Ceph as images and volumes backend

[Alexei Sheplyakov (asheplyakov)]

> 'Install Ironic': Ironic requires Swift or RadosGW for Glance images.

radosgw is an optional component of a ceph cluster, quite a number of installations don't use radosgw at all
(they use ceph mainly via rbd to provide the storage for VMs). Therefore whatever deploys Ironic should explicitly enable radosgw (and adjust settings according to Ironic requirements)

[Alexei Sheplyakov (asheplyakov)]

As a work around one can manually enable radosgw (in Fuel UI: Settings -> Storage -> Ceph RadosGW for objects (Swift API))

[Vasyl Saienko (vsaienko)]

The following restriction doesn't work on UI. Please have look at the attachment.
https://github.com/openstack/fuel-web/blob/0643e340095c7a190931ffd2d2d7b653340ec7fe/nailgun/nailgun/fixtures/openstack.yaml#L983

[Dmitry Tantsur (divius)]

Please clarify if you need some work done on the Ironic side, it's not quite obvious at first glance.

[Julia Aranovich (jkirnosova)]

Vasyl, thank you for the video recording. It perfectly demonstrates the bug.

The issue is that user is able to turn on Ironic in the wizard when Ceph for objects (Rados GW) is off: http://s32.postimg.org/h4y4vyttx/wizard_storage.png and http://s32.postimg.org/94xmcph1h/wizard_services.png

If user does not enable Ironic from the wizard, it is impossible to do on Settings tab: http://s32.postimg.org/6u00rwn7p/storage_tab.png and http://s32.postimg.org/jph765h8l/services_tab.png
That's an expected behaviour.

So, we need to fix restrictions for Ironic in wizard to require Ceph for objects (Rados GW) to be enabled.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Change abandoned by Andriy Popovych (<email address hidden>) on branch: master
Review: https://review.openstack.org/315942

[Andriy Popovych (popovych-andrey)]

Let me explain how wizard tab working. It operates in terms of compatible/incompatible not restrictions and hasn't condition operator for now (some improvements will be in 10.0 https://blueprints.launchpad.net/fuel/+spec/component-registry-improvements). Settings page has more deep level of validation and it's expected behavior to raise validation message for restriction. As was written above possible workaround is just enable one option on settings tab. I agree that wizard should more smart but currently I don't see it as critical blocker for deployment so change priority to medium.

[Ruby Loo (rloo)]

This was opened over a year ago, and after reading the comments, it isn't clear to me that this is an ironic issue, so I'm going to invalidate it. If you have more information, please comment and update the status. Thanks.