Project

General

Profile

Actions

Architecture #25869

open

metrics survey are never sent because technique wants yes and get complete

Added by François ARMAND about 1 month ago. Updated 20 days ago.

Status:
New
Priority:
N/A
Assignee:
-
Category:
System integration
Target version:
Effort required:
Name check:
To do
Fix check:
To do
Regression:
No

Description

The survey script is managed :

- in the webapp by the settings that is a tri-state: no, minimal, complete
- in the cfengine file: /var/rudder/config-repo/techniques/system/rudder-service-webapp/1.0/metrics-reporting.cf

This file seems to expect "yes", which never happens:

      "accept_metrics" expression => strcmp("${server_roles_common.send_metrics}", "yes");
      "run_metrics"    expression => "accept_metrics";

It should be "not no".

I tested to change it to:

     "accept_metrics"        not => strcmp("${server_roles_common.send_metrics}", "no");
      "run_metrics"    expression => "accept_metrics";

But the file is still not send:

rudder agent run:

...
E| n/a           Rudder Webapp             Send metrics to rudder-p|                    Sending metrics to rudder-project.org is not enabled. Skipping.

[root@server configuration-repository]# grep -i "send_metrics" /var/rudder/cfengine-community/inputs/*
/var/rudder/cfengine-community/inputs/rudder.json:  "SEND_METRICS":"complete",

Subtasks 2 (1 open1 closed)

Architecture #25918: Disable send metrics pop-up and menu in RudderRejectedActions
Enhancement #25982: Disable metrics pop-up and formPending releaseVincent MEMBRÉActions

Related issues 1 (1 open0 closed)

Related to Rudder - Architecture #25841: Create an instance id on each root serverPending releaseFrançois ARMANDActions
Actions #1

Updated by François ARMAND about 1 month ago

Actions #2

Updated by Nicolas CHARLES about 1 month ago

The script also uses information which are not available anymore (like tables nodes, directives, rules)

Actions #3

Updated by Nicolas CHARLES about 1 month ago

  • Tracker changed from Bug to Architecture
  • Priority deleted (0)

Reclassifying to architecture, because the impacts are multiple (script is not valid, endpoint needs to be tested, etc)

Actions #4

Updated by Nicolas CHARLES about 1 month ago

  • Priority changed from To review to N/A
Actions #5

Updated by François ARMAND about 1 month ago

  • Subtask #25918 added
Actions #6

Updated by Vincent MEMBRÉ 25 days ago

  • Target version changed from 8.1.9 to 8.1.10
Actions #7

Updated by Vincent MEMBRÉ 20 days ago

  • Target version changed from 8.1.10 to 8.1.11
Actions #8

Updated by François ARMAND 20 days ago

  • Subtask #25982 added
Actions

Also available in: Atom PDF