Collect IPv6 Networking Information in Log Collector Script #1693
Conversation
Co-authored-by: GitHub <noreply@github.com>
Co-authored-by: GitHub <noreply@github.com>
Co-authored-by: GitHub <noreply@github.com>
| get_network_policy_ebpf_info() { | ||
| try "collect network policy ebpf loaded data" | ||
| echo "*** EBPF loaded data ***" >> "${COLLECT_DIR}"/networking/ebpf-data.txt | ||
| LOADED_EBPF=$(/opt/cni/bin/aws-eks-na-cli ebpf loaded-ebpfdata | tee -a "${COLLECT_DIR}"/networking/ebpf-data.txt) |
There was a problem hiding this comment.
Should we test for the presence of this binary aws-eks-na-cli before using it?
There was a problem hiding this comment.
I see the related discussion here https://github.com/awslabs/amazon-eks-ami/pull/1595/files
There was a problem hiding this comment.
Sine this is already merged, not sure why this is here in this diff.
There was a problem hiding this comment.
Looks like this came from either Carter committing directly to this branch or fallout from the name of the primary branch being changed to main. Either way, I will delete and recreate this PR to fix it
| @@ -27,5 +27,5 @@ jobs: | |||
| - run: git remote add codecommit ${{ secrets.AWS_CODECOMMIT_REPO_URL }} | |||
| - run: git checkout master | |||
| - run: git push codecommit master | |||
| - run: git checkout al2023 | |||
| - run: git push codecommit al2023 | |||
| - run: git checkout main | |||
There was a problem hiding this comment.
Probably need to revert the unintentional changes in this PR
There was a problem hiding this comment.
These came from Carter committing to this branch. Not sure what is happening there, so I will reach out to him
Issue #, if available:
#1474
Description of changes:
This PR updates the log collector script for Linux nodes to collect IPv6 networking information that will assist in debugging. The following commands are run on IPv4 and IPv6 nodes:
ip6tables --wait 1 --numeric --verbose --list --table mangleip6tables --wait 1 --numeric --verbose --list --table filterip6tables --wait 1 --numeric --verbose --list --table natip6tables --wait 1 --numeric --verbose --listip6tables-savetimeout 75 conntrack -L -f ipv6timeout 75 ip -6 rule showtimeout 75 ip -6 route show table allBy submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Testing Done
I ran the updated script against IPv4 and IPv6 Linux nodes and validated that the expected information was collected.
See this guide for recommended testing for PRs. Some tests may not apply. Completing tests and providing additional validation steps are not required, but it is recommended and may reduce review time and time to merge.