Project

General

Profile

Actions

Architecture #3028

closed

Make the CFEngine process checking promise in Rudder more clean and targeted

Added by Matthieu CERDA about 10 years ago. Updated about 10 years ago.

Status:
Released
Priority:
3
Assignee:
Matthieu CERDA
Category:
System techniques
Target version:
Effort required:
Regression:

Description

Make the CFEngine process checking promise in Rudder more clean and targeted.

As of now, this promise contains a lot of code duplication, and targets ALL the CFEngine processes (even the currently running one.)

We should avoid this and make it more clean.

Pull request linked to this US: https://github.com/Normation/rudder-techniques/pull/4


Files

Actions #1

Updated by Matthieu CERDA about 10 years ago

  • Description updated (diff)
Actions #2

Updated by Matthieu CERDA about 10 years ago

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

Pull request available, to be reviewed.

Actions #3

Updated by Matthieu CERDA about 10 years ago

I removed the feature Jon was talking about in the diff (the process_select), but this feature also applies to this user story as well. Should I add a subtask to this ticket ?

Enclosed to this comment is the patch to apply this new feature, for easier review and applying later.

Actions #4

Updated by Nicolas CHARLES about 10 years ago

  • Status changed from Pending technical review to 10

The pull request has been merged

Actions #5

Updated by Nicolas CHARLES about 10 years ago

  • Status changed from 10 to Released

Thank you Matthieu !

Actions #6

Updated by Jonathan CLARKE about 10 years ago

  • Tracker changed from User story to Architecture
Actions #7

Updated by Jonathan CLARKE about 10 years ago

  • Status changed from Released to Pending release
Actions #8

Updated by Jonathan CLARKE about 10 years ago

Matthieu CERDA wrote:

I removed the feature Jon was talking about in the diff (the process_select), but this feature also applies to this user story as well. Should I add a subtask to this ticket ?

Enclosed to this comment is the patch to apply this new feature, for easier review and applying later.

No, this is a different change. Please create a new ticket and explain the logic behind your suggestion (and make a pull request with the patch, of course :) )

Actions #9

Updated by Jonathan CLARKE about 10 years ago

  • Target version changed from 2.4.0~rc2 to 2.5.0~beta1

This was NOT on 2.4 branch, but on master! Please pay attention to target versions...

Actions #10

Updated by Matthieu CERDA about 10 years ago

  • Status changed from Pending release to Released
Actions

Also available in: Atom PDF