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

Logcollector IT: Skip flaky tests #3217

Closed
wants to merge 12 commits into from
Closed

Conversation

Deblintrake09
Copy link
Contributor

@Deblintrake09 Deblintrake09 commented Aug 30, 2022

Related issue
#3195

Description

During research of Issue #3195 related to the test module test_logcollector/test_configuration/test_basic_configuration_age.py it was found that some modules had flaky behaviour causing random fails. The tests have been skipped. An Issue requesting a proper Refactor/Fix has been opened here.

Skipped

  • test_logcollector/test_configuration/test_basic_configuration_age.py
  • test_logcollector/test_configuration/test_basic_configuration_query.py
  • test_logcollector/test_command_monitoring/test_command_execution_freq.py
  • test_logcollector/test_configuration/test_basic_configuration_only_future_events.py
  • test_logcollector/test_command_monitoring/test_command_execution_dbg.py

Testing performed

Tester Test path Jenkins Local OS Commit Notes
@user (Developer) test_logcollector/ 🟢🟢 🟢 🟢 🟢 🟢🟢 🟢 CentOS - Ubuntu - Windows Nothing to highlight
Seyla (Reviewer) test_logcollector/ 🟢 🟢 🟢 CentOS - Ubuntu - Windows Nothing to highlight

Copy link
Member

@damarisg damarisg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please solve the conflicts.

@Deblintrake09
Copy link
Contributor Author

Closing - Replaced by #3263 aimed at 4.4

@Deblintrake09 Deblintrake09 deleted the 3195.logcollector.fix branch September 7, 2022 14:42
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