User story #7490
Make the agent resend the inventory if sending failed
On first runs, the webdav is not correctly configured on the rudder server root, so the promises will configure the webdav password, and restart apache, and the root server will send its inventory to the webdav, that may be not yet configured, or restarting.
So there is a slight possibility that it fails, and keep an incomplete inventory of the server (with file staying in /var/rudder/inventory), and there is also a possibility that the initial promises never had a chance to quick in to send the inventory to the default apache endpoint
To be on the safe side, the root server should always send its inventory to the endpoint
This won't be done in the initial promises (as the web interface may not be configured when we are in intial promises), but really in normal operation (so when promises have been generated)
#19 Updated by Jonathan CLARKE over 1 year ago
- Severity set to Minor - inconvenience | misleading | easy workaround
- User visibility set to Getting started - demo | first install | level 1 Techniques
Sending directly to the endpoint from promises is not a good idea, because if the endpoint is overloaded it will simply return an error (as designed) but promises don't have any retry mechanism to handle this (which is why this bug is a problem in the first place).
We could and probably should add a retry mechanism to send failed inventories in promises.
Also I don't think we should bypass the webdav to send inventories. Everything is much easier when there are no exceptions, and all inventories are processed in the same way. We have plans to use apache hooks to add actions when inventories are received for example.
#33 Updated by Nicolas CHARLES 10 months ago
Benoît PECCATTE wrote:
I don't see the inconvenience, if the inventory is not sent, the server still works, and the inventory will be sent again the day after.
It shows in the interface "No machine inventory", which is not great for a first contact with Rudder
#40 Updated by Benoît PECCATTE 10 months ago
- Status changed from In progress to Pending technical review
- Assignee changed from Benoît PECCATTE to Nicolas CHARLES
- Pull Request set to https://github.com/Normation/rudder-techniques/pull/1250
#47 Updated by Benoît PECCATTE 6 months ago
- Status changed from Discussion to Pending release
Applied in changeset rudder-techniques|3b07321885fbc3664ec7aa22d48a8d5830decd3f.
#48 Updated by Vincent MEMBRÉ 5 months ago
- Status changed from Pending release to Released