Project

General

Profile

Actions

Bug #20415

closed

The technique editor allows saving methods with missing parameters

Added by Alexis Mousset over 2 years ago. Updated about 2 years ago.

Status:
Released
Priority:
N/A
Category:
Web - Technique editor
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
To do
Fix check:
To do
Regression:

Description

This leads to useless error messages for the user:

Error when saving technique: The server had a problem, try again later

and an error log in the webapp:

[2021-12-14 13:03:18+0000] ERROR techniques.writer - An error occurred when compiling technique 'Inventory hook for updates' (id : 'BundleName(Inventory_hook_for_updates)') with rudderc, error details in /var/log/rudder/rudderc/failures/2021-12-14T13:03:18.901Z_Inventory_hook_for_updates.log, falling back to old saving process
Fiber failed.
An unchecked error was produced.
com.normation.cfclerk.xmlparsers.EmptyReportKeysValue: In 'expectedReportKey File from local source', the element REPORTKEYS must have a non empty list of provided values: <REPORTKEYS><VALUE>val foo</VALUE><VALUE>...
    at com.normation.cfclerk.domain.SectionVariableSpec$.apply(VariableAndSectionSpec.scala:410)
    at com.normation.cfclerk.xmlparsers.VariableSpecParser.parseSectionVariableSpec(VariableSpecParser.scala:134)
    at com.normation.cfclerk.xmlparsers.SectionSpecParser.parseOneVariable$1(SectionSpecParser.scala:192)
    at com.normation.cfclerk.xmlparsers.SectionSpecParser.$anonfun$parseChildren$4(SectionSpecParser.scala:205)
    at cats.data.Chain$.$anonfun$traverseViaChain$1(Chain.scala:797)
    at cats.Later.value$lzycompute(Eval.scala:148)
Actions #1

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 7.0.0~rc2 to 7.0.0~rc3
Actions #2

Updated by Elaad FURREEDAN over 2 years ago

  • Status changed from New to In progress
  • Assignee set to Elaad FURREEDAN
Actions #3

Updated by Elaad FURREEDAN about 2 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Elaad FURREEDAN to Nicolas CHARLES
  • Pull Request set to https://github.com/Normation/rudder/pull/4073
Actions #4

Updated by Anonymous about 2 years ago

  • Status changed from Pending technical review to Pending release
Actions #5

Updated by Vincent MEMBRÉ about 2 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 7.0.0~rc3 which was released today.

Actions

Also available in: Atom PDF