Skip to content

ENT-14271: Agent driven valgrind tests - #6306

Merged
olehermanse merged 1 commit into
cfengine:masterfrom
victormlg:agent-driven--valgrind-tests
Sep 2, 2026
Merged

ENT-14271: Agent driven valgrind tests#6306
olehermanse merged 1 commit into
cfengine:masterfrom
victormlg:agent-driven--valgrind-tests

Conversation

@victormlg

Copy link
Copy Markdown
Contributor

No description provided.

@cf-bottom

Copy link
Copy Markdown

Thanks for submitting a PR! Maybe @craigcomstock can review this?

@victormlg
victormlg force-pushed the agent-driven--valgrind-tests branch 3 times, most recently from 6537e73 to 44462ba Compare August 26, 2026 10:55
Comment thread tests/valgrind-check/valgrind.sh Outdated
Ticket: ENT-14271
Signed-off-by: Victor Moene <victor.moene@northern.tech>
@victormlg
victormlg force-pushed the agent-driven--valgrind-tests branch from 44462ba to a788a10 Compare September 2, 2026 12:57
@victormlg
victormlg marked this pull request as ready for review September 2, 2026 13:10

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

Not quite ready? I see you have several reactor_pid related commands commented out...

@victormlg

Copy link
Copy Markdown
Contributor Author

Not quite ready? I see you have several reactor_pid related commands commented out...

So the idea is to get this merged so I can run it inside this PR: #6335

And then I would uncommented the things. Otherwise this will fail since cf-reactor is not a deamon yet

@olehermanse
olehermanse merged commit 6d8b880 into cfengine:master Sep 2, 2026
12 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