Project

General

Profile

Actions
Copy link

Bug #11048

closed

Bug #11047: rudder agent update -q bad exit code

Rudder agent update returns 0 even if update fails

Added by Benoît PECCATTE over 7 years ago. Updated almost 7 years ago.

Status:
Released
Priority:
N/A
Assignee:
Alexis Mousset
Category:
Agent
Target version:
4.1.10
Pull Request:
https://github.com/Normation/rudder-a...
Severity:
Major - prevents use of part of Rudder | no simple workaround
UX impact:
User visibility:
Operational - other Techniques | Technique editor | Rudder settings
Effort required:
Small
Priority:
60
Name check:
Fix check:
Regression:

Description

Even if cfengine fails to return properly, we should be able to communicate the error with the caller.
For example with specific line in the output.

Related issues 3 (0 open — 3 closed)

Has duplicate Rudder - Bug #9673: "rudder agent update" always ays "ok" even when it could not update the promisesRejectedActions
Has duplicate Rudder - Bug #11622: "rudder agent update" always displays success message, even if update failedRejectedActions
Has duplicate Rudder - User story #9580: Add detection of CFEngine return code and display an error message when it is not zeroRejectedActions
Actions
Copy link
 #1

Updated by François ARMAND over 7 years ago

  • Severity set to Major - prevents use of part of Rudder | no simple workaround
  • User visibility set to Operational - other Techniques | Technique editor | Rudder settings
  • Priority changed from 0 to 52
Actions
Copy link
 #2

Updated by Benoît PECCATTE over 7 years ago

  • Effort required set to Small
  • Priority changed from 52 to 55

Let's grep output to manage 90% of cases.

Actions
Copy link
 #3

Updated by Vincent MEMBRÉ over 7 years ago

  • Target version changed from 3.1.22 to 3.1.23
Actions
Copy link
 #4

Updated by Vincent MEMBRÉ over 7 years ago

  • Target version changed from 3.1.23 to 3.1.24
  • Priority changed from 55 to 54
Actions
Copy link
 #5

Updated by Alexis Mousset about 7 years ago

  • Subject changed from Rudder agent update ruturns 0 even if update fails to Rudder agent update returns 0 even if update fails
  • Priority changed from 54 to 53
Actions
Copy link
 #6

Updated by Alexis Mousset about 7 years ago

  • Has duplicate Bug #9673: "rudder agent update" always ays "ok" even when it could not update the promises added
Actions
Copy link
 #7

Updated by Vincent MEMBRÉ about 7 years ago

  • Target version changed from 3.1.24 to 3.1.25
Actions
Copy link
 #8

Updated by Benoît PECCATTE about 7 years ago

  • Has duplicate Bug #11622: "rudder agent update" always displays success message, even if update failed added
Actions
Copy link
 #9

Updated by Benoît PECCATTE about 7 years ago

  • Priority changed from 53 to 64
Actions
Copy link
 #10

Updated by Benoît PECCATTE about 7 years ago

  • Target version changed from 3.1.25 to 4.1.9
Actions
Copy link
 #11

Updated by Vincent MEMBRÉ about 7 years ago

  • Target version changed from 4.1.9 to 4.1.10
  • Priority changed from 64 to 63
Actions
Copy link
 #12

Updated by Benoît PECCATTE almost 7 years ago

  • Status changed from New to In progress
  • Assignee set to Benoît PECCATTE
  • Priority changed from 63 to 61
Actions
Copy link
 #13

Updated by Benoît PECCATTE almost 7 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Benoît PECCATTE to Alexis Mousset
  • Pull Request set to https://github.com/Normation/rudder-agent/pull/151
Actions
Copy link
 #14

Updated by Benoît PECCATTE almost 7 years ago

  • Has duplicate User story #9580: Add detection of CFEngine return code and display an error message when it is not zero added
Actions
Copy link
 #15

Updated by Benoît PECCATTE almost 7 years ago

  • Status changed from Pending technical review to Pending release
  • Priority changed from 61 to 60
Actions
Copy link
 #16

Updated by Vincent MEMBRÉ almost 7 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 4.1.10 and 4.2.4 which were released today.

Actions
Copy link

Also available in: Atom PDF