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

IT FIM - Fix Solaris and Macos fails - For 4.4 #2976

Merged
merged 3 commits into from
Jun 9, 2022

Conversation

Deblintrake09
Copy link
Contributor

Related issues #2881

Description

During runs of the Nightly pipeline, it was found that the test module test_basic_usage_realtime_unsupported.py was having flaky behaviour, on Solaris OS. This PR applies a fix to that test, as well as two other modules that also presented flaky behaviour during testing.

Also, during testing it was found that the modify_file_inode function on the deps/wazuh_testing/fim.py module would cause OS ERROR 18: Invalid cross device link on some systems, because of the function os.replace. The function was replaced by shutil.move, that has the same functionality and parameters but does not generate this error.


Affected Test Modules

  • test_fim/test_files/test_basic_usage/test_basic_usage_realtime_unsupported.py
  • test_fim/test_files/test_follow_symbolic_link/test_change_target.py
  • test_fim/test_files/test_follow_symbolic_link/test_delete_symlink.py
  • test_fim/test_files/test_follow_symbolic_link/test_monitor_symlink.py

Configuration options

OS Version Branch Type
Solaris 4.4.0 4.4 Agent
MacOs 4.4.0 4.4 Agent

Local Internal Options:

Agent

syscheck.debug=2
agent.debug=2
windows.debug=2
monitord.rotate_log=0


Tests Results

Test Path Os/Type Local Jenkins Date By
test_fim/test_synchronization/ Windows 🟢 🟢 🟢 🟢 🟢 🟢 06/07/2022 @Deblintrake09

Required

  • Proven that tests pass when they have to pass.
  • Proven that tests fail when they have to fail.
  • Python codebase satisfies PEP-8 style style guide. pycodestyle --max-line-length=120 --show-source --show-pep8 file.py.
  • Python codebase is documented following the Google Style for Python docstrings.

@Deblintrake09 Deblintrake09 self-assigned this Jun 8, 2022
@Deblintrake09 Deblintrake09 changed the title 2881 solaris fim fail 44 IT FIM - Fix Solaris and Macos fails - For 4.4 Jun 8, 2022
@Deblintrake09 Deblintrake09 marked this pull request as ready for review June 8, 2022 15:48
@jmv74211 jmv74211 merged commit 0a092f4 into 4.4 Jun 9, 2022
@jmv74211 jmv74211 deleted the 2881-solaris-fim-fail-44 branch June 9, 2022 11:09
jmv74211 added a commit that referenced this pull request Jun 9, 2022
Added changes:

- IT FIM - Fix Solaris and Macos fails #2976
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