Project

General

Profile

Actions

User story #4200

closed

User story #4199: Addition of "Technique with values" in Rudder - ncf integration

Add PREDEF variable type and ProvideCsvReportTemplate in metadata.xml

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

Status:
Released
Priority:
1
Category:
Web - Config management
Target version:
UX impact:
Suggestion strength:
User visibility:
Effort required:
Name check:
Fix check:
Regression:

Description

From an implementation point of view, we need to add a new variable type that allows to represent a variable whose values are provided in the spec.
Here, we don't talk about a default value, but a list of variables with a value, maybe different each time.
Those variable will be use to know what are the expected report keys name.

So, the following spec:

    <SECTION name="Some generic method">
      <REPORTKEYS>
        <VALUE> val 1 </VALUE><VALUE> val 2 </VALUE>
      </REPORTKEY>
    </SECTION>

Leads to the creation of two variable that will inform Rudder to await for expected reports for component "Some generic method" with key name "val 1" and "val 2"

Some more specification details:

- only one reportkeys is allowed for a given section
- a section with a <REPORTKEYS> element is marked "multivalued=true, isComponent=true, componentKey=[calculated from section name]"

Also, for the integration with ncf, new Technique will comes with a template of expected reports.
We need to be able to know in a Technique object if the Technique is a new one, with the CSV file, or an old one, without it.
Knowing that at parsing time for that will allow to validate the presence of the CSV at the expeted place and adapt the expected report process to use it.

We can infer the "ProvideCsvReportTemplate" nature by the existence of at least "REPORTKEYS" tag.


Related issues 1 (0 open1 closed)

Related to Rudder - User story #4202: Update documentation about TechniqueXML with new elements RejectedAlexis Mousset2013-11-29Actions
Actions #1

Updated by François ARMAND over 10 years ago

  • Subject changed from Add PREDEF variable type in metadata.xml to Add PREDEF variable type and ProvideCsvReportTemplate in metadata.xml
  • Description updated (diff)
Actions #2

Updated by François ARMAND over 10 years ago

  • Description updated (diff)
Actions #4

Updated by François ARMAND over 10 years ago

  • Description updated (diff)
Actions #5

Updated by François ARMAND over 10 years ago

  • Description updated (diff)
  • Status changed from In progress to Discussion

As seen in a simplification pass:

- we don't need a tag for the kind of technique, we can infer it from the existence of at least one provided reports element in a section
- for the expetect reports keys value, we will use the <REPORTKEYS> tag.

Actions #6

Updated by François ARMAND over 10 years ago

  • Status changed from Discussion to In progress

Bad status modification in previous update.

Actions #7

Updated by Nicolas CHARLES over 10 years ago

based on this code, i created the https://github.com/Normation/cf-clerk/pull/27 that included code necessary for #4201 also

Actions #8

Updated by Nicolas CHARLES over 10 years ago

  • Pull Request set to https://github.com/Normation/cf-clerk/pull/28
Actions #9

Updated by François ARMAND over 10 years ago

  • Status changed from In progress to Pending technical review
Actions #10

Updated by François ARMAND over 10 years ago

  • Description updated (diff)
Actions #11

Updated by François ARMAND over 10 years ago

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

Applied in changeset clerk:commit:c3fb01d0ac5844c2597dd0d1074bce26fbf1b50b.

Actions #12

Updated by Vincent MEMBRÉ over 10 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 2.9.0~rc1, which was released on Friday 13/12/2013.
Check out:

Actions #13

Updated by Benoît PECCATTE almost 9 years ago

  • Tracker changed from Enhancement to User story
Actions

Also available in: Atom PDF