Project

General

Profile

Actions
Copy link

Bug #8023

closed

command_exection_result defaulting to 'error' state is not documented

Added by Florian Heigl almost 9 years ago. Updated over 2 years ago.

Status:
Rejected
Priority:
N/A
Assignee:
Alexis Mousset
Category:
Generic methods
Target version:
ncf-0.x
Pull Request:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
Fix check:
Regression:

Description

This class can define a repair and a kept outcome.
It cannot define an error.

This is kind of, umm, problematic.

I have failing commands that are interpreted as happy commands.

(If possible, make it so that one is able to configure it so that any unexpected state means an error.
Doesn't need to be default, but that was the one big PITA from start with the normal scheduler.

We use script wrappers for this now.)

Related issues 1 (0 open — 1 closed)

Is duplicate of Rudder - User story #7879: Add documentation for ncf generic methodsReleasedBenoît PECCATTEActions
Actions
Copy link
 #1

Updated by Benoît PECCATTE almost 9 years ago

I will assume that you didn't mean scheduler here, but just command_exection_result

There is no error state in the command_exection_result genric method since any state not in success or repaired is considered as an error.
The acceptance test for this methods says it is the case.

Do you want tro have e default other than error or have I just missed something that happen to be your case ?

Actions
Copy link
 #2

Updated by Florian Heigl almost 9 years ago

Hi,

then it should be all as I wanted it. I could not see the text saying it's defaulting to error. might be fine print ;)

I'll re-test tomorrow and close if it was just PBKAC

Actions
Copy link
 #3

Updated by Florian Heigl almost 9 years ago

Ah, yeah its really just in the method documentation. Not obvious for mr. clickyclicky ;-)

Actions
Copy link
 #4

Updated by Benoît PECCATTE almost 9 years ago

  • Subject changed from Scheduler command_exection_result is missing 'error' state to command_exection_result defaulting to 'error' state is not documented
  • Target version set to 0.x

Sorry, changing the subject to patch documentation then

Actions
Copy link
 #5

Updated by Alexis Mousset almost 9 years ago

  • Status changed from New to In progress
  • Assignee set to Alexis Mousset
Actions
Copy link
 #6

Updated by Alexis Mousset almost 9 years ago

  • Status changed from In progress to Rejected

Handled in #7879. Closing this one.

Actions
Copy link
 #7

Updated by Alexis Mousset almost 9 years ago

  • Is duplicate of User story #7879: Add documentation for ncf generic methods added
Actions
Copy link
 #8

Updated by Alexis Mousset over 2 years ago

  • Target version changed from 0.x to ncf-0.x
  • Priority set to 0
Actions
Copy link
 #9

Updated by Alexis Mousset over 2 years ago

  • Project changed from 41 to Rudder
  • Category set to Generic methods
Actions
Copy link

Also available in: Atom PDF