Project

General

Profile

Actions

Bug #8606

closed

Condition from command: false codes is actually "repaired"

Added by Florian Heigl almost 8 years ago. Updated almost 2 years ago.

Status:
Rejected
Priority:
N/A
Assignee:
-
Category:
Generic methods
Target version:
-
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
Fix check:
Regression:

Description

Hi,

there's a mismatch in "Condition from command", it behaves much like "command execution result"
The help says:

false_codes:
List of codes that produce a false status separated with commas (ex: 3,4,6)
(I entered "1")

But if you look at the agent output you'll find this:
rudder info: Command related to promiser 'grep -qw /var/lib/one /proc/mounts' returned code defined as promise repaired 1

This looks like a bug.


Related issues 1 (0 open1 closed)

Related to Rudder - Bug #8610: Some usage docs are not correctly generated in html outputReleasedBenoƮt PECCATTEActions
Actions #1

Updated by Florian Heigl almost 8 years ago

footnote, if i've run into a deprected method again (I remember I had something like it last year: please mark them as deprecated in NCF where it's visible. I just can't keep track it in my head only)

Actions #2

Updated by Alexis Mousset almost 8 years ago

Hi Florian,

The command promise is repaired (we use the repaired and kept outcomes to create the right condition classes, as we want to keep error for unknown codes) but the ${condition_prefix}_false class should be defined.

It made me see that the HTML doc is broken for this generic method (I just opened #8610 to fix this), and is not understandable currently, you can read the source for explanation before it gets fixed.

I will change the help to explain the repaired/ok outcomes, do you think it would enough to make things clear?

Actions #3

Updated by Alexis Mousset almost 8 years ago

  • Related to Bug #8610: Some usage docs are not correctly generated in html output added
Actions #4

Updated by Florian Heigl over 7 years ago

Hi,

the heading was the misleading part for me - i assumed false would relate to "error".
Fixing the html will definitely help, not sure if it figure out what the method does while looking at ncf.

if uncertain close this bug, the improved doc should help some.

Actions #5

Updated by Alexis Mousset over 7 years ago

  • Status changed from New to Rejected

Fixed the doc build issue and the unclear doc, closing. I also opened #8612 to integrate the doc directly into the interface.

Actions #6

Updated by Alexis Mousset almost 2 years ago

  • Project changed from 41 to Rudder
  • Category set to Generic methods
  • Priority set to 0
Actions

Also available in: Atom PDF