Project

General

Profile

Actions
Copy link

Architecture #16858

closed

Architecture #16856: Don't replace RudderUniqueId in generated policies with a replace, but with StringTemplate

create a migration script to replace all RudderUniqueID by & RudderUniqueID& in techniques

Added by Nicolas CHARLES almost 5 years ago. Updated over 4 years ago.

Status:
Released
Priority:
N/A
Assignee:
Benoît PECCATTE
Category:
Packaging
Target version:
6.1.0~beta1
Pull Request:
https://github.com/Normation/rudder-p...
Effort required:
Name check:
To do
Fix check:
To do
Regression:

Description

see parent ticket

Actions
Copy link
 #1

Updated by Nicolas CHARLES almost 5 years ago

regex

 sed "s/\([^&]\)RudderUniqueID\([^&]\)/\1\&RudderUniqueID\&\2/g"

seems to do the trick, except when a line starts or finished by RudderUniqueID

Actions
Copy link
 #2

Updated by Nicolas CHARLES almost 5 years ago

4 seds work but seems a bit overkill

sed "s/\([^&]\)RudderUniqueID\([^&]\)/\1\&RudderUniqueID\&\2/g" test.st | sed "s/^RudderUniqueID\([^&]\)/\&RudderUniqueID\&\1/g" | sed "s/\([^&]\)RudderUniqueID$/\1\&RudderUniqueID\&/g" | sed "s/^RudderUniqueID$/\&RudderUniqueID\&/g"

Actions
Copy link
 #3

Updated by Nicolas CHARLES almost 5 years ago

sed "s/\([^&]\|^\)RudderUniqueID\([^&]\|$\)/\1\&RudderUniqueID\&\2/g"

Actions
Copy link
 #4

Updated by Nicolas CHARLES almost 5 years ago

  • Status changed from New to In progress
  • Assignee set to Nicolas CHARLES
Actions
Copy link
 #5

Updated by Nicolas CHARLES almost 5 years ago

  • Assignee changed from Nicolas CHARLES to Benoît PECCATTE
  • Pull Request set to https://github.com/Normation/rudder-packages/pull/2214
Actions
Copy link
 #6

Updated by Nicolas CHARLES over 4 years ago

  • Status changed from In progress to Pending release
Actions
Copy link
 #7

Updated by François ARMAND over 4 years ago

  • Tracker changed from Bug to Architecture
  • Priority deleted (0)
Actions
Copy link
 #8

Updated by Vincent MEMBRÉ over 4 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 6.1.0~beta1 which was released today.

Actions
Copy link

Also available in: Atom PDF