[doc] Fast Path ICMP flow redirection updates

[devenjagasia]

No description provided.

[mssonicbld]

/azp run

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: devenjagasia / name: Deven Jagasia (3ba92ba, ae4a8c1, d5c04f1, 96f170a, b5c3a28, 935737a, f4e68d2, e690e8e, 91dc9be, 9424a47, 1f6af7c, b62e2a7, 2712384, 0fa1f16, cea2c2e)

[azure-pipelines]

Commenter does not have sufficient privileges for PR 666 in repo sonic-net/DASH

[mssonicbld]

/azp run

[azure-pipelines]

Commenter does not have sufficient privileges for PR 666 in repo sonic-net/DASH

[mssonicbld]

/azp run

[azure-pipelines]

Commenter does not have sufficient privileges for PR 666 in repo sonic-net/DASH

[mssonicbld]

/azp run

[azure-pipelines]

Commenter does not have sufficient privileges for PR 666 in repo sonic-net/DASH

[mssonicbld]

/azp run

[azure-pipelines]

Commenter does not have sufficient privileges for PR 666 in repo sonic-net/DASH

[mssonicbld]

/azp run

[azure-pipelines]

Commenter does not have sufficient privileges for PR 666 in repo sonic-net/DASH

[r12f]

hi @devenjagasia , I am seeing this PR is in draft state and there are some CI failures as well. Please let me know, once it is ready for review! : D

[r12f]

There is a mermaid graph to show the packet format, it will be much easier to write this using that.

[r12f]

I am a bit concerned on this one, and will be great to get it confirmed. With this design, icmpv6 redirect will be tied to PE and ST, and cannot be used by other scenarios. Not sure how this would affect IPv6 VIP and etc.

[devenjagasia]

ok I made the setting name generic to address this: SAI_ENI_ATTR_DISABLE_FAST_PATH_ICMPV6_FLOW_REDIRECTION,

Let me know if this works

[mssonicbld]

/azp run

[azure-pipelines]

Commenter does not have sufficient privileges for PR 666 in repo sonic-net/DASH

[mssonicbld]

/azp run

[azure-pipelines]

Commenter does not have sufficient privileges for PR 666 in repo sonic-net/DASH

[mssonicbld]

/azp run

[azure-pipelines]

Commenter does not have sufficient privileges for PR 666 in repo sonic-net/DASH

[mssonicbld]

/azp run

[azure-pipelines]

Commenter does not have sufficient privileges for PR 666 in repo sonic-net/DASH

[mssonicbld]

/azp run

[azure-pipelines]

Commenter does not have sufficient privileges for PR 666 in repo sonic-net/DASH

[mssonicbld]

/azp run

[azure-pipelines]

Commenter does not have sufficient privileges for PR 666 in repo sonic-net/DASH

[mssonicbld]

/azp run

[azure-pipelines]

Commenter does not have sufficient privileges for PR 666 in repo sonic-net/DASH

[github-actions]

The pre-commit check detected issues in the files touched by this pull request.

Detailed pre-commit check results:

[INFO] Initializing environment for https://github.com/pre-commit/pre-commit-hooks. [WARNING] repo https://github.com/pre-commit/pre-commit-hooks uses deprecated stage names (commit, push) which will be removed in a future version. Hint: often pre-commit autoupdate --repo https://github.com/pre-commit/pre-commit-hooks will fix this. if it does not -- consider reporting an issue to that repo. [INFO] Installing environment for https://github.com/pre-commit/pre-commit-hooks. [INFO] Once installed this environment will be reused. [INFO] This may take a few minutes... mixed line ending........................................................Failed - hook id: mixed-line-ending - exit code: 1 - files were modified by this hook documentation/load-bal-service/fast-path-icmp-flow-redirection.md: fixed mixed line endings
To run the pre-commit checks locally, you can follow below steps:

1. Ensure that default python is python3.

2. Ensure that the pre-commit package is installed:

sudo pip install pre-commit

3. Go to repository root folder

4. Install the pre-commit hooks:

pre-commit install

5. Use pre-commit to check staged file:

pre-commit

6. Alternatively, you can check committed files using:

pre-commit run --from-ref <commit_id> --to-ref <commit_id>
or
pre-commit run --all-files

[mssonicbld]

/azp run

[azure-pipelines]

Commenter does not have sufficient privileges for PR 666 in repo sonic-net/DASH

[github-actions]

The pre-commit check detected issues in the files touched by this pull request.

Detailed pre-commit check results:

[INFO] Initializing environment for https://github.com/pre-commit/pre-commit-hooks. [WARNING] repo https://github.com/pre-commit/pre-commit-hooks uses deprecated stage names (commit, push) which will be removed in a future version. Hint: often pre-commit autoupdate --repo https://github.com/pre-commit/pre-commit-hooks will fix this. if it does not -- consider reporting an issue to that repo. [INFO] Installing environment for https://github.com/pre-commit/pre-commit-hooks. [INFO] Once installed this environment will be reused. [INFO] This may take a few minutes... mixed line ending........................................................Failed - hook id: mixed-line-ending - exit code: 1 - files were modified by this hook documentation/load-bal-service/fast-path-icmp-flow-redirection.md: fixed mixed line endings
To run the pre-commit checks locally, you can follow below steps:

1. Ensure that default python is python3.

2. Ensure that the pre-commit package is installed:

sudo pip install pre-commit

3. Go to repository root folder

4. Install the pre-commit hooks:

pre-commit install

5. Use pre-commit to check staged file:

pre-commit

6. Alternatively, you can check committed files using:

pre-commit run --from-ref <commit_id> --to-ref <commit_id>
or
pre-commit run --all-files

[mssonicbld]

/azp run

[azure-pipelines]

Commenter does not have sufficient privileges for PR 666 in repo sonic-net/DASH

[github-actions]

The pre-commit check detected issues in the files touched by this pull request.

Detailed pre-commit check results:

[INFO] Initializing environment for https://github.com/pre-commit/pre-commit-hooks. [WARNING] repo https://github.com/pre-commit/pre-commit-hooks uses deprecated stage names (commit, push) which will be removed in a future version. Hint: often pre-commit autoupdate --repo https://github.com/pre-commit/pre-commit-hooks will fix this. if it does not -- consider reporting an issue to that repo. [INFO] Installing environment for https://github.com/pre-commit/pre-commit-hooks. [INFO] Once installed this environment will be reused. [INFO] This may take a few minutes... mixed line ending........................................................Failed - hook id: mixed-line-ending - exit code: 1 - files were modified by this hook documentation/load-bal-service/fast-path-icmp-flow-redirection.md: fixed mixed line endings
To run the pre-commit checks locally, you can follow below steps:

1. Ensure that default python is python3.

2. Ensure that the pre-commit package is installed:

sudo pip install pre-commit

3. Go to repository root folder

4. Install the pre-commit hooks:

pre-commit install

5. Use pre-commit to check staged file:

pre-commit

6. Alternatively, you can check committed files using:

pre-commit run --from-ref <commit_id> --to-ref <commit_id>
or
pre-commit run --all-files

[r12f]

maybe better to call it ST / PE / V6 fast path toggle.

[r12f]

thanks a lot Deven for addressing the comments!