Project

General

Profile

Bug #11715

When "policy-server.dat" is missing, generated inventory is not legit

Added by Nicolas CHARLES about 2 years ago. Updated about 1 year ago.

Status:
Released
Priority:
N/A
Assignee:
Benoît PECCATTE
Category:
Agent
Target version:
4.1.17
Pull Request:
https://github.com/Normation/rudder-p...
Severity:
Minor - inconvenience | misleading | easy workaround
User visibility:
Operational - other Techniques | Technique editor | Rudder settings
Effort required:
Very Small
Priority:
0

Description

When we install the rudder-agent, it can assume that the defaut Rudder server is named "rudder" - when policy-server.dat is non existant, techniques fallback to "rudder"
However, the generated inventory is incomplete: if policy-server.dat file is not there, then the section RUDDER/AGENT is not created on the inventory, so generated inventory is invalid

We should decide which behaviour is the right one, and be consistent

It may or may not be related to https://www.rudder-project.org/redmine/issues/11593

Related issues

Related to Rudder - Architecture #13819: Remove default server value from system techniquesReleasedActions

Associated revisions

Revision 11fc9c3c (diff)
Added by Benoît PECCATTE about 1 year ago

Fixes #11715: When \"policy-server.dat\" is missing, generated inventory is not legit

History

#1

Updated by Benoît PECCATTE about 2 years ago

  • Subject changed from Inconsistent behaviour between Techniques et Inventory to Inconsistent behaviour between Techniques and Inventory
  • User visibility set to Operational - other Techniques | Technique editor | Rudder settings
  • Priority changed from 0 to 32
#2

Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 4.1.9 to 4.1.10
#3

Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 4.1.10 to 4.1.11
  • Priority changed from 32 to 31
#4

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 4.1.11 to 4.1.12
  • Priority changed from 31 to 30
#5

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 4.1.12 to 4.1.13
#6

Updated by Benoît PECCATTE over 1 year ago

  • Target version changed from 4.1.13 to 411
#7

Updated by Benoît PECCATTE over 1 year ago

  • Target version changed from 411 to 4.1.13
#8

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 4.1.13 to 4.1.14
  • Priority changed from 30 to 29
#9

Updated by Benoît PECCATTE over 1 year ago

  • Target version changed from 4.1.14 to 4.1.15
#10

Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 4.1.15 to 4.1.16
  • Priority changed from 29 to 27
#11

Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 4.1.16 to 4.1.17
#12

Updated by François ARMAND about 1 year ago

  • Subject changed from Inconsistent behaviour between Techniques and Inventory to When "policy-server.dat" is missing, generated inventory is not legit
  • Effort required set to Very Small
  • Priority changed from 27 to 51

We should just create the file when it's missing (with the default we want) and inventory will use it.

#13

Updated by Benoît PECCATTE about 1 year ago

  • Status changed from New to In progress
  • Assignee set to Benoît PECCATTE
  • Priority changed from 51 to 0
#14

Updated by Benoît PECCATTE about 1 year ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Benoît PECCATTE to Alexis MOUSSET
  • Pull Request set to https://github.com/Normation/rudder-packages/pull/1716
#15

Updated by Rudder Quality Assistant about 1 year ago

  • Assignee changed from Alexis MOUSSET to Benoît PECCATTE
#16

Updated by Benoît PECCATTE about 1 year ago

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

Updated by Benoît PECCATTE about 1 year ago

#18

Updated by Vincent MEMBRÉ about 1 year ago

  • Status changed from Pending release to Released
This bug has been fixed in Rudder 4.1.17, 4.3.7 and 5.0.3 which were released today. 
Changelog
 Changelog
 Changelog

Also available in: Atom PDF