rhel-osp-director: [RFE] Need to add a check for the total count of the nodes - that it doesn't exceed the amount of available nodes in ironic. Environment: puppet-ironic-9.4.1-1.el7ost.noarch openstack-tripleo-heat-templates-5.1.0-7.el7ost.noarch python-ironicclient-1.7.0-1.el7ost.noarch openstack-ironic-api-6.2.2-2.el7ost.noarch python-ironic-lib-2.1.1-2.el7ost.noarch python-ironic-inspector-client-1.9.0-2.el7ost.noarch instack-undercloud-5.1.0-4.el7ost.noarch openstack-ironic-inspector-4.2.1-1.el7ost.noarch openstack-ironic-conductor-6.2.2-2.el7ost.noarch openstack-ironic-common-6.2.2-2.el7ost.noarch I attempted a deployment with this amount of nodes: ControllerCount: 3 ComputeCount: 2 CephStorageCount: 3 ServiceApiCount: 2 Total: 10 The amount of nodes in ironic: ceph-0 ceph-1 serviceapi-0 serviceapi-1 controller-0 controller-1 controller-2 compute-0 compute-1 Total: 9 When I started the overcloud deployment - it proceeded until failed with: 2016-12-23 20:03:29Z [overcloud.Compute]: CREATE_FAILED CREATE aborted 2016-12-23 20:03:30Z [overcloud]: CREATE_FAILED Resource CREATE failed: ResourceInError: resources.CephStorage.resources[2].resources.CephStorage: Went to status ERROR due to "Message: No valid host was found. Th ere are not enough hosts available., Code: 500" Expected result: The deployment shouldn't even start when the count of nodes per role in deployment exceeds the count of nodes with respective tag in ironic.
UI now validates Node counts against count of ironic nodes available for deployment
Can you add the gerrit id please so that we can check if we've the relevant change? Thanks
Jiri, can you explain me what is the fix here? The GUI always allowed you to choose only up to the number of available nodes, so what's new here? I also think they meant to validate this in the templates, so that the CLI users will see a clear error soon when the deployment starts. This fix is only for the GUI, right? Thanks.
The referenced patch links to the work which introduced changes in nodes tagging/assignment. It is referenced to ensure that these changes did retain the GUI behavior. So yes, that patch is only GUI related. This RFE needs deploy_plan workflow in tripleo-common to validate the node counts.
(In reply to Jiri Tomasek from comment #9) > The referenced patch links to the work which introduced changes in nodes > tagging/assignment. It is referenced to ensure that these changes did retain > the GUI behavior. So yes, that patch is only GUI related. > > This RFE needs deploy_plan workflow in tripleo-common to validate the node > counts. The intention of the request was adding functionality in CLI. The fix is partial, only in UI, CLI part is missing. Proposing to move the CLI part to Queens.