fix: remove set -e from agent, guard all commands with || true #46

Merged
rafaga21 merged 1 commits from feat/push-monitoring-agent-v2 into master 2026-06-11 20:22:21 -04:00
Owner
  • Removed 'set -e' so a failing metric collection command doesn't kill
    the agent process
  • Added '|| true' to every command in collect_metrics() to prevent
    pipeline failures
  • Fixed push_metrics() to capture curl exit code with 'local code; code=$(...) || code=0'
  • The agent now keeps running even if individual metrics or push fail
- Removed 'set -e' so a failing metric collection command doesn't kill the agent process - Added '|| true' to every command in collect_metrics() to prevent pipeline failures - Fixed push_metrics() to capture curl exit code with 'local code; code=$(...) || code=0' - The agent now keeps running even if individual metrics or push fail
rafaga21 added 1 commit 2026-06-11 20:21:44 -04:00
- Removed 'set -e' so a failing metric collection command doesn't kill
  the agent process
- Added '|| true' to every command in collect_metrics() to prevent
  pipeline failures
- Fixed push_metrics() to capture curl exit code with 'local code; code=$(...) || code=0'
- The agent now keeps running even if individual metrics or push fail
rafaga21 merged commit db29f5b50b into master 2026-06-11 20:22:21 -04:00
rafaga21 deleted branch feat/push-monitoring-agent-v2 2026-06-11 20:22:22 -04:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: devlab/server-manager#46