Skip to content

ENT-14398: Moved event driven cfengine into cf-reactor - #6290

Open
victormlg wants to merge 1 commit into
cfengine:masterfrom
victormlg:agent-driven-cf-reactor
Open

ENT-14398: Moved event driven cfengine into cf-reactor#6290
victormlg wants to merge 1 commit into
cfengine:masterfrom
victormlg:agent-driven-cf-reactor

Conversation

@victormlg

Copy link
Copy Markdown
Contributor

No description provided.

@victormlg
victormlg force-pushed the agent-driven-cf-reactor branch from a26acb5 to 22ff75c Compare August 14, 2026 11:17
Comment thread cf-reactor/cf-reactor.c
/* long options only */
case 0:
{
// const char *const option_name = OPTIONS[longopt_idx].name;

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.

There's some commented out code here

@victormlg
victormlg marked this pull request as ready for review August 18, 2026 08:09
@victormlg
victormlg force-pushed the agent-driven-cf-reactor branch from 22ff75c to 4407d5a Compare August 19, 2026 15:08
Ticket: ENT-14398
Signed-off-by: Victor Moene <victor.moene@northern.tech>
@victormlg
victormlg force-pushed the agent-driven-cf-reactor branch from 4407d5a to be99f9c Compare August 20, 2026 08:45
@victormlg
victormlg requested a review from larsewi August 20, 2026 09:45

@larsewi larsewi 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.

I see that cf-watchd has some options not present here. Is this intended?

Comment thread cf-reactor/cf-reactor.c
case 'M':
{
Writer *out = FileWriter(stdout);
ManPageWrite(out, "cf-watchd", time(NULL),

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.

Suggested change
ManPageWrite(out, "cf-watchd", time(NULL),
ManPageWrite(out, "cf-reactor", time(NULL),

Comment thread cf-reactor/cf-reactor.c
#include <platform.h>
#include <logging.h>
#include <eval_context.h>
#include <stdio.h>

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.

This header is not used

Comment thread cf-reactor/cf-reactor.c
/* long options only */
case 0:
{
// const char *const option_name = OPTIONS[longopt_idx].name;

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.

There's some commented out code here

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.

3 participants