Skip to content
This repository has been archived by the owner on Feb 7, 2019. It is now read-only.

Fixes #4497: limit the max number of inqueue inventory waiting to be processed #28

Merged

Conversation

fanf
Copy link
Member

@fanf fanf commented Feb 24, 2014

The idea is to add an intermediary actor that counts the number of wainting inventory inqueue.

BE CAREFUL: we also need to provides a migration script to add the new configuration property. It is missing for now (so no merge for now !)

See #27 for history.

@jooooooon
Copy link
Member

Yes, and before this is merged, we also need to adapt the send-clean.sh script to retry when it gets HTTP error code 503 (see https://github.com/Normation/rudder-techniques/blob/master/tools/send-clean.sh)

@@ -33,3 +33,10 @@ history.inventories.rootdir=/var/rudder/inventories/historical
# where to store debug information about LDAP modification requests
ldif.tracelog.rootdir=/var/rudder/inventories/debug

#
# Max number of reports waiting to be processed internally.
# For a rought estimation, you can consider that a report in queue
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/rought/rough/

@jooooooon
Copy link
Member

@jooooooon wrote:

Yes, and before this is merged, we also need to adapt the send-clean.sh script to retry when it gets HTTP error code 503 (see https://github.com/Normation/rudder-techniques/blob/master/tools/send-clean.sh)

I just added a PR for this: Normation/rudder-techniques#300

@jooooooon
Copy link
Member

@fanf wrote:

BE CAREFUL: we also need to provides a migration script to add the new configuration property. It is missing for now (so no merge for now !)

And I also made a PR for this: Normation/rudder-packages#236

So once all 3 have been technically reviewed, this can be merged!

@ncharles
Copy link
Member

looks good to me ! we need to have the others two PR fixed before we can merge this one

jooooooon added a commit that referenced this pull request Feb 26, 2014
…ry_inqueue

Fixes #4497: limit the max number of inqueue inventory waiting to be processed
@jooooooon jooooooon merged commit 1a20056 into Normation:master Feb 26, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants