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

[intfutil] Check whether the FEC mode is supported on the platform before configuring it to CONFIG_DB #2223

Merged
merged 5 commits into from
Aug 1, 2022

Conversation

stephenxs
Copy link
Collaborator

@stephenxs stephenxs commented Jun 20, 2022

What I did

Check whether the FEC mode is supported on the platform before configuring it to CONFIG_DB

It depends on sonic-net/sonic-swss#2333

Signed-off-by: Stephen Sun [email protected]

How I did it

The supported FEC mode is exposed to STATE_DB.PORT_TABLE|<port>.supported_fecs by orchagent
CLI checks whether the FEC mode to be set is in the supported_fecs list before configuring it to CONFIG_DB.
Specifically, in case the vendor SAI doesn’t support fetching supported FEC mode, N/A should be exposed. In this case, any FEC model should be supported.

How to verify it

Run mock test and manual test

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

@stephenxs stephenxs changed the title Check whether the FEC mode is supported on the platform before configuring it to CONFIG_DB [intfutil] Check whether the FEC mode is supported on the platform before configuring it to CONFIG_DB Jun 22, 2022
@stephenxs stephenxs marked this pull request as ready for review June 23, 2022 00:15
scripts/portconfig Show resolved Hide resolved
scripts/portconfig Show resolved Hide resolved
scripts/portconfig Show resolved Hide resolved
tests/dump_input/dump/default/state_db.json Outdated Show resolved Hide resolved
tests/dump_input/dump/default/state_db.json Outdated Show resolved Hide resolved
tests/dump_tests/dump_state_test.py Outdated Show resolved Hide resolved
scripts/portconfig Show resolved Hide resolved
@liat-grozovik
Copy link
Collaborator

@prgeor could you please help to review?

@prgeor prgeor merged commit 92b889b into sonic-net:master Aug 1, 2022
dprital added a commit to dprital/sonic-buildimage that referenced this pull request Aug 9, 2022
Update sonic-utilities submodule pointer to include the following:
* Fix GCU bug when backend service modifying config ([sonic-net#2295](sonic-net/sonic-utilities#2295))
* Fix issues for sonic_installer upgrade-docker and sonic_installer rollback-docker ([sonic-net#2278](sonic-net/sonic-utilities#2278))
* [crm] add checking for CRM interval range ([sonic-net#2293](sonic-net/sonic-utilities#2293))
* Fix the issue that sonic_platform is not installed on vs image ([sonic-net#2300](sonic-net/sonic-utilities#2300))
* Add FEC correctable and uncorrectable port stats ([sonic-net#2027](sonic-net/sonic-utilities#2027))
* Add CLI to configure YANG config validation ([sonic-net#2147](sonic-net/sonic-utilities#2147))
* Add override testcase to verify removal ([sonic-net#2288](sonic-net/sonic-utilities#2288))
* Fix version in db_migrator  for  ([sonic-net#2289](sonic-net/sonic-utilities#2289))
* [intfutil] Check whether the FEC mode is supported on the platform before configuring it to CONFIG_DB ([sonic-net#2223](sonic-net/sonic-utilities#2223))
* Transfer organization from Azure to sonic-net ([sonic-net#2284](sonic-net/sonic-utilities#2284))
* [watermarkstat] Fix CLI script for unconfigured PG counters ([sonic-net#2239](sonic-net/sonic-utilities#2239))
* Improve the way to check port type of RJ45 port ([sonic-net#2249](sonic-net/sonic-utilities#2249))

Signed-off-by: dprital <[email protected]>
@stephenxs stephenxs deleted the fec-capability branch August 11, 2022 06:51
yxieca pushed a commit that referenced this pull request Aug 11, 2022
…fore configuring it to CONFIG_DB (#2223)

* Support FEC capability

Signed-off-by: Stephen Sun <[email protected]>

* Fix review comments

Signed-off-by: Stephen Sun <[email protected]>

* Adjust dump table test according to STATE_DB change

Signed-off-by: Stephen Sun <[email protected]>

* Add missing field in the test

Signed-off-by: Stephen Sun <[email protected]>

* Do not change dump state test

Signed-off-by: Stephen Sun <[email protected]>
dprital added a commit to dprital/sonic-buildimage that referenced this pull request Aug 12, 2022
Update sonic-utilities submodule pointer to include the following:
* Fix test failure in dump table test in 202205 ([sonic-net#2307](sonic-net/sonic-utilities#2307))
* Convert IPv6 addresses to lowercase in apply-patch ([sonic-net#2299](sonic-net/sonic-utilities#2299))
* [config][muxcable] add support to enable/disable ycable telemetry ([sonic-net#2297](sonic-net/sonic-utilities#2297))
* Fix GCU bug when backend service modifying config ([sonic-net#2295](sonic-net/sonic-utilities#2295))
* [intfutil] Check whether the FEC mode is supported on the platform before configuring it to CONFIG_DB ([sonic-net#2223](sonic-net/sonic-utilities#2223))
* Improve the way to check port type of RJ45 port ([sonic-net#2249](sonic-net/sonic-utilities#2249))
* sonic-utils: initial support for link-training ([sonic-net#2071](sonic-net/sonic-utilities#2071))
* Support to enable fips for the command sonic_installer (sonic-net#2154) ([sonic-net#2303](sonic-net/sonic-utilities#2303))

Signed-off-by: dprital <[email protected]>
preetham-singh pushed a commit to preetham-singh/sonic-utilities that referenced this pull request Nov 21, 2022
…fore configuring it to CONFIG_DB (sonic-net#2223)

* Support FEC capability

Signed-off-by: Stephen Sun <[email protected]>

* Fix review comments

Signed-off-by: Stephen Sun <[email protected]>

* Adjust dump table test according to STATE_DB change

Signed-off-by: Stephen Sun <[email protected]>

* Add missing field in the test

Signed-off-by: Stephen Sun <[email protected]>

* Do not change dump state test

Signed-off-by: Stephen Sun <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants