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.

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

Add new logging tests for analysisd EPS limitation #3509

Merged
merged 3 commits into from
Oct 26, 2022

Conversation

jmv74211
Copy link
Contributor

@jmv74211 jmv74211 commented Oct 26, 2022

Description

This PR adds a new test to check that the following logs are shown in case of dropping events when the events queue is full and there are no analysis credits due to EPS limitation.

The following tests cases have been added:

  • First time that queues are full, there are no available credits, and events are being dropped:

    WARNING: Queues are full and no EPS credits, dropping events.
    
  • First time that the queues are not full, there are available credits, so the events are not being dropped:

      INFO: Queues back to normal and EPS credits, no dropping events.
    
  • Second time that queues are full, there are no available credits, and events are being dropped (note DEBUG level):

    DEBUG: Queues are full and no EPS credits, dropping events.
    
  • Second time that the queues are not full, there are available credits, so the events are not being dropped (note DEBUG level):

      DEBUG: Queues back to normal and EPS credits, no dropping events.
    

Testing performed

Tester Test path Jenkins Local OS Commit Notes
@jmv74211 (Developer) test_analysisd/test_limit_eps/test_logging.py 🟢🟢🟢 🟢🟢🟢 CentOS 7 93cea0e Nothing to highlight

@damarisg damarisg merged commit 2c89c3b into 4.4 Oct 26, 2022
@damarisg damarisg deleted the eps-limit-logging-tests branch October 26, 2022 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants