Project

General

Profile

Actions

Bug #18303

closed

Calling the relay-api to trigger a remote run on all nodes do an infinite loop on root server, killing it

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

Status:
Released
Priority:
N/A
Category:
Relay server or API
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
Reviewed
Fix check:
Checked
Regression:

Description

calling
curl --request POST https://rudder.example.com/rudder/relay-api/remote-run/all --data keep_output=true

cause hundreds of cf-runagent, killing the server under the load


Files

nodeslist.json (435 Bytes) nodeslist.json Nicolas CHARLES, 2020-10-06 15:17
Actions #2

Updated by Alexis Mousset over 3 years ago

Problem is that root is both a next hop and a neighbor

 DEBUG relayd::api::remote_run: Starting remote run (asynchronous: false, keep_output: true)
 DEBUG relayd::api::remote_run: Neighbors: [
    "relay",
    "server.rudder.local",
    "agent2.rudder.local",
]
 DEBUG relayd::api::remote_run: Remote run command: '"sudo" "/opt/rudder/bin/rudder" "remote" "run" "relay,server.rudder.local,agent2.rudder.local"'
 DEBUG relayd::api::remote_run: Next-hops: [
    (
        "server.rudder.local",
        All,
    ),
    (
        "relay",
        All,
    ),
]
Actions #3

Updated by Alexis Mousset over 3 years ago

  • Status changed from New to In progress
  • Assignee set to Alexis Mousset
Actions #4

Updated by Alexis Mousset over 3 years ago

  • Subject changed from calling the relay-api to trigger a remote run on all nodes do an infinite loop on root server, killing it to Calling the relay-api to trigger a remote run on all nodes do an infinite loop on root server, killing it
Actions #5

Updated by Alexis Mousset over 3 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Alexis Mousset to Nicolas CHARLES
  • Pull Request set to https://github.com/Normation/rudder/pull/3246
Actions #6

Updated by Alexis Mousset over 3 years ago

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

Updated by Alexis Mousset over 3 years ago

  • Name check changed from To do to Reviewed
Actions #8

Updated by Alexis Mousset over 3 years ago

  • Fix check changed from To do to Checked
Actions #9

Updated by Vincent MEMBRÉ over 3 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 6.1.6 and 6.2.0~beta1 which were released today.

Actions

Also available in: Atom PDF