Project

General

Profile

Actions
Copy link

Architecture #6948

closed

Merge cf-clerk into Rudder

Architecture #6948: Merge cf-clerk into Rudder

Added by François ARMAND over 10 years ago. Updated almost 10 years ago.

Status:
Released
Priority:
1 (highest)
Assignee:
Vincent MEMBRÉ
Category:
Architecture - Code maintenance
Target version:
3.2.0~beta1
Pull Request:
https://github.com/Normation/rudder/p...
Effort required:
Name check:
Fix check:
Regression:

Description

There is no reason to still have cf-clerk and Rudder splitted - it only brings bloat for no gain. Moreover, if the separation should be done today, it would certainly be done differently.

Moreover, the split will grandly complexify the dual promise generation logic in Rudder 3.2 to support one promise by directive output.

This first ticket is only to get source code from cf-clerk and put it into Rudder. It does not:

- change packaging so that cf-clerk is no more built or needed by Rudder
- refactor code to take advantage of the merge - not even renaming packages. Further ticket will allow to really take benefits from the merge.

Subtasks 3 (0 open — 3 closed)

Architecture #6990: Remove cf-clerk from rudder-packagesReleasedBenoît PECCATTEActions
Bug #7302: Test TestPackageReader.scala in errorReleasedVincent MEMBRÉActions
Bug #7360: Exception: Exception caught during polcy update process: empty.maxReleasedNicolas CHARLESActions

Related issues 2 (0 open — 2 closed)

Related to Rudder - Architecture #7310: Refactor & clean up promise writting partReleasedFrançois ARMANDActions
Has duplicate Rudder - Architecture #5311: Move code related to promise generation from cf-clerck to RudderRejectedFrançois ARMANDActions

Updated by François ARMAND over 10 years ago Actions
Copy link
 #1

  • Status changed from In progress to Pending technical review
  • Assignee changed from François ARMAND to Vincent MEMBRÉ
  • Pull Request set to https://github.com/Normation/rudder/pull/895

Updated by François ARMAND over 10 years ago Actions
Copy link
 #2

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

Updated by Jonathan CLARKE over 10 years ago Actions
Copy link
 #4

  • Has duplicate Architecture #5311: Move code related to promise generation from cf-clerck to Rudder added

Updated by François ARMAND about 10 years ago Actions
Copy link
 #5

Updated by Vincent MEMBRÉ almost 10 years ago Actions
Copy link
 #6

  • Status changed from Pending release to Released

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

Actions
Copy link

Also available in: PDF Atom