Project

General

Profile

Bug #6729

After a partial promise generation, some nodes being a relay may have their promises stored in a non reachable location

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

Status:
Released
Priority:
N/A
Category:
Web - Config management
Target version:
Severity:
User visibility:
Effort required:
Priority:

Description

If we do a partial promise regeneration, some node may have their new promises in the folder
/path/to/relay.new/path/to/node

rather than the good old /path/to/relay/path/to/node

More presillely, we have the following case to manage (node3 was removed in the generation):

1/ Relay and all nodes updated (historical defautl behaviour)

Generation gives:

relay/
  relay_promises
  share/{ node1, node2, node3 }
relay.new/
  relay_promises
  share/{ node1.new, node2.new }

We want ([U] means updated):

relay/
  relay_promises [U]
  share/{ node1 [U], node2 [U] }

2/ Relay promises not updated, some children updated:

Generation gives:

relay/
  relay_promises
  share/{ node1, node2, node3 }
relay.new/
  share/{ node1.new }

We want:

relay/
  relay_promises 
  share/{ node1 [U], node2 }

3/ Relay promises updated, some children updated:

Generation gives:

relay/
  relay_promises
  share/{ node1, node2, node3 }
relay.new/
  relay_promises
  share/{ node1.new }

We want:

relay/
  relay_promises [U]
  share/{ node1 [U], node2 }

We see that all the naive scheme of moving or copying can't be used, and that some set union and intersection of nodes modified or not modified must be done.


Subtasks

Bug #6914: Delete the folder with .new after promise generationReleasedFrançois ARMAND2015-07-01Actions

Related issues

Related to Rudder - Bug #7056: A test is not compiling on branch 2.11 since #6729 was mergedReleasedFrançois ARMAND2015-07-30Actions
#1

Updated by François ARMAND over 5 years ago

  • Description updated (diff)
#2

Updated by Nicolas CHARLES over 5 years ago

  • Status changed from New to Pending technical review
  • Assignee set to Vincent MEMBRÉ
  • Priority changed from N/A to 1
  • Pull Request set to https://github.com/Normation/rudder/pull/883
#3

Updated by Nicolas CHARLES over 5 years ago

  • Assignee changed from Vincent MEMBRÉ to François ARMAND
#4

Updated by Nicolas CHARLES over 5 years ago

  • Status changed from Pending technical review to Pending release
#6

Updated by Vincent MEMBRÉ over 5 years ago

This bug has been fixed in Rudder 2.10.15, 2.11.12, 3.0.7 and 3.1.0 which were released today.

#7

Updated by Vincent MEMBRÉ over 5 years ago

  • Status changed from Pending release to Released
#8

Updated by Vincent MEMBRÉ over 5 years ago

  • Related to Bug #7056: A test is not compiling on branch 2.11 since #6729 was merged added

Also available in: Atom PDF