Bug #3741
closed
Technique "Name resolution": Regexp to check value of Nameserver does not allow the use of variable
Description
We should be able to use CFEngine variables in the Nameserver option of the Technique Name resolution. Actually, this is not possible since the regexp prevent from using other values than IP adresses:
Please modify Nameserver to match the requested format : You must enter a valid IP
Updated by Vincent MEMBRÉ over 11 years ago
- Status changed from New to 8
- Priority changed from N/A to 1 (highest)
The regexp should not accept cfengine variable, because we can't check their value in Rudder and they may not respect the regexp.
We should try to replace all cfengine variable by their value (only if there is a regexp defined) before the validation,
this exclude direct cfengine variable ($sys.xxx) because Rudder can't check them.
Variable defined using Technique "CFEngine variable definition" may be checked, but I don't know if this is something we want to do
Inventory variable should be checked too, but I don't know how and which way to validate it (maybe the value does not exists for that node, maybe it is not a valide value,)
The easiest way is to use global parameters, global parameters can be replaced easily. and checked easily (replace the global parameters by its value)
Updated by Vincent MEMBRÉ over 11 years ago
Another solution would be to accept any variable, but to display a warning explaining that Rudder can't validate that value, and let you use cfengine varaible at your own risk
Updated by Vincent MEMBRÉ about 11 years ago
- Target version changed from 2.4.13 to 2.6.11
Since 2.4 is not maintained anymore, retargeting to branch 2.6
Updated by Nicolas PERRON almost 11 years ago
- Status changed from 8 to Discussion
- Assignee set to Jonathan CLARKE
Vincent MEMBRÉ wrote:
Another solution would be to accept any variable, but to display a warning explaining that Rudder can't validate that value, and let you use cfengine varaible at your own risk
I suppose this solution is better, what's your opinion Jon ?
Updated by Matthieu CERDA 
Updated by 
Updated by
Updated by Jonathan CLARKE about 9 years ago
- Status changed from In progress to Pending technical review
- Assignee changed from Jonathan CLARKE to Nicolas CHARLES
- Pull Request set to https://github.com/Normation/rudder-techniques/pull/851
Updated by Jonathan CLARKE about 9 years ago
- Status changed from Pending technical review to Pending release
- % Done changed from 0 to 100
Applied in changeset rudder-techniques|415c448299438cd5eb7eeda532ff894c0ec7b29b.
Updated by Vincent MEMBRÉ about 9 years ago
- Status changed from Pending release to Released