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.

Sign up for GitHub

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

Jump to bottom

Cherry-picking PRs from master to resolve database not initialize issue #3279

Closed
wants to merge 4 commits into from
Closed

Cherry-picking PRs from master to resolve database not initialize issue #3279

wants to merge 4 commits into from

Commits on Apr 22, 2024

  1. Fix sfputil CLI failure for multi-asic platforms (sonic-net#3168) 

    * Fix sfputil invalid namesapce error

* Add test case for loading port configuration

* Improve cov
    @longhuan-cisco @wumiaont
    longhuan-cisco authored and wumiaont committed Apr 22, 2024
    Configuration menu
    Copy the full SHA
    91f465d View commit details
    Browse the repository at this point in the history

  2. Update port2alias (sonic-net#3217) 

    Basically port2alias Cli became broken on multi-asic platforms after introduction of sonic-net/sonic-buildimage#10960 which removed the initialization of global DB config from portconfig.py (library side) and expects application to do it, but here application side (port2alias) was not updated accordingly.

How I did it
Add load_db_config call to port2alias for initialization
    @abdosi @wumiaont
    abdosi authored and wumiaont committed Apr 22, 2024
    Configuration menu
    Copy the full SHA
    227348a View commit details
    Browse the repository at this point in the history

  3. Fix db_migrator.py where need check before calling initializeGlobalCo… 

    …nfig or initialize(). This is a porting fix effort from master sonic-net#3257.
    @wumiaont
    wumiaont committed Apr 22, 2024
    Configuration menu
    Copy the full SHA
    5b281bb View commit details
    Browse the repository at this point in the history

  4. Revert "Fix db_migrator.py where need check before calling initialize… 

    …GlobalConfig or initialize(). This is a porting fix effort from master sonic-net#3257."

This reverts commit 5b281bb.
    @wumiaont
    wumiaont committed Apr 22, 2024
    Configuration menu
    Copy the full SHA
    3e9a1a9 View commit details
    Browse the repository at this point in the history