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

CA-358870: Backport the fixes from #67 to filter the XAPI ClusterD DB for XS8, Yangtze #78

Merged
merged 3 commits into from
Feb 14, 2024
Merged

CA-358870: Backport the fixes from #67 to filter the XAPI ClusterD DB for XS8, Yangtze #78

merged 3 commits into from
Feb 14, 2024

Commits on Feb 14, 2024

  1. CA-358870: Fix 1: filter_xapi_clusterd_db: remove token from the report 

    The 'token' is an API token that is used to authenticate all API
requests to the xapi-clusterd daemon.

- Apply the fix originally developed and tested by Edwin.
- Update the tests:

  - Ensure that the fix works and keeps working across future changes.

Co-authored-by: Bernhard Kaindl <[email protected]>
Signed-off-by: Edwin Török <[email protected]>
(cherry picked from commit 9a64b44)
    @edwintorok @bernhardkaindl
    edwintorok authored and bernhardkaindl committed Feb 14, 2024
    Configuration menu
    Copy the full SHA
    b8918b2 View commit details
    Browse the repository at this point in the history

  2. CA-388587: Fix 2: Filter xapi-clusterd DB even when PEMs are in files… 

    … instead

Handle the situations where configs doesn't have "pems" keys:

Yangtze does not have REQ-403 and the REQ-403 API changed later to pass
the PEMs in files, so the clusterd DB only has their path names:

Ignore the pems keys if not there and sanitize the rest of the config.

- Applies the fix originally developed and tested by Edwin.
- Update the tests:

  - Ensure that the fix works and keeps working across future changes.

Co-authored-by: Bernhard Kaindl <[email protected]>
Signed-off-by: Edwin Török <[email protected]>
(cherry picked from commit e4a80b6)
    @bernhardkaindl
    bernhardkaindl committed Feb 14, 2024
    Configuration menu
    Copy the full SHA
    77776c7 View commit details
    Browse the repository at this point in the history

  3. CA-388587: Fix 3: Log any errors in filter_xapi_clusterd_db() 

    Signed-off-by: Bernhard Kaindl <[email protected]>
(cherry picked from commit 5fd22f8)
    @bernhardkaindl
    bernhardkaindl committed Feb 14, 2024
    Configuration menu
    Copy the full SHA
    946a873 View commit details
    Browse the repository at this point in the history