Bug 1316771 - Ceph Installer not allowing to specify custom name for cluster
Summary: Ceph Installer not allowing to specify custom name for cluster
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Storage Console
Classification: Red Hat Storage
Component: ceph-installer
Version: 2
Hardware: Unspecified
OS: Unspecified
unspecified
urgent
Target Milestone: ---
: 2
Assignee: Christina Meno
QA Contact: ceph-qe-bugs
URL:
Whiteboard:
Depends On:
Blocks: 1291304
TreeView+ depends on / blocked
 
Reported: 2016-03-11 03:29 UTC by Nishanth Thomas
Modified: 2016-08-23 19:48 UTC (History)
13 users (show)

Fixed In Version: ceph-installer-1.0.3
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-08-23 19:48:26 UTC
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHEA-2016:1754 0 normal SHIPPED_LIVE New packages: Red Hat Storage Console 2.0 2017-04-18 19:09:06 UTC

Description Nishanth Thomas 2016-03-11 03:29:16 UTC
For USM to integrate with ceph-installer, there should be a provision configure the custom name for ceph cluster. This is not provided

Comment 2 Christina Meno 2016-03-11 17:14:11 UTC
I'm going to push back a little here.

There are several solutions that don't require a change to the installer.

Storage console can map FSID to cluster name.

Storage console can show clusters names with part of the FSID in it
e.g. ceph(12beaf5)

Nishanth would you please help me understand what barrier to integration exists?

Comment 3 Nishanth Thomas 2016-03-12 05:11:41 UTC
Couple of this

Comment 4 Nishanth Thomas 2016-03-12 05:57:14 UTC
1. MVP-007, allow specify and edit the cluster name. Also affect currently implemented flow as per UX. Please see https://docs.google.com/a/redhat.com/presentation/d/1MAgpVG2Fi2UtBYUhuyMScO8zObAYvhiHLp2HQ3_YpMI/edit?usp=sharing  for details(slide 12). This is because, as per your suggestion we need to auto generate the cluster name not allow the admin to change it

2. There will be a mismatch in the cluster name(between USM and ceph cluster). Any requests send to ceph cluster, we append the cluster name(--cluster 'cluster-name'). Now with the above approach, we need to re-look at that and hard code it to ceph in all outgoing requests

3. Most of the events generated from calamari/skynet carries cluster name. We will have tough time re-mapping it to the cluster name which configured in USM.

  
These are few things which comes into my mind at this point. There may be other cases too.So my point is that if we go with this approach and miss some cases will end up in problem. So I am more inclined to fix this problem in the installer itself, so that we will have same name in both USM and ceph cluster

Comment 5 Christina Meno 2016-03-15 16:58:10 UTC
two questions:
What are the technical limitation of many clusters tracked by Storage Console all being named ceph? Or is it just bad UX?

Alternatively would you please share all the placed you want to show cluster name and don't have the FSID to do the correlation.

Comment 6 Nishanth Thomas 2016-03-17 09:22:55 UTC
As we discussed today's meeting, it will be a good amount of work to figure out pieces of code where we use cluster name and make translations. Also the UI flow needs to changed. Again at some point I am sure that this will come back and then we need revert all these changes back.

Comment 7 Christina Meno 2016-03-18 16:02:22 UTC
moving to to 2.1 until we negotiate it back in 2.0
This is 3 weeks of effort for the ceph-installer to make this change

Comment 8 Andrew Schoen 2016-04-04 15:16:10 UTC
Support for this in ceph-ansible is currently in the 1.0.3 version. Support for this in ceph-installer was merged with https://github.com/ceph/ceph-installer/pull/129, but does not exist in a built version of ceph-installer yet.

Comment 17 Martin Kudlej 2016-06-21 09:38:40 UTC
Checked with ceph-installer-1.0.11-1.el7scon.noarch and it is possible to create cluster with user defined name. -> Verified

Comment 19 errata-xmlrpc 2016-08-23 19:48:26 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHEA-2016:1754


Note You need to log in before you can comment on or make changes to this bug.