Storage settings in Fuel UI

Bug #1323555 reported by Sergii Golovatiuk on 2014-05-27
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Medium
Dmitry Russkikh
7.0.x
Medium
Fuel UI Team
Mitaka
Medium
Fuel UI Team
Newton
Medium
Dmitry Russkikh

Bug Description

Fuel UI has a logical mistake in "Settings" tab. Fuel UI allows to save settings if Storage settings are not set. UI should allow to store settings when "Cinder LVM over iSCSI for volumes" or "Ceph RDB for volumes" is selected. Otherwise, user will get error when he clicks on "Deploy" button. I've attached image with error description. Here is another scenario as a sample.

User selected Ceph in Wizard, but then he realized he didn't have free nodes for Ceph. He deselects "Ceph" in Settings but doesn't select "Cinder LVM over iSCSI for volumes" as he has no clue what it means. UI allows to save settings though it's not correct and user is totally puzzled.

Sergii Golovatiuk (sgolovatiuk) wrote :
description: updated
Vitaly Kramskikh (vkramskikh) wrote :

+1, these 2 checkboxes should be turned into 2 radiobuttons

Vitaly Kramskikh (vkramskikh) wrote :

Dmitry, what do you think? Changing this will require manifests modification as instead of checking for storge.volumes_lvm.value == true or storge.volumes_ceph.value == true you will need to check for storge.volumes.value == "lvm" or "ceph"

Changed in fuel:
assignee: nobody → Dmitry Borodaenko (dborodaenko)
Dmitry Borodaenko (angdraug) wrote :

We originally kept the checkboxes because we were going to implement support for multiple cinder backends:
https://blueprints.launchpad.net/fuel/+spec/fuel-cinder-multi-backend

The above blueprint has Low priority so it's likely that it won't be completed in 5.1. This means that we should fix the UI now and update it later if multiple cinder backends gets done.

What I don't really like about the proposed solution is how it highlights the tight coupling between settings UI and Puppet manifests. We should nail down astute.yaml structure so that we can limit impact of UI changes like that to Nailgun. Do you think it's worth a separate blueprint?

One more thing that concerns me is that the attached screenshot has nothing to do with Cinder. Can you explain the whole chain of events from misconfiguring Cinder storage to Nailgun raising an error about Image Storage? I think there's a third problem hiding there.

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

Changed in fuel:
status: New → In Progress
Changed in fuel:
importance: Undecided → Medium
assignee: Dmitry Borodaenko (dborodaenko) → Vitaly Kramskikh (vkramskikh)
Vitaly Kramskikh (vkramskikh) wrote :

Dmitry,

I don't really see how it is possible. I really love the how easy new settings can be added to cluster and you can choose their restrictions and their representation on UI in one place. As the only way to solve it I see creating new intermediate variables in manifests which values are determined from astute.yaml and these variable used in conditions in manifests.

Changed in fuel:
assignee: Vitaly Kramskikh (vkramskikh) → Dmitry Borodaenko (dborodaenko)
Changed in fuel:
milestone: none → 5.1
Dmitry Pyzhov (dpyzhov) on 2014-07-28
Changed in fuel:
status: In Progress → Confirmed
milestone: 5.1 → 6.0
Changed in fuel:
milestone: 6.0 → 6.1
tags: added: docs release-notes
Nastya Urlapova (aurlapova) wrote :

Guys, could you set one correct status for issue in 6.1

Changed in fuel:
status: Confirmed → Won't Fix
no longer affects: fuel/6.1.x
tags: added: qa-agree-7.0

Change abandoned by Dmitry Borodaenko (<email address hidden>) on branch: master
Review: https://review.openstack.org/96252
Reason: Obsolete change, not relevant anymore.

Dmitry Pyzhov (dpyzhov) on 2015-10-12
Changed in fuel:
milestone: 6.1 → 8.0
status: Won't Fix → Confirmed
no longer affects: fuel/8.0.x
tags: removed: release-notes
Dmitry Pyzhov (dpyzhov) on 2015-10-22
tags: added: area-ui
Changed in fuel:
status: Confirmed → Won't Fix

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

Changed in fuel:
assignee: Fuel UI Team (fuel-ui) → Dmitry Russkikh (drusskikh)
status: Won't Fix → In Progress

Change abandoned by Fuel DevOps Robot (<email address hidden>) on branch: master
Review: https://review.openstack.org/334542
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers