Project

General

Profile

Actions

Bug #25704

open

event log detail show shouldBeReplacedByDiff

Added by Nicolas CHARLES 2 months ago. Updated 23 days ago.

Status:
New
Priority:
2
Category:
Web - Maintenance
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
To do
Fix check:
To do
Regression:
No

Description

A node modified diff displays shouldBeReplacedByDiff


Files


Subtasks 2 (0 open2 closed)

Bug #25977: Node update event does not correctly record old nodeReleasedClark ANDRIANASOLOActions
Bug #25978: CustomProperties (inventory properties) are not always correctly retrieved from inventoryReleasedClark ANDRIANASOLOActions
Actions #1

Updated by Vincent MEMBRÉ about 2 months ago

  • Target version changed from 8.2.0 to 8.2.1
Actions #2

Updated by Vincent MEMBRÉ about 2 months ago

  • Target version changed from 8.2.1 to 8.2.2
Actions #3

Updated by Nicolas CHARLES about 1 month ago

  • Assignee set to François ARMAND
  • Priority changed from N/A to 2

Exists also probably in 8.1
to reproduce: have a node with property, send an inventory with node inventory properties, you get this result

Actions #4

Updated by Vincent MEMBRÉ 28 days ago

  • Target version changed from 8.2.2 to 8.2.3
Actions #5

Updated by François ARMAND 28 days ago

So, the problem is that storage.save(updated) is lying:

- it always return inventory properties in the new part, but not in the old node part of the event,
- it does not return the change in rudder settings: it seens in both case the new value.

Actions #6

Updated by François ARMAND 28 days ago

  • Target version changed from 8.2.3 to 8.1.10

It happens in 8.1 too.

Updated by François ARMAND 28 days ago

"updated" and "s.newNode" have the correct values:

but s.oldNode doesn't:

Actions #8

Updated by François ARMAND 28 days ago

  • Subtask #25977 added
Actions #9

Updated by François ARMAND 28 days ago

  • Subtask #25978 added
Actions #10

Updated by Vincent MEMBRÉ 24 days ago

  • Target version changed from 8.1.10 to 8.1.11
Actions

Also available in: Atom PDF