Project

General

Profile

Actions

Architecture #7310

closed

Refactor & clean up promise writting part

Architecture #7310: Refactor & clean up promise writting part

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

Status:
Released
Priority:
1 (highest)
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 CHARLESActions

Related issues 2 (0 open2 closed)

Related to Rudder - Architecture #6948: Merge cf-clerk into RudderReleasedVincent MEMBRÉActions
Related to Rudder - Bug #7442: License file is not generated any moreReleasedNicolas CHARLESActions

Updated by François ARMAND about 10 years ago Actions #2

Updated by François ARMAND about 10 years ago Actions #3

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

Updated by Nicolas CHARLES about 10 years ago Actions #5

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

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

  • 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: PDF Atom