Project

General

Profile

Actions
Copy link

Bug #11648

closed

Unexpected report for Send inventories to CMDB after #11257

Bug #11648: Unexpected report for Send inventories to CMDB after #11257

Added by Janos Mattyasovszky almost 9 years ago. Updated over 8 years ago.

Status:
Released
Priority:
N/A
Assignee:
Alexis Mousset
Category:
System techniques
Target version:
3.1.25
Pull Request:
https://github.com/Normation/rudder-t...
Severity:
Minor - inconvenience | misleading | easy workaround
UX impact:
User visibility:
Operational - other Techniques | Technique editor | Rudder settings
Effort required:
Very Small
Priority:
61
Name check:
Fix check:
Regression:

Description

In #11257 a flag was introduced to disable send-clean.sh and a n/a output log was additionally made.

This apparently conflicts with the case if no reports are to be sent, because then you get double reporting:

E| n/a           DistributePolicy          Send inventories to CMDB                     Sending inventories has been disabled
E| compliant     DistributePolicy          Send inventories to CMDB                     No inventory to send

Files

unexpected_Send_to_CMDB.png (22 KB) unexpected_Send_to_CMDB.png Janos Mattyasovszky, 2017-10-23 10:58
unexpected_Send_to_CMDB.png

Subtasks 1 (0 open — 1 closed)

Bug #11847: Mismatched report and metadatat in distribute policyReleasedNicolas CHARLESActions

Related issues 1 (0 open — 1 closed)

Related to Rudder - User story #11257: Do not send inventory if a forwarder/uploader daemon existsReleasedAlexis MoussetActions

Updated by Janos Mattyasovszky almost 9 years ago Actions
Copy link
 #1

  • File deleted (unexpected_Send_to_CMDB.png)

Updated by Janos Mattyasovszky almost 9 years ago Actions
Copy link
 #3

  • Description updated (diff)

Updated by Janos Mattyasovszky almost 9 years ago Actions
Copy link
 #4

This fixed it for me on top of latest nightly (rudder-techniques-4.1.8.rc1.git201710220155-1.SLES.11):

From 99f497ece40b2ce1e101e1ecad2fde443ec2a4bc Mon Sep 17 00:00:00 2001
From: "root user (CLI)" <root@localhost>
Date: Mon, 23 Oct 2017 10:59:49 +0200
Subject: [PATCH] Fixes #11648: Double reporting for disabled inventory sending

---
 techniques/system/distributePolicy/1.0/propagatePromises.st | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/techniques/system/distributePolicy/1.0/propagatePromises.st b/techniques/sy
stem/distributePolicy/1.0/propagatePromises.st
index d2dfabc..867d3f0 100644
--- a/techniques/system/distributePolicy/1.0/propagatePromises.st
+++ b/techniques/system/distributePolicy/1.0/propagatePromises.st
@@ -286,7 +286,7 @@ bundle agent sendInventoryToCmdb
     pass3.rudder_inventory_processing_error::
       "any" usebundle => rudder_common_report("DistributePolicy", "result_error", "&TRA
CKINGKEY&", "Send inventories to CMDB", "None", "Some incoming inventories could not be
sent to the inventory API");

-    pass3.((root_server|role_rudder_relay_top|role_rudder_relay_promises_only).!rudder_
inventory_processing_repaired.!rudder_inventory_processing_error)::
+    pass3.((root_server|role_rudder_relay_top|role_rudder_relay_promises_only).!rudder_
inventory_processing_repaired.!rudder_inventory_processing_error.!disable_upload)::
       "any" usebundle => rudder_common_report("DistributePolicy", "result_success", "&T
RACKINGKEY&", "Send inventories to CMDB", "None", "No inventory to send");

     pass3.(rudder_inventory_relay_repaired.!rudder_inventory_relay_error)::
--
2.14.2

Updated by Janos Mattyasovszky almost 9 years ago Actions
Copy link
 #5

  • Related to User story #11257: Do not send inventory if a forwarder/uploader daemon exists added

Updated by Benoît PECCATTE almost 9 years ago Actions
Copy link
 #6

  • Effort required set to Very Small
  • Priority changed from 32 to 49

Updated by Benoît PECCATTE almost 9 years ago Actions
Copy link
 #7

  • Priority changed from 49 to 61

Updated by Nicolas CHARLES almost 9 years ago Actions
Copy link
 #8

  • Target version set to 3.1.25

Updated by Nicolas CHARLES almost 9 years ago Actions
Copy link
 #9

  • Status changed from New to In progress
  • Assignee set to Nicolas CHARLES

Updated by Nicolas CHARLES almost 9 years ago Actions
Copy link
 #10

  • Status changed from In progress to Pending technical review
  • Assignee changed from Nicolas CHARLES to Alexis Mousset
  • Pull Request set to https://github.com/Normation/rudder-techniques/pull/1218

Updated by Nicolas CHARLES almost 9 years ago Actions
Copy link
 #11

  • Status changed from Pending technical review to Pending release

Updated by Vincent MEMBRÉ over 8 years ago Actions
Copy link
 #12

  • Category changed from Techniques to System techniques

Updated by Vincent MEMBRÉ over 8 years ago Actions
Copy link
 #13

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 3.1.25, 4.1.9 and 4.2.3 which were released today.

Actions
Copy link

Also available in: PDF Atom