Bug #19767: Error when saving a technique with an empty block - Rudder - Issue Tracker

Actions

Copy link

Bug #19767

closed

Error when saving a technique with an empty block

Added by Nicolas CHARLES over 3 years ago. Updated about 3 years ago.

Status:

Rejected

Priority:

N/A

Assignee:

Category:

Web - Technique editor

Target version:

7.0.0~rc1

Pull Request:

Severity:

UX impact:

User visibility:

Effort required:

Priority:

Name check:

To do

Fix check:

To do

Regression:

Description

[2021-08-12 16:38:43+0200] INFO  compliance - [metrics] global compliance (number of components): undefined
[2021-08-12 16:54:15+0200] ERROR application - An error occurred when compiling technique 'Bob' (id : 'BundleName(Bob)') with rudderc, error details in /var/log/rudder/rudderc/failures/2021-08-12T16:54:15.619+02:00_Bob.log, falling back to old saving process
Fiber failed.
An unchecked error was produced.
com.normation.cfclerk.xmlparsers.EmptyReportKeysValue: In 'expectedReportKey File absent', 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:403)
        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)
        at cats.Later.value(Eval.scala:147)
        at cats.Eval$.loop$1(Eval.scala:338)

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by François ARMAND about 3 years ago

Target version changed from 7.0.0~beta1 to 900

We need to choose what is the expected behavior: accept (and correct code), or forbid empty block at all. (I think empty blocks don't make sens, and if it's really the case, we need to reject them).

Actions

Copy link