Project

General

Profile

Actions

Bug #10973

closed

When hooks are exiting with an error we can have stackoverflow error if there are many of them

Added by Nicolas CHARLES almost 4 years ago. Updated over 2 years ago.

Status:
Released
Priority:
N/A
Category:
Web - Config management
Target version:
Severity:
Major - prevents use of part of Rudder | no simple workaround
User visibility:
Operational - other Techniques | Rudder settings | Plugins
Effort required:
Medium
Priority:
29

Description

I have an install with 1600 nodes
I had a stackoverflow error at generation - increasing Xss to 128m allowed me to circumvent the issue
The I got thousands of

stderr: '2017-06-21T18:21:55+0200 error: Can't stat file '/var/rudder/ncf//bin/sh: 0: Can't open /var/rudder/ncf/common/10_ncf_internals/list-compatible-inputs' for parsing. (stat: No such file or directory) ' ; Exit code=1 for hook: '/opt/rudder/etc/hooks.d/policy-generation-node-ready/10-cf-promise-check'. stdout: stderr: '2017-06-21T18:21:55+0200 error: Can't stat file '/var/rudder/ncf//bin/sh: 0: Can't open /var/rudder/ncf/common/10_ncf_internals/list-compatible-inputs' for parsing. (stat: No such file or directory) ' ; Exit code=1 for hook: '/opt/rudder/etc/hooks.d/policy-generation-node-ready/10-cf-promise-check'. stdout: stderr: '2017-06-21T18:21:55+0200 error: Can't stat file '/var/rudder/ncf//bin/sh: 0: Can't open /var/rudder/ncf/common/10_ncf_internals/list-compatible-inputs' for parsing. (stat: No such file or directory) ' ; Exit code=1 for hook: '/opt/rudder/etc/hooks.d/policy-generation-node-ready/10-cf-promise-check'.

as you can see, the nodeid is missing there

Actions #1

Updated by Alexis MOUSSET almost 4 years ago

  • Target version changed from 4.1.5 to 4.1.6
Actions #2

Updated by Benoît PECCATTE almost 4 years ago

  • Effort required set to Very Small
  • Priority changed from 0 to 58
Actions #3

Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 4.1.6 to 4.1.7
  • Priority changed from 58 to 57
Actions #4

Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 4.1.7 to 4.1.8
  • Priority changed from 57 to 56
Actions #5

Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 4.1.8 to 4.1.9
  • Priority changed from 56 to 55
Actions #6

Updated by Benoît PECCATTE over 3 years ago

  • Priority changed from 55 to 66
Actions #7

Updated by François ARMAND over 3 years ago

  • Assignee set to François ARMAND
  • Priority changed from 66 to 65
Actions #8

Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 4.1.9 to 4.1.10
Actions #9

Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 4.1.10 to 4.1.11
  • Priority changed from 65 to 62
Actions #10

Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 4.1.11 to 4.1.12
  • Priority changed from 62 to 60
Actions #11

Updated by Vincent MEMBRÉ almost 3 years ago

  • Target version changed from 4.1.12 to 4.1.13
  • Priority changed from 60 to 59
Actions #12

Updated by Benoît PECCATTE almost 3 years ago

  • Target version changed from 4.1.13 to 411
Actions #13

Updated by Benoît PECCATTE almost 3 years ago

  • Target version changed from 411 to 4.1.13
Actions #14

Updated by Vincent MEMBRÉ almost 3 years ago

  • Target version changed from 4.1.13 to 4.1.14
  • Priority changed from 59 to 58
Actions #15

Updated by Benoît PECCATTE almost 3 years ago

  • Target version changed from 4.1.14 to 4.1.15
Actions #16

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 4.1.15 to 4.1.16
Actions #17

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 4.1.16 to 4.1.17
Actions #18

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 4.1.17 to 4.1.18
Actions #19

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 4.1.18 to 4.1.19
Actions #20

Updated by François ARMAND over 2 years ago

  • Subject changed from When hooks are exiting an error, we don't get the node that is failing, and we can have stackoverfloww error if there are many of them to When hooks are exiting an error we can have stackoverfloww error if there are many of them
  • User visibility changed from Infrequent - complex configurations | third party integrations to Operational - other Techniques | Rudder settings | Plugins
  • Effort required changed from Very Small to Medium
  • Priority changed from 58 to 29

I'm take that "very small" is to look at the real weight.

Actions #21

Updated by François ARMAND over 2 years ago

  • Status changed from New to In progress
Actions #22

Updated by François ARMAND over 2 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from François ARMAND to Vincent MEMBRÉ
  • Pull Request set to https://github.com/Normation/rudder/pull/2112
Actions #23

Updated by Rudder Quality Assistant over 2 years ago

  • Assignee changed from Vincent MEMBRÉ to François ARMAND
Actions #24

Updated by François ARMAND over 2 years ago

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

Updated by Alexis MOUSSET over 2 years ago

  • Subject changed from When hooks are exiting an error we can have stackoverfloww error if there are many of them to When hooks are exiting with an error we can have stackoverflow error if there are many of them
Actions #26

Updated by Alexis MOUSSET over 2 years ago

  • Status changed from Pending release to Released
This bug has been fixed in Rudder 4.1.19, 4.3.9 and 5.0.5 which were released today.
Changelog
Changelog
Changelog
Actions

Also available in: Atom PDF