Issue description:
When the audit function in runners/inventory.py runs without any changes in minion data, it updates last_audit but not last_seen. Since a successful audit implies that a minion was reachable, it would make more sense for last_seen to be updated as well.
Currently, this leads to inconsistent states where an audit appears to have occurred before the minion was last seen. Because last_seen is used as an availability metric, updating it alongside last_audit would provide a more accurate reflection of minion activity.
The purpose of last_audit is also not entirely clear. It is not obvious to the end user whether it indicates that a minion has new data, or simply that the audit ran successfully. Updating last_seen may help clarify this distinction, otherwise, a mention of this in the documentation or a UI tooltip would make it more obvious.
Issue description:
When the audit function in runners/inventory.py runs without any changes in minion data, it updates last_audit but not last_seen. Since a successful audit implies that a minion was reachable, it would make more sense for last_seen to be updated as well.
Currently, this leads to inconsistent states where an audit appears to have occurred before the minion was last seen. Because last_seen is used as an availability metric, updating it alongside last_audit would provide a more accurate reflection of minion activity.
The purpose of last_audit is also not entirely clear. It is not obvious to the end user whether it indicates that a minion has new data, or simply that the audit ran successfully. Updating last_seen may help clarify this distinction, otherwise, a mention of this in the documentation or a UI tooltip would make it more obvious.