Skip to content

ENT-14695: Record events promise outcome - #6383

Open
victormlg wants to merge 3 commits into
cfengine:masterfrom
victormlg:cf-reactor.RecordPromiseOutcome
Open

victormlg wants to merge 3 commits into
cfengine:masterfrom
victormlg:cf-reactor.RecordPromiseOutcome

Conversation

@victormlg

Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Victor Moene <victor.moene@northern.tech>
Ticket: ENT-14695
Signed-off-by: Victor Moene <victor.moene@northern.tech>
Signed-off-by: Victor Moene <victor.moene@northern.tech>
Comment thread cf-reactor/file_watcher.c
{
char *path;
time_t last_seen; /* Time the file was last observed present, 0 if absent */
bool lstat_failing; /* Whether the last lstat() failed, to log a failure only once */

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is this necessary? Won't the event only be triggered once? Or is it being re-triggered repeatedly? It should probably only be triggered once on the actual event of deleting it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants