Add a design document for Conmon v3 #27053
Conversation
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mheon The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
openshift-ci
bot
added
the
approved
| These should be E2E tests. | ||
| Once a comprehensive test suite exists, we can use it to test both legacy Conmon and Conmon v3. | ||
|
|
||
| In addition to these E2E tests, we can add unit tests to critical functionality in the Rust code as it is written. |
There was a problem hiding this comment.
Also to state the obvious all existing podman tests in the podman repo should pass as well with the new conmon.
|
We should also consider executing Healthchecks with ConmonV3, which would allow them to run on non-systemd systems. ref: #27033 |
I would consider this out of scope for the initial work here though I like the idea. |
|
I'll put it in the doc so we don't forget, but I agree it's out of scope for initial version - let's get it done, then start adding |
|
Sure, it's out of scope for the MVP, but I don't want this to be forgotten. |
|
The doc looks good. I think it clearly summarizes what we are trying to achieve and also how. |
Signed-off-by: Matt Heon <mheon@redhat.com>
mheon
force-pushed
the
design_doc_conmon3
branch
from
d386add to
9bca0d0
Compare
|
/lgtm |
openshift-merge-bot
bot
merged commit f8f6248
into
containers:main
A summary of our intentions with regards to a replacement of Conmon
Does this PR introduce a user-facing change?