Bug 1253628 - external ceph patches break tuskar based deploys
Summary: external ceph patches break tuskar based deploys
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat OpenStack
Classification: Red Hat
Component: openstack-tuskar
Version: 7.0 (Kilo)
Hardware: Unspecified
OS: Unspecified
high
unspecified
Target Milestone: y1
: 7.0 (Kilo)
Assignee: Marios Andreou
QA Contact: Luigi Toscano
URL:
Whiteboard:
: 1258631 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-08-14 10:22 UTC by Marios Andreou
Modified: 2023-02-22 23:02 UTC (History)
11 users (show)

Fixed In Version: openstack-tuskar-0.4.18-4.el7ost
Doc Type: Bug Fix
Doc Text:
Support for using a pre-existing external Ceph Storage cluster with Overcloud deployments caused all deployments using a Tuskar plan to fail. This included deployments through the web UI and CLI when --plan is used instead of --templates. This was irrespective of whether any Ceph Storage nodes were deployed (external or otherwise). Failed deployments resulted in the following error: ERROR: openstack ERROR: Property error : CephClusterConfig: ceph_storage_count The Parameter (Ceph-Storage-1::CephStorageCount) was not provided. This fix modifies Tuskar so that it can work with nested references to the top level Heat templates scaling parameters (like CephStorageCount in this case). Deployments using Tuskar now function without encountering the CephStorageCount error.
Clone Of:
Environment:
Last Closed: 2015-10-08 12:16:47 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
OpenStack gerrit 213179 0 None None None Never
Red Hat Product Errata RHSA-2015:1862 0 normal SHIPPED_LIVE Moderate: Red Hat Enterprise Linux OpenStack Platform 7 director update 2015-10-08 16:05:50 UTC

Description Marios Andreou 2015-08-14 10:22:21 UTC
Description of problem:
After https://review.openstack.org/#/c/197734/ lands into tht tuskar based deploys will fail like

ERROR: openstack ERROR: Property error : CephClusterConfig: ceph_storage_count The Parameter (Ceph-Storage-1::CephStorageCount) was not provided.


This is because the scaling param CephStorageCount which is used to signal 'enable_ceph' in that patch is namespaced as Ceph-Storage-1::CephStorageCount

I think this may be due to special handling for the count params, am still investigating.

To be clear, once that lands, all --tuskar deploys will fail.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1. Use tripleo heat templates that include https://review.openstack.org/#/c/197734
2. openstack overcloud deploy plan --overcloud --compute-scale 1 --control-scale 1
3. ERROR: openstack ERROR: Property error : CephClusterConfig: ceph_storage_count The Parameter (Ceph-Storage-1::CephStorageCount) was not provided.

Actual results:
ERROR: openstack ERROR: Property error : CephClusterConfig: ceph_storage_count The Parameter (Ceph-Storage-1::CephStorageCount) was not provided.

Expected results:
No explosions.

Additional info:

Comment 3 Marios Andreou 2015-08-14 11:55:09 UTC
So it is indeed a clash with the special handling we do for the count params. Basically, what is CephStorageCount, becomes CephStorage-1::count and the original param is no more. The external ceph patch tries to reference this like  get_param: Ceph-Storage-1::CephStorageCount (tuskar also namespaces it).

Long-story short, using any other param name here would work fine. Am trying 'LeParam' for example:

in overcloud-without-mergepy.yaml

1181   CephClusterConfig:
1182     type: OS::TripleO::CephClusterConfig::SoftwareConfig
1183     properties:
1184       ceph_storage_count: {get_param: LeParam} # instead of CephStorageCount
1185       ceph_fsid: {get_param: CephClusterFSID}

 624   CephStorageCount:
 625     type: number
 626     default: 0
 627   LeParam:
 628     type: number
 629     default: 0

Using this the deploy with tuskar is OK. Am trying to come up with a happy compromise in the templates.

Comment 4 Marios Andreou 2015-08-14 14:46:27 UTC
propsed fixup @  https://review.openstack.org/#/c/213179/ (still needs more testing but fyi)

Comment 5 chris alfonso 2015-08-18 18:48:38 UTC
With us removing tuskar, we can probably not fix this.

Comment 6 Mike Burns 2015-09-01 02:00:54 UTC
*** Bug 1258631 has been marked as a duplicate of this bug. ***

Comment 7 Mike Burns 2015-09-01 02:04:50 UTC
It turns out that this breaks all deployments with tuskar which currently includes the UI.  Unless we plan on the UI not being an option even for POC, we need to at least make it work.  

One possible hackish solution would be to hard code the UI to pass the right value as 0 in all cases and document that you can't do any ceph deployment with the UI.

Comment 8 chris alfonso 2015-09-01 16:08:43 UTC
Ana, would you please sync up with marios on perhaps setting this in the UI, he can give you some guidance on where it could possibly be done.

Comment 9 Ana Krivokapic 2015-09-02 12:20:25 UTC
We decided to go with the tuskar fix rather than disable all Ceph deployments in the GUI.

Comment 11 nlevinki 2015-09-16 09:29:14 UTC
Since Tusker deprecated I think we should close this ticket ?

Comment 12 Marios Andreou 2015-09-16 09:31:26 UTC
I think the fix was still needed because of the UI like comment 7

Comment 13 Luigi Toscano 2015-10-01 17:07:36 UTC
Deployed two configurations from the web UI:
a) 1 controller + 1 compute + 1 Ceph storage
b) 1 controller + 1 compute

Each type of node had a specific flavor, which was associated (if defined) to the matching deployment role. The remaining roles were assigned to baremetal flavor.

Verified on a RHEL7.1 environment, relevant packages:

openstack-puppet-modules-2015.1.8-21.el7ost.noarch
openstack-tripleo-0.0.7-0.1.1664e566.el7ost.noarch
openstack-tripleo-common-0.0.1.dev6-3.git49b57eb.el7ost.noarch
openstack-tripleo-heat-templates-0.8.6-69.el7ost.noarch
openstack-tripleo-image-elements-0.9.6-10.el7ost.noarch
openstack-tripleo-puppet-elements-0.0.1-5.el7ost.noarch
openstack-tuskar-0.4.18-4.el7ost.noarch
openstack-tuskar-ui-0.4.0-3.el7ost.noarch
openstack-tuskar-ui-extras-0.0.4-1.el7ost.noarch
python-tuskarclient-0.1.18-4.el7ost.noarch

Comment 15 errata-xmlrpc 2015-10-08 12:16:47 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/RHSA-2015:1862


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