-
Notifications
You must be signed in to change notification settings - Fork 73
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes #19920: root inventory is missing and need to be resent after install #4281
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I havent finished the review, but some small remarks
.../rudder/rudder-core/src/main/scala/com/normation/rudder/inventory/InventoryFileWatcher.scala
Outdated
Show resolved
Hide resolved
.../rudder/rudder-core/src/main/scala/com/normation/rudder/inventory/InventoryFileWatcher.scala
Outdated
Show resolved
Hide resolved
32ce6ff
to
9dc81bd
Compare
Rebasing to prepare for merge |
("queueMaxSize" -> inventoryProcessor.maxQueueSize) | ||
~ ("queueSaturated" -> saturated) | ||
("queueMaxSize" -> Int.MaxValue) | ||
~ ("queueSaturated" -> false) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I kept that to not break the API, but it makes little sense.
I propose to:
- do that for up to version 13
- create a version 14 with info
{"incomingInventories": XXX}
the number of inventories in incoming+accepted-nodes-updates
It can be done after merging this PR.
OK, merging this PR |
trait
to make more clear what are the public API in services and how the are used in other services (code on interfaces)inventory.parse.parallelization
, default1
because it seems that in general, there is little advantage of making it more appart stressing LDAPinventories.watcher.max.age.before.deletion
, remove unmatched pairshttps://issues.rudder.io/issues/19920