Missed doc for section nodes in config file

Bug #1660326 reported by Mikhail Zhnichkov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
fuel-ccp
Fix Released
Undecided
Mikhail Zhnichkov

Bug Description

For section nodes in config we can specify node[1-9], but can not
node[2-12], because it is a wrong regex expression. We should use
node([2-9]|1[0-2]) and its should be documented

Revision history for this message
Sergey Reshetnyak (sreshetniak) wrote :

node[2-12] is not valid regexp

Changed in fuel-ccp:
status: New → Invalid
description: updated
summary: - Wrong node matcher
+ Missed doc for section nodes in config file
Changed in fuel-ccp:
status: Invalid → New
Revision history for this message
Kirill Bespalov (k-besplv) wrote :

> We should use
> node([2-9]|1[0-2])

Also must be documented that a user always must specify the "end-of-line" in reg-exp like:

 node1$:
   roles: [a,b,c]

otherwise will be matched node10, node11, etc

Changed in fuel-ccp:
assignee: nobody → Mikhail Zhnichkov (mzhnichkov)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-ccp (master)

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

Changed in fuel-ccp:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-ccp (master)

Reviewed: https://review.openstack.org/427063
Committed: https://git.openstack.org/cgit/openstack/fuel-ccp/commit/?id=b0bae382d70a87cba6385627d53ecb24f0d188ab
Submitter: Jenkins
Branch: master

commit b0bae382d70a87cba6385627d53ecb24f0d188ab
Author: Mikhail <email address hidden>
Date: Tue Jan 31 12:54:35 2017 +0400

    Add description for nodes section

    Change-Id: I16462dc851c46262bbb43a109cc5bb1380a6ca3e
    Closes-Bug: #1660326

Changed in fuel-ccp:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.