[yozhao101]

Signed-off-by: Yong Zhao yozhao@microsoft.com

What I did

This PR aims to implement the subcommands of show/config which can help users configure the status of high memory restart and memory threshold of a specific container.

How I did it

I implemented two functions in config/feature.py and show/feature.py files separately according to the following logic:

• Check whether the container is configured in the FEATURE table of CONFIG_DB
• Check whether the high_mem_restart or mem_threshold field exists in configuration of the container
• Get or update the value of high_mem_restart or mem_threshold field which was provided by user

How to verify it

I tested this implementation on DuT str-msn2700-01.

Test result of Unit Test:

        test/feature_test.py::TestFeature::test_show_feature_status_no_kube_status PASSED                                                                               
        tests/feature_test.py::TestFeature::test_show_feature_status PASSED                                                                                              
        tests/feature_test.py::TestFeature::test_show_feature_config PASSED                                                                                             
        tests/feature_test.py::TestFeature::test_show_feature_status_abbrev_cmd PASSED                                                                                 
        tests/feature_test.py::TestFeature::test_show_bgp_feature_status PASSED                                                                                         
        tests/feature_test.py::TestFeature::test_show_unknown_feature_status PASSED                                                                                     
        tests/feature_test.py::TestFeature::test_show_feature_autorestart PASSED                                                                                         
        tests/feature_test.py::TestFeature::test_fail_autorestart PASSED                                                                                                 
        tests/feature_test.py::TestFeature::test_show_bgp_autorestart_status PASSED                                                                                      
        tests/feature_test.py::TestFeature::test_show_unknown_autorestart_status PASSED                                                                                  
        tests/feature_test.py::TestFeature::test_config_bgp_feature_state PASSED                                                                                         
        tests/feature_test.py::TestFeature::test_config_snmp_feature_owner PASSED                                                                                        
        tests/feature_test.py::TestFeature::test_config_unknown_feature_owner PASSED                                                                                     
        tests/feature_test.py::TestFeature::test_config_snmp_feature_fallback PASSED                                                                                     
        tests/feature_test.py::TestFeature::test_config_bgp_autorestart PASSED                                                                                           
        tests/feature_test.py::TestFeature::test_config_database_feature_state PASSED                                                                                    
        tests/feature_test.py::TestFeature::test_config_database_feature_autorestart PASSED                                                                              
        tests/feature_test.py::TestFeature::test_config_unknown_feature PASSED                                                                                           
        tests/feature_test.py::TestFeature::test_config_lldp_feature_high_mem_restart PASSED                                                                             
        tests/feature_test.py::TestFeature::test_config_lldp_feature_mem_threshold PASSED                                                                                
        tests/feature_test.py::TestFeatureMultiAsic::test_config_bgp_feature_inconsistent_state PASSED                                                                   
        tests/feature_test.py::TestFeatureMultiAsic::test_config_bgp_feature_inconsistent_autorestart PASSED                                                             
        tests/feature_test.py::TestFeatureMultiAsic::test_config_bgp_feature_consistent_state PASSED                                                                     
        tests/feature_test.py::TestFeatureMultiAsic::test_config_bgp_feature_consistent_autorestart PASSED                                                              

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)

[yozhao101]

/Azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[github-actions]

This pull request has been automatically marked as stale because it has not had any activity for 365 days. It will be closed in 14 days if no further activity occurs.

If this PR is still relevant, please leave a comment or push a new commit — this will reset the inactivity timer.

Important: before this PR can be merged it must be rebased onto the latest master. After a year of inactivity it is almost certainly out of date and will not build or merge cleanly against the current tree. Please rebase, resolve any conflicts, and force-push so CI can re-run against current master; a PR that cannot be rebased cleanly will not be merged.

If you believe this PR should remain open regardless of activity, add the never-stale label.

Thank you for your contribution to SONiC!

[github-actions]

This pull request has been closed due to inactivity (no updates in over 1 year). If you would like to continue this work, please re-open the PR or submit a new one rebased onto the latest master.

Thank you for your contribution to SONiC!