Project

General

Profile

Architecture #12335

typo in name of method used in write_all_techniques

Added by Vincent MEMBRÉ 8 months ago. Updated 8 months ago.

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

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

Related to ncf - Architecture #12044: Add a script to rewrite a technique from command lineReleased

Associated revisions

Revision ef6eec5e (diff)
Added by Vincent MEMBRÉ 8 months ago

Fixes #12335: typo in name of method used in write_all_techniques

History

#1 Updated by Vincent MEMBRÉ 8 months ago

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

#2 Updated by Vincent MEMBRÉ 8 months 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

#3 Updated by Vincent MEMBRÉ 8 months ago

  • Parent task deleted (#12044)

#4 Updated by Vincent MEMBRÉ 8 months ago

#5 Updated by Normation Quality Assistant 8 months ago

  • Assignee changed from Nicolas CHARLES to Vincent MEMBRÉ

#6 Updated by Vincent MEMBRÉ 8 months ago

  • Status changed from Pending technical review to Pending release

#7 Updated by Vincent MEMBRÉ 8 months ago

  • Target version changed from 4.3.0~rc3 to 4.3.0~rc2

#8 Updated by Vincent MEMBRÉ 8 months ago

  • Status changed from Pending release to Released

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

Also available in: Atom PDF