Skip to content

ENT-14195 3. Event handling for file deletion - #6355

Merged
larsewi merged 2 commits into
cfengine:masterfrom
victormlg:event-handling-3-file-deletion
Sep 25, 2026
Merged

larsewi merged 2 commits into
cfengine:masterfrom
victormlg:event-handling-3-file-deletion

Conversation

@victormlg

Copy link
Copy Markdown
Contributor

File deletion

In this PR we create a new event type for file deletion with all the necessary code to be working in the logic of watcher.c

@victormlg victormlg changed the title Event handling 3 file deletion ENT-14195 3. Event handling for file deletion Sep 7, 2026
@cf-bottom

Copy link
Copy Markdown

Thank you for submitting a pull request! Maybe @larsewi can review this?

@victormlg
victormlg force-pushed the event-handling-3-file-deletion branch 4 times, most recently from 0321282 to c033d62 Compare September 23, 2026 14:40

@craigcomstock craigcomstock left a comment

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.

Looks good to me. Maybe some semantic word changes to be clear that all of this is about files being deleted instead of files existing.

Comment thread cf-reactor/file_watcher.c Outdated
Comment thread cf-reactor/watcher.c Outdated
Comment thread cf-reactor/file_watcher.c Outdated
Comment thread cf-reactor/file_watcher.c Outdated
Comment thread cf-reactor/file_watcher.c Outdated
@victormlg
victormlg force-pushed the event-handling-3-file-deletion branch from fc172c0 to 57c2aea Compare September 24, 2026 07:17
@victormlg
victormlg requested a review from larsewi September 24, 2026 09:05
Comment thread cf-reactor/file_watcher.c Outdated
Comment thread cf-reactor/file_watcher.c Outdated
Comment thread cf-reactor/file_watcher.c Outdated
Comment thread cf-reactor/file_watcher.c Outdated
Signed-off-by: Victor Moene <victor.moene@northern.tech>
Signed-off-by: Victor Moene <victor.moene@northern.tech>
@victormlg
victormlg force-pushed the event-handling-3-file-deletion branch from 57c2aea to bfaad9e Compare September 24, 2026 10:33
@larsewi

larsewi commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

@cf-bottom Jenkins please :)

@cf-bottom

Copy link
Copy Markdown

Alright, I triggered a build:

Build Status

Jenkins: https://ci.cfengine.com/job/pr-pipeline/14701/

Packages: http://buildcache.cfengine.com/packages/testing-pr/jenkins-pr-pipeline-14701/

@larsewi
larsewi merged commit 8fbdd4a into cfengine:master Sep 25, 2026
45 checks passed
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.

4 participants