Project

General

Profile

Actions

User story #3153

closed

User story #3137: Using certains CFEngine variables in directives may lead to unexpected errors in the promises generation

Update the LDAP init script to use the new internal Rudder variable format

Added by Nicolas CHARLES almost 12 years ago. Updated over 9 years ago.

Status:
Released
Priority:
1 (highest)
Category:
Architecture - Code maintenance
Target version:
UX impact:
Suggestion strength:
User visibility:
Effort required:
Name check:
Fix check:
Regression:

Description

We changed the format of internal Rudder variable from ${xxx.yyy} to ${rudder.xxx.yyy}, we should update the ldap init script accordingly

Actions #1

Updated by Nicolas CHARLES almost 12 years ago

  • Status changed from 8 to In progress
Actions #2

Updated by Nicolas CHARLES almost 12 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Nicolas CHARLES to Jonathan CLARKE

The pull request is here :
https://github.com/Normation/rudder/pull/64

I guess this modification should be enough, but I'm not sure if there are no scripts in the packaging relying on this

Actions #3

Updated by Jonathan CLARKE almost 12 years ago

  • Status changed from Pending technical review to 12
  • Assignee changed from Jonathan CLARKE to Nicolas CHARLES

Looks good to me.

Actions #4

Updated by Jonathan CLARKE almost 12 years ago

Since #3149 has been merged (a bit prematurely), I'll merge this too.

Actions #5

Updated by Nicolas CHARLES almost 12 years ago

  • Status changed from 12 to Pending release
  • % Done changed from 0 to 100
Actions #6

Updated by Jonathan CLARKE almost 12 years ago

Actions #8

Updated by Jonathan CLARKE almost 12 years ago

  • Category changed from 11 to Architecture - Code maintenance
Actions #9

Updated by François ARMAND over 11 years ago

  • Status changed from Pending release to Released
Actions #10

Updated by Benoît PECCATTE over 9 years ago

  • Tracker changed from 6 to User story
Actions

Also available in: Atom PDF