Skip to content

Fix commands in EKS kube-proxy free GSG#12174

Merged
borkmann merged 2 commits intomasterfrom
pr/tklauser/docs-kubeproxy-free-eks
Jun 18, 2020
Merged

Fix commands in EKS kube-proxy free GSG#12174
borkmann merged 2 commits intomasterfrom
pr/tklauser/docs-kubeproxy-free-eks

Conversation

@tklauser
Copy link
Copy Markdown
Member

@tklauser tklauser commented Jun 18, 2020

Small fixes for the commands used in the EKS kube-proxy free GSG:

  • Missing escaping of backslashes
  • Installation of ethtool using yum install instead of amazon-linux-extras install

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
@tklauser tklauser added area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. needs-backport/1.8 labels Jun 18, 2020
@tklauser tklauser requested a review from borkmann June 18, 2020 10:10
@tklauser tklauser requested a review from a team as a code owner June 18, 2020 10:10
@maintainer-s-little-helper
Copy link
Copy Markdown

Please set the appropriate release note label.

2 similar comments
@maintainer-s-little-helper
Copy link
Copy Markdown

Please set the appropriate release note label.

@maintainer-s-little-helper
Copy link
Copy Markdown

Please set the appropriate release note label.

The current command will complain with:

  $ sudo amazon-linux-extras install -y ethtool kernel-ng
  Topic ethtool is not found.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
@tklauser tklauser changed the title docs: fix backslashes in commands for EKS kube-proxy free GSG Fix commands in EKS kube-proxy free GSG Jun 18, 2020
@borkmann borkmann merged commit 753d315 into master Jun 18, 2020
@borkmann borkmann deleted the pr/tklauser/docs-kubeproxy-free-eks branch June 18, 2020 11:27
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

None yet

Development

Successfully merging this pull request may close these issues.

2 participants