Project

General

Profile

Actions

Architecture #6948

closed

Merge cf-clerk into Rudder

Added by François ARMAND over 9 years ago. Updated about 9 years ago.

Status:
Released
Priority:
1 (highest)
Category:
Architecture - Code maintenance
Target version:
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 open3 closed)

Architecture #6990: Remove cf-clerk from rudder-packagesReleasedBenoît PECCATTE2015-07-15Actions
Bug #7302: Test TestPackageReader.scala in errorReleasedVincent MEMBRÉ2015-10-23Actions
Bug #7360: Exception: Exception caught during polcy update process: empty.maxReleasedNicolas CHARLES2015-11-03Actions

Related issues 2 (0 open2 closed)

Related to Rudder - Architecture #7310: Refactor & clean up promise writting partReleasedFrançois ARMAND2015-10-28Actions
Has duplicate Rudder - Architecture #5311: Move code related to promise generation from cf-clerck to RudderRejectedFrançois ARMAND2014-07-24Actions
Actions #1

Updated by François ARMAND over 9 years ago

  • 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
Actions #2

Updated by François ARMAND over 9 years ago

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

Updated by Jonathan CLARKE over 9 years ago

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

Updated by François ARMAND about 9 years ago

Actions #6

Updated by Vincent MEMBRÉ about 9 years ago

  • Status changed from Pending release to Released

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

Actions

Also available in: Atom PDF