Project

General

Profile

Actions

Architecture #7310

closed

Refactor & clean up promise writting part

Added by François ARMAND over 8 years ago. Updated over 8 years ago.

Status:
Released
Priority:
1
Category:
Architecture - Code maintenance
Target version:
Effort required:
Name check:
Fix check:
Regression:

Description

The part of the generation of promise that write the files used to be splitted between cf-clerk and rudder project, leading to arbitrary changes, intermediate data structure, and on the whole, quite obscure code logic.

In the short time, we are going to introduce a "verify only" policy type, which need as a prerequisite to have Rudder be able to generate promises directive by directive. So this is a major change in the way promise file are written, and with the current tech debt on that part of the code, it's really, really hard to be done.

So that ticket is here to trace the first step on the path: refactore all the writting of promise, taking advantage of the reunification of rudder and cf-clerk, and clean the whole thing.

A direct user benefit is that it will be MUCH easier to optimise (and particulary, to parallelize) things after the rewrite, allowing better generation time.


Subtasks 1 (0 open1 closed)

Bug #7321: Cf3PromisesFileWriterServiceImpl.writeTemplate use a bad version of sequenceParReleasedNicolas CHARLES2015-10-28Actions

Related issues 2 (0 open2 closed)

Related to Rudder - Architecture #6948: Merge cf-clerk into RudderReleasedVincent MEMBRÉ2015-07-15Actions
Related to Rudder - Bug #7442: License file is not generated any moreReleasedNicolas CHARLES2015-11-23Actions
Actions #2

Updated by François ARMAND over 8 years ago

Actions #3

Updated by François ARMAND over 8 years ago

  • Status changed from In progress to Pending release
  • % Done changed from 0 to 100
Actions #5

Updated by Nicolas CHARLES over 8 years ago

  • Related to Bug #7442: License file is not generated any more added
Actions #6

Updated by Vincent MEMBRÉ over 8 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