Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: consolidate BPF map documentation in concepts/ebpf/intro.rst #12183

Merged
merged 1 commit into from Jun 18, 2020

Conversation

tklauser
Copy link
Member

@tklauser tklauser commented Jun 18, 2020

The documentation about the BPF map scale and limitation is currently
duplicated. The version in the "Introduction" section (intro.rst) was
updated with the most recent changes, while the version in the "Maps"
section (which was split out in #11979 into maps.rst) contains an
outdated version. Move all the up-to-date info to maps.rst.
Incidentally, this also makes the last sentence in the previous section
"Below we show the following possible flows..." make sense again since
it refers to the figure in section "Kubernetes Integration".

@tklauser tklauser added area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. release-note/misc This PR makes changes that have no direct user impact. needs-backport/1.8 labels Jun 18, 2020
@tklauser tklauser requested review from tgraf and aanm June 18, 2020 15:52
@tklauser tklauser requested a review from a team as a code owner June 18, 2020 15:52
@maintainer-s-little-helper maintainer-s-little-helper bot added this to In progress in 1.8.0 Jun 18, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.8.0 Jun 18, 2020
The documentation about the BPF map scale and limitation is currently
duplicated. The version in the "Introduction" section (intro.rst) was
updated with the most recent changes, while the version in the "Maps"
section (which was split out in #11979 into maps.rst) contains an
outdated version.  Move all the up-to-date info to maps.rst.
Incidentally, this also makes the last sentence in the previous section
"Below we show the following possible flows..." make sense again since
it refers to the figure in section "Kubernetes Integration".

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
@tklauser tklauser force-pushed the pr/tklauser/docs-bpf-maps-dedup branch from 6e29071 to 176352a Compare June 18, 2020 15:52
@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 37.116% when pulling 176352a on pr/tklauser/docs-bpf-maps-dedup into e601e8f on master.

@joestringer joestringer merged commit 3ab3d68 into master Jun 18, 2020
1.8.0 automation moved this from In progress to Merged Jun 18, 2020
@joestringer joestringer deleted the pr/tklauser/docs-bpf-maps-dedup branch June 18, 2020 17:31
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.8 in 1.8.0 Jun 18, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.8 to Backport done to v1.8 in 1.8.0 Jun 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. release-note/misc This PR makes changes that have no direct user impact.
Projects
No open projects
1.8.0
  
Merged
1.8.0
Backport done to v1.8
Development

Successfully merging this pull request may close these issues.

None yet

4 participants