Rudder

Nicolas CHARLES wrote:

I have deleted nodes for the sake of testing. Then I wanted to add them back in Rudder, however, since their inventory were in the RemovedInventory branch, new inventories goes there.
What would be the expected behaviour ?

The current behaviour would completely prevent someone from doing this: "Delete that node from Rudder. [A few weeks later] OK, add it back in".

I would suspect that we'd want it moved to the Pending Inventory,

I agree with this proposed behaviour. Ideally, we would like to see an extra field, saying "Deleted from Rudder on: 2012-06-11 18:40".

but if it is so, deleted nodes would keep poping back in the pending nodes, unless somebody goes and removes rudder-agent on them

This limitation is already the same as for refused nodes - it's really up to the admin to be able to remove rudder-agent from the node in question... Or, of course, we could have a "blacklist" button to make sure that this node doesn't keep appearing...

Nicolas, do you need any more information on this to be able to fix?

Applied in changeset 1be8a0d219258df6022d7879bed8de14937b3a7e.

That's OK and I'm really happy with the cleaness of the code! Only that to add that feature is pretty cool :)