Cert snapshot logging is too verbose in sysinv.log

Bug #2064925 reported by ayyappa
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Fix Released
Low
ayyappa

Bug Description

Brief Description
-----------------
Cert snapshot logging is too verbose in sysinv.log whenever "system health-query" commands are executed

Severity
--------
minor

Steps to Reproduce
------------------
run "system health-query", "system health-query-kube-upgrade", "system health-query-upgrade" logs the full certificate snapshot to /var/log/sysinv.log

Expected Behavior
------------------
limit the full cert snapshot logging

Actual Behavior
----------------
cert snapshot logging is too verbose in sysinv.log

Reproducibility
---------------
100% reproducible.

System Configuration
--------------------
Any.

Branch/Pull Time/Commit
-----------------------
NA.

Last Pass
---------
NA.

Timestamp/Logs
--------------
NA.

Test Activity
-------------
NA.

Workaround
----------
na

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to config (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/starlingx/config/+/918318

Changed in starlingx:
status: New → In Progress
Ghada Khalil (gkhalil)
tags: added: stx.10.0 stx.fault stx.security
Changed in starlingx:
importance: Undecided → Low
assignee: nobody → ayyappa (mantri425)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to config (master)

Reviewed: https://review.opendev.org/c/starlingx/config/+/918318
Committed: https://opendev.org/starlingx/config/commit/f49374ecea6bf302092c03283cdc2743a6eb9eec
Submitter: "Zuul (22348)"
Branch: master

commit f49374ecea6bf302092c03283cdc2743a6eb9eec
Author: amantri <email address hidden>
Date: Mon May 6 09:39:35 2024 -0400

    Change certificate snapshot to debug logging

    When user executes "system health-query" commands, it logs the
    full certificate snapshot to sysinv.log, this is happening
    because the CertAlarmAudit is imported to health.py module
    to check for any expiry/expired alarms before upgrade activity.
    This fix addresses this issue by changing the "info" log to
    "debug".

    Test Cases:
    PASS: Run "system health-query", "system health-query-kube-upg
          rade" , "system health-query-upgrade" and verify that
          cert snapshot is logging only in the debug mode.

    Closes-bug: 2064925

    Change-Id: Ia0482a557931afdef89a6fa88017ea488a6dca59
    Signed-off-by: amantri <email address hidden>

Changed in starlingx:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.