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

[Namespace]: Fix SNMP AgentX socket connection timeout when using Namespace.get_all() #140

Merged
merged 9 commits into from
Jul 11, 2020
Merged

[Namespace]: Fix SNMP AgentX socket connection timeout when using Namespace.get_all() #140

merged 9 commits into from
Jul 11, 2020

Commits on Jul 10, 2020

  1. [Namespace]: Remove key exists check in dbs_get_all 

    to reduce time taken.

Signed-off-by: SuvarnaMeenakshi <sumeenak@microsoft.com>
    @SuvarnaMeenakshi
    SuvarnaMeenakshi committed Jul 10, 2020
    Configuration menu
    Copy the full SHA
    16f36d8 View commit details
    Browse the repository at this point in the history

  2. Fix as per review comment. 

    Signed-off-by: SuvarnaMeenakshi <sumeenak@microsoft.com>
    @SuvarnaMeenakshi
    SuvarnaMeenakshi committed Jul 10, 2020
    Configuration menu
    Copy the full SHA
    af7d4f4 View commit details
    Browse the repository at this point in the history

  3. Minor fix. 

    Signed-off-by: SuvarnaMeenakshi <sumeenak@microsoft.com>
    @SuvarnaMeenakshi
    SuvarnaMeenakshi committed Jul 10, 2020
    Configuration menu
    Copy the full SHA
    7cfc43d View commit details
    Browse the repository at this point in the history

  4. Connect to all required namespace dbs during init and periodic 

    sync_d functions to reduce time taken in dbs_get_all.

Signed-off-by: SuvarnaMeenakshi <sumeenak@microsoft.com>
    @SuvarnaMeenakshi
    SuvarnaMeenakshi committed Jul 10, 2020
    Configuration menu
    Copy the full SHA
    e264498 View commit details
    Browse the repository at this point in the history

  5. Change test_mibs to use generic wrapper function added. 

    The Namespace functions will be used for both single
and multi-asic platforms.

Signed-off-by: SuvarnaMeenakshi <sumeenak@microsoft.com>
    @SuvarnaMeenakshi
    SuvarnaMeenakshi committed Jul 10, 2020
    Configuration menu
    Copy the full SHA
    a10f55f View commit details
    Browse the repository at this point in the history

Commits on Jul 11, 2020

  1. Fix review comment. 

    Signed-off-by: SuvarnaMeenakshi <sumeenak@microsoft.com>
    @SuvarnaMeenakshi
    SuvarnaMeenakshi committed Jul 11, 2020
    Configuration menu
    Copy the full SHA
    2fd0960 View commit details
    Browse the repository at this point in the history

  2. Fix as per review comment. 

    Signed-off-by: SuvarnaMeenakshi <sumeenak@microsoft.com>
    @SuvarnaMeenakshi
    SuvarnaMeenakshi committed Jul 11, 2020
    Configuration menu
    Copy the full SHA
    7e9eb50 View commit details
    Browse the repository at this point in the history

  3. Remove whitespace. 

    Signed-off-by: SuvarnaMeenakshi <sumeenak@microsoft.com>
    @SuvarnaMeenakshi
    SuvarnaMeenakshi committed Jul 11, 2020
    Configuration menu
    Copy the full SHA
    9c4efd6 View commit details
    Browse the repository at this point in the history

  4. Fix review comment. Avoid modifying mutable object kwargs, 

    do a shallow copy instead.

Signed-off-by: SuvarnaMeenakshi <sumeenak@microsoft.com>
    @SuvarnaMeenakshi
    SuvarnaMeenakshi committed Jul 11, 2020
    Configuration menu
    Copy the full SHA
    fbba260 View commit details
    Browse the repository at this point in the history