Skip to content

Commit 020133b

Browse files
committed
Sort out conditions and scrub comments and naming
Signed-off-by: Andre Fredette <[email protected]>
1 parent aad67d2 commit 020133b

28 files changed

+1494
-482
lines changed

TODO.md

Lines changed: 16 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -5,27 +5,24 @@ The operator is integrated with Dave's load/attach split code.
55
# Testing:
66

77
- Unit tests for the agent and the operator
8-
- The following working samples contain all program types:
9-
- config/samples/bpfman.io_v1alpha1_bpfapplication.yaml
10-
- config/samples/bpfman.io_v1alpha1_clusterbpfapplication.yaml
8+
- All sample yamls work (except for bpfman.io_v1alpha1_test1_clusterbpfapplication.yaml which is a negative test.)
119

1210
# TODO:
1311

1412
(In no particular order.)
1513

16-
- ~~Implement Fentry/Fexit solution.~~
17-
- ~~Integrate with the new bpfman code with load/attach split (of course)~~
18-
- ~~Create a bpf.o file with all the application types for both cluster and~~
19-
~~namespace scoped BpfApplicaitons.~~
20-
- Review the status/condition values. I redid most of the *ApplicationState
21-
item status values, but the actual overall status is still using an existing
22-
ReconcileSuccess value.
23-
- Review all comments and logs.
24-
- ~~Support namespace-scoped BPF Application CRD~~
25-
- ~~Support the rest of the program types.~~
26-
- ~~Delete old directories.~~
27-
- Rename cluster-scoped CRDs and structures to "Cluster*", and remove "ns" from
28-
namespace-scoped CRDs and structures.
29-
- Make sure we have all the old sample programs covered by BpfApplication-based
30-
sample programs and then delete the *Program-based ones.
31-
- More cleanup and testing.
14+
- Review all comments and logs. There is still some BpfProgram-related wording.
15+
- Review cluster and namespace naming in controllers.
16+
- Scrub operator controller for BpfProgram-related naming.
17+
- Scrub logs (need to remove some and make some Info logs Debug)
18+
- Figure out why `k apply -f
19+
config/samples/bpfman.io_v1alpha1_test1_clusterbpfapplication.yaml` works for
20+
kprobe, fentry and fexit, but is rejected for the other program types.
21+
- Delete the "test" sample programs when I'm done using them for testing.
22+
- Review all the ANF-TODOs and create issues if necessary.
23+
- Get the bpfman examples and regression tests working (probably after)
24+
- Scrub logs (need to remove some and make some Info logs Debug)
25+
26+
# TODO (After merge)
27+
- Possibly some of the scrubbing mentioned above.
28+
- Make more of the bpfman-agent controller code common.

0 commit comments

Comments
 (0)