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

[integ-tests] Fix test_essential_features test #6584

Merged
merged 2 commits into from
Dec 4, 2024
Merged

Conversation

hgreebe
Copy link
Contributor

@hgreebe hgreebe commented Dec 2, 2024

Description of changes

  • Fix test_essential_features.py integ test for rhel9 and rocky9

    • The default /usr/share/logrotate/logrotate.conf has dateext option enabled. With this option, all rotated log files will get suffix including the current date with day granularity. Since we force rotate, some files might already exist with the suffic of that current date, leading to the following error: error: destination /var/log/hawkey.log-20241117 already exists, skipping rotation
    • This fix removes the date suffix in the test
  • Fix test_essential_features.py integ test for ubuntu

    • Authorization required, but no authorization protocol specified error was added to mpirun_out_ip array, making it's length greater than the expected 3
    • Checking array length in not necessary
    • Authorization protocol error is not related to this test. It has to do with hwloc

Tests

  • Ran test_essential_features with rhel9, rock9, and ubuntu2004

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@hgreebe hgreebe requested review from a team as code owners December 2, 2024 17:07
@hgreebe hgreebe added the skip-changelog-update Disables the check that enforces changelog updates in PRs label Dec 3, 2024
@hanwen-cluster hanwen-cluster changed the title Fix test_essential_features test [integ-tests] Fix test_essential_features test Dec 4, 2024
@hgreebe hgreebe merged commit d836322 into aws:develop Dec 4, 2024
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-changelog-update Disables the check that enforces changelog updates in PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants