[Feature/CI-CD]: Add core API tests #198

Activity

1. LorenzoTettamanti

   added this to CortexFlow v0.1on Jul 8, 2026
2. LorenzoTettamanti

   added

   enhancementNew feature or requestNew feature or request

   help wantedExtra attention is neededExtra attention is needed

   on Jul 8, 2026
3. LorenzoTettamanti

   added the

   Feature

   issue type on Jul 8, 2026
4. LorenzoTettamanti

   added

   rustPull requests that update Rust codePull requests that update Rust code

   CI/CDci/cd workflowsci/cd workflows

   on Jul 8, 2026
5. github-actions

   assigned

   LorenzoTettamanti

   on Jul 8, 2026
6. LorenzoTettamanti

   added

   v0.1.5Version 0.1.5 issues trackerVersion 0.1.5 issues tracker

   on Jul 9, 2026
7. Daniele-Cangi commented on Sep 2, 2026

   

   Daniele-Cangi

   on Sep 2, 2026

   Contributor

   More actions

   Hi Lorenzo,

   I reviewed #198 together with the current cortexbrain-common feature layout and
   would like to contribute a small, hermetic first slice.

   Proposed scope:

   • add a CI cargo-check matrix for meaningful cortexbrain-common feature
     combinations;
   • fix feature-isolation problems surfaced by the matrix, including the
     SslEvent Pod implementation currently not gated by monitoring-structs;
   • add integration contract tests for the public eBPF/userspace event layouts,
     protocol conversion and address conversion;
   • avoid Docker, Kubernetes and privileged eBPF runtime tests in this first PR;
   • avoid metadata and service-discovery files currently touched by Fixed container.name null bug when using Docker. Updated metadata enrichment for k8s environments. Added service cache #216.

   The PR would reference #198 and #158 without attempting to close either umbrella
   issue.

   Please confirm that this focused scope is aligned with the current direction.
8. LorenzoTettamanti commented on Sep 2, 2026

   

   LorenzoTettamanti

   on Sep 2, 2026

   MemberAuthor

   More actions

   Hi @Daniele-Cangi, tests are absolutely welcomed! The first three points are great to implement. I'm finishing the service cache feature in #216 so it would be great not to touch the metadata and service-cache/service-discovery files. Please make sure to send the PR to the 0.1.5 branch. In case you have any question you can ping me here or you can send me an email!

   👍React with 👍1Reacted by DaC
9. Daniele-Cangi commented on Sep 2, 2026

   

   Daniele-Cangi

   on Sep 2, 2026

   Contributor

   More actions

   perfect I'll start working on it right away!
10. LorenzoTettamanti

    added a commit that references this issue on Sep 2, 2026

    [#198]: add common feature isolation and contract tests

    b515300
11. LorenzoTettamanti

    added a commit that references this issue on Sep 10, 2026

    [#198]: add common feature isolation and contract tests

    55e052e
12. LorenzoTettamanti commented on Sep 10, 2026

    

    LorenzoTettamanti

    on Sep 10, 2026

    Last edited by LorenzoTettamanti

    MemberAuthor

    More actions

    Hi @Daniele-Cangi , I've merged PR #216 in case you want to continue working on this issue! Right now I'm pushing the cortexbrain-common@0.1.4 to the crates.io registries
13. siddh34 commented on Sep 14, 2026

    

    siddh34

    on Sep 14, 2026

    Collaborator

    More actions

    Hey guys picking up with CLI tests. I will submit my PR next week!
14. 18 remaining items

    Load more
15. LorenzoTettamanti

    added 12 commits that reference this issue on Sep 28, 2026

    [CortexFlow#198]: use command runner downstream & tests

    aba2cc1

    [CortexFlow#198]: add CLI tests and update test file names for clarity

    c3cfb0b

    [#198]: Command runner impl

    bec976e

    [#198]: use command runner downstream & tests

    27bb03d

    [#198]: add CLI tests and update test file names for clarity

    1bd2dcf

    [#198]: refactor & enhance install logic and add new dependencies for…

    ...

    c875bf9

    [#198]: use command runner downstream & tests

    1b40eb3

    [#198]: add CLI tests and update test file names for clarity

    c053d58

    [#198]: use command runner downstream & tests

    aad33bc

    [#198]: add CLI tests and update test file names for clarity

    bf5d4b2

    [#198]: use command runner downstream & tests

    2419041

    [#198]: add CLI tests and update test file names for clarity

    d0e84d5
16. Daniele-Cangi commented on Oct 2, 2026

    

    Daniele-Cangi

    on Oct 2, 2026

    Contributor

    More actions

    Hi Lorenzo, following #217 and your invitation to continue on #198, I'm taking a second small test-only slice against 0.1.5: hermetic contract tests for cortexbrain-common OpenTelemetry metric recording using synthetic events and an in-memory reader. I'll keep Docker, Kubernetes, root, loaded eBPF programs, and metadata/service-discovery changes out of scope. I also noticed the record_enter_mem_alloc documentation disagrees with its events_total increment; I'll surface that in the PR rather than silently changing behavior. Please let me know if you prefer a different boundary.
17. LorenzoTettamanti

    added a commit that references this issue on Oct 3, 2026

    [#198]: add hermetic OpenTelemetry metric contract tests

    ae91dea
18. LorenzoTettamanti commented on Oct 3, 2026

    

    LorenzoTettamanti

    on Oct 3, 2026

    MemberAuthor

    More actions

    @Daniele-Cangi everything is working fine! Thanks!