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

logging: better timestamp precision / log when CNI request finishes processing #381

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

logging: better timestamp precision / log when CNI request finishes processing #381

wants to merge 2 commits into from

Commits on Aug 17, 2023

  1. logging: use the RFC3399Nano timestamp format 

    The old format - RFC3339 - does not have enough precision - we need to
see milliseconds (at least) to understand how long does it take for a
CNI ADD / DEL to be processed in the IPAM stage to understand how long
does it take for a CNI ADD / DEL to be processed in the IPAM stage.

Signed-off-by: Miguel Duarte Barroso <mdbarroso@redhat.com>
    @maiqueb
    maiqueb committed Aug 17, 2023
    Configuration menu
    Copy the full SHA
    f324246 View commit details
    Browse the repository at this point in the history

  2. logging: add logging to measure the CNI processing times 

    We were missing a clear log indicating the IPAM CNI process is
finishing. Now, it should be possible to understand how long a request
takes from the logs.

Signed-off-by: Miguel Duarte Barroso <mdbarroso@redhat.com>
    @maiqueb
    maiqueb committed Aug 17, 2023
    Configuration menu
    Copy the full SHA
    55ab680 View commit details
    Browse the repository at this point in the history