Closed zoidyzoidzoid closed 4 years ago
Thanks for the great feedback, @qmonnet and @pchaigno.
I moved it to the top as @pchaigno suggested, since ebpf.io does seem to be not too Cilium / Isovalent focused, though I think going with the great description @qmonnet suggested is a good idea if we moved it back.
Thank you both for helping contribute to this project.
Great suggestion!
Looks great! Should we also add it to the list itself? Asking because I tend to often skip the intro and my eyes would jump directly to the bullet points, making it easy to skip the link. We could add it just above the Cilium guide, and rename the subsection (maybe Cilium Guide
-> Main Entry Points
or eBPF-Specific Documentation
or ...?). But it really depends on what you prefer, I'm also fine with the current state of the PR.
Were you suggesting we move the Cilium guide
's reference documentation under the eBPF-Specific Documentation
header too?
Yes, the item is named Cilium's BPF and XDP Reference Guide
, so it already gives the information, and hardly needs a Cilium Guide
section that we won't be able to reuse for any other link, I think?
Would be great to have this merged. @zoidbergwill I pushed a commit on top of yours, what do you think about it? Let's decide if we keep it or not and merge the result?
Whoops, my bad. Thought I'd updated the MR.
Your commit looks great! LGTM :+1:
Closes https://github.com/zoidbergwill/awesome-ebpf/issues/35