Project

General

Profile

Actions

Bug #3741

closed

Technique "Name resolution": Regexp to check value of Nameserver does not allow the use of variable

Added by Nicolas PERRON over 11 years ago. Updated almost 9 years ago.

Status:
Released
Priority:
N/A
Category:
Techniques
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

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


Subtasks 1 (0 open1 closed)

Bug #7806: Technique "Name resolution": Regexp to check value of Nameserver does not allow the use of variable - version 3.0 (branch 3.0)ReleasedNicolas CHARLES2016-01-19Actions
Actions #1

Updated by Nicolas PERRON over 11 years ago

  • Target version changed from 2.4.7 to 2.4.8
Actions #2

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)

Actions #3

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

Actions #4

Updated by Nicolas PERRON about 11 years ago

  • Target version changed from 2.4.8 to 2.4.9
Actions #5

Updated by Nicolas PERRON about 11 years ago

  • Target version changed from 2.4.9 to 2.4.10
Actions #6

Updated by Nicolas PERRON about 11 years ago

  • Target version changed from 2.4.10 to 2.4.11
Actions #7

Updated by Nicolas PERRON about 11 years ago

  • Target version changed from 2.4.11 to 2.4.12
Actions #8

Updated by Nicolas PERRON about 11 years ago

  • Target version changed from 2.4.12 to 2.4.13
Actions #9

Updated by Vincent MEMBRÉ almost 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

Actions #10

Updated by Vincent MEMBRÉ over 10 years ago

  • Target version changed from 2.6.11 to 2.6.12
Actions #11

Updated by Vincent MEMBRÉ over 10 years ago

  • Target version changed from 2.6.12 to 2.6.13
Actions #12

Updated by Nicolas PERRON over 10 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 ?

Actions #13

Updated by Vincent MEMBRÉ over 10 years ago

  • Target version changed from 2.6.13 to 2.6.14
Actions #14

Updated by Jonathan CLARKE over 10 years ago

  • Target version changed from 2.6.14 to 2.6.16
Actions #15

Updated by Jonathan CLARKE over 10 years ago

  • Target version changed from 2.6.16 to 2.6.17
Actions #16

Updated by Nicolas PERRON over 10 years ago

  • Target version changed from 2.6.17 to 2.6.18
Actions #17

Updated by Matthieu CERDA about 10 years ago

  • Target version changed from 2.6.18 to 2.6.19
Actions #18

Updated by Vincent MEMBRÉ about 10 years ago

  • Target version changed from 2.6.19 to 2.6.20
Actions #19

Updated by François ARMAND almost 10 years ago

  • Assignee deleted (Jonathan CLARKE)
  • Target version changed from 2.6.20 to 2.10.10
Actions #20

Updated by Vincent MEMBRÉ almost 10 years ago

  • Target version changed from 2.10.10 to 2.10.11
Actions #21

Updated by Vincent MEMBRÉ over 9 years ago

  • Target version changed from 2.10.11 to 2.10.12
Actions #22

Updated by Benoît PECCATTE over 9 years ago

  • Project changed from 24 to Rudder
  • Category set to Techniques
Actions #23

Updated by Vincent MEMBRÉ over 9 years ago

  • Target version changed from 2.10.12 to 2.10.13
Actions #24

Updated by Vincent MEMBRÉ over 9 years ago

  • Target version changed from 2.10.13 to 2.10.14
Actions #25

Updated by Vincent MEMBRÉ over 9 years ago

  • Target version changed from 2.10.14 to 2.10.15
Actions #26

Updated by Vincent MEMBRÉ over 9 years ago

  • Target version changed from 2.10.15 to 2.10.16
Actions #27

Updated by Vincent MEMBRÉ over 9 years ago

  • Target version changed from 2.10.16 to 2.10.17
Actions #28

Updated by Vincent MEMBRÉ about 9 years ago

  • Target version changed from 2.10.17 to 2.10.18
Actions #29

Updated by Vincent MEMBRÉ about 9 years ago

  • Target version changed from 2.10.18 to 2.10.19
Actions #30

Updated by Vincent MEMBRÉ about 9 years ago

  • Target version changed from 2.10.19 to 2.10.20
Actions #31

Updated by Vincent MEMBRÉ almost 9 years ago

  • Target version changed from 2.10.20 to 2.11.18
Actions #32

Updated by Jonathan CLARKE almost 9 years ago

  • Status changed from Discussion to In progress
  • Assignee set to Jonathan CLARKE
Actions #33

Updated by Jonathan CLARKE almost 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
Actions #34

Updated by Jonathan CLARKE almost 9 years ago

  • Status changed from Pending technical review to Pending release
  • % Done changed from 0 to 100
Actions #35

Updated by Vincent MEMBRÉ almost 9 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 2.11.18, 3.0.13, 3.1.6 and 3.2.0 which were released today.

Actions

Also available in: Atom PDF