Project

General

Profile

Actions

Architecture #12335

closed

typo in name of method used in write_all_techniques

Added by Vincent MEMBRÉ almost 5 years ago. Updated 9 months ago.

Status:
Released
Priority:
N/A
Category:
Web - Technique editor
Target version:
Effort required:
Regression:

Description

Almost all information of a ncf techniques is read for the metadata lines, but only one thing is missing ... method calls.

They are currently set using a cf-promises command on each technique then parsing this output to extract method calls.

I clearly think we should not do this anymore and instead store that information in metadata.

This would allow to create a technique without writing a single line of cfengine!

This will also give a performance improvement when loading techniques ( N cf-promises less, with N less json parsing !)


Related issues 1 (0 open1 closed)

Related to Rudder - Architecture #12044: Add a script to rewrite a technique from command lineReleasedVincent MEMBRÉActions
Actions #1

Updated by Vincent MEMBRÉ almost 5 years ago

  • Status changed from New to In progress
  • Assignee set to Vincent MEMBRÉ
Actions #2

Updated by Vincent MEMBRÉ almost 5 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Vincent MEMBRÉ to Nicolas CHARLES
  • Pull Request set to https://github.com/Normation/ncf/pull/722
Actions #3

Updated by Vincent MEMBRÉ almost 5 years ago

  • Parent task deleted (#12044)
Actions #4

Updated by Vincent MEMBRÉ almost 5 years ago

Actions #5

Updated by Rudder Quality Assistant almost 5 years ago

  • Assignee changed from Nicolas CHARLES to Vincent MEMBRÉ
Actions #6

Updated by Vincent MEMBRÉ almost 5 years ago

  • Status changed from Pending technical review to Pending release

Applied in changeset commit:ef6eec5e81af0f1847037128a4d73bcd9f3f0733.

Actions #7

Updated by Vincent MEMBRÉ almost 5 years ago

  • Target version changed from 4.3.0~rc3 to 4.3.0~rc2
Actions #8

Updated by Vincent MEMBRÉ almost 5 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 4.3.0~rc2 which was released today.

Actions #9

Updated by Alexis Mousset 9 months ago

  • Project changed from 41 to Rudder
  • Category changed from Technique editor - API to Web - Technique editor
Actions

Also available in: Atom PDF