Project

General

Profile

Actions
Copy link

User story #6099

closed

We should be able to call actions as many times as we want

User story #6099: We should be able to call actions as many times as we want

Added by Benoît PECCATTE over 11 years ago. Updated about 4 years ago.

Status:
Resolved
Priority:
N/A
Assignee:
Félix DALLIDET
Category:
Generic methods
Target version:
7.1.0~rc1
Pull Request:
UX impact:
Suggestion strength:
User visibility:
Effort required:
Medium
Name check:
Fix check:
Regression:

Description

Some promises are not state promises but action promises.
Namely service_restart and service_reloas and maby some others are action promises.

Unlike state promises that can be checked and called only once, we should be able to call such promises as many times as we want.
For this purpose, https://dev.cfengine.com/issues/6893 must be solved, and we should use ifelapsed 0 in action promises.

Subtasks 3 (0 open — 3 closed)

Architecture #15783: Make class_prefix uniqueResolvedNicolas CHARLESActions
User story #15784: Base log on the directive id and add a canonified directive id in report_dataReleasedNicolas CHARLESActions
Bug #16078: Applying directives using the exact same GM with same parameters will always result in missing reportsResolvedAlexis MoussetActions

Related issues 2 (0 open — 2 closed)

Related to Rudder - Bug #5991: On a freshly installed node, rsyslog is dropping messagesReleasedBenoît PECCATTEActions
Related to Rudder - Architecture #6101: We should be able to call ncf generic methods more than onceRejectedActions

Updated by Alexis Mousset almost 10 years ago Actions
Copy link
 #1

  • Category set to Generic methods

Updated by Alexis Mousset over 9 years ago Actions
Copy link
 #2

  • Target version changed from 0.x to master

Updated by Vincent MEMBRÉ over 8 years ago Actions
Copy link
 #3

  • Target version changed from master to 4.2.2

Updated by Vincent MEMBRÉ over 8 years ago Actions
Copy link
 #4

  • Target version changed from 4.2.2 to 4.2.3

Updated by Vincent MEMBRÉ over 8 years ago Actions
Copy link
 #5

  • Target version changed from 4.2.3 to 4.2.4

Updated by Benoît PECCATTE over 8 years ago Actions
Copy link
 #6

  • Target version changed from 4.2.4 to Ideas (not version specific)

Updated by Félix DALLIDET over 6 years ago Actions
Copy link
 #7

  • Target version changed from Ideas (not version specific) to 5.0.14

Updated by Félix DALLIDET over 6 years ago Actions
Copy link
 #8

  • Status changed from New to In progress
  • Assignee set to Félix DALLIDET

Updated by Vincent MEMBRÉ over 6 years ago Actions
Copy link
 #9

  • Target version changed from 5.0.14 to 5.0.15

Updated by Félix DALLIDET over 6 years ago Actions
Copy link
 #11

  • Effort required set to Medium

We should define a dedicated bundle to save the current GM index call:


  methods:
    "Variable string_context_${report_data.directive_id}_0" usebundle => _method_reporting_context("Variable string", "cat");
                                                         ^
                                                         |
                                                current GM index call

This way, we could use this index, concatenated with the directive id to make GM/promiser re-usable by adding comments, ifvarclass, etc...
based on: directive_id_index

In the same fashion suggested in a mix of: 15783 and 16078.
It solves both issues.

Updated by Vincent MEMBRÉ over 6 years ago Actions
Copy link
 #12

  • Target version changed from 5.0.15 to 5.0.16

Updated by Alexis Mousset over 6 years ago Actions
Copy link
 #13

  • Target version changed from 5.0.16 to 5.0.17

Updated by Vincent MEMBRÉ about 6 years ago Actions
Copy link
 #14

  • Target version changed from 5.0.17 to 5.0.18

Updated by Benoît PECCATTE about 6 years ago Actions
Copy link
 #15

  • Target version changed from 5.0.18 to 6.2.0~beta1

Updated by Vincent MEMBRÉ over 5 years ago Actions
Copy link
 #16

  • Target version changed from 6.2.0~beta1 to 6.2.0~rc1

Updated by François ARMAND over 5 years ago Actions
Copy link
 #17

  • Target version changed from 6.2.0~rc1 to 7.0.0~beta1

Updated by Alexis Mousset almost 5 years ago Actions
Copy link
 #18

  • Status changed from In progress to New

Updated by Vincent MEMBRÉ almost 5 years ago Actions
Copy link
 #19

  • Target version changed from 7.0.0~beta1 to 7.0.0~beta2

Updated by Benoît PECCATTE over 4 years ago Actions
Copy link
 #20

  • Project changed from 41 to Rudder
  • Category changed from Generic methods to Generic methods

Updated by Vincent MEMBRÉ over 4 years ago Actions
Copy link
 #21

  • Target version changed from 7.0.0~beta2 to 7.0.0~beta3

Updated by Alexis Mousset over 4 years ago Actions
Copy link
 #22

  • Target version changed from 7.0.0~beta3 to 7.1.0~beta1

Updated by Vincent MEMBRÉ about 4 years ago Actions
Copy link
 #23

  • Target version changed from 7.1.0~beta1 to 7.1.0~beta2

Updated by Vincent MEMBRÉ about 4 years ago Actions
Copy link
 #24

  • Target version changed from 7.1.0~beta2 to 7.1.0~rc1

Updated by Alexis Mousset about 4 years ago Actions
Copy link
 #25

done in 7.1!

Updated by Alexis Mousset about 4 years ago Actions
Copy link
 #26

  • Status changed from New to Resolved
Actions
Copy link

Also available in: PDF Atom