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

oom-score: retry in case container pid not ready #531

Merged
merged 6 commits into from
Sep 3, 2024
Merged

oom-score: retry in case container pid not ready #531

merged 6 commits into from
Sep 3, 2024

Conversation

dougsland
Copy link
Collaborator

test has being failing, let's retry to catch the
pid 10x before exiting.

Fixes: #523

@dougsland dougsland force-pushed the fixqmoomscore branch 3 times, most recently from ef3ebc2 to 8ad3f4e Compare September 2, 2024 21:18
@dougsland
oom-score: retry in case container pid not ready
5a4b6ec 
test has being failing, let's retry to catch the
pid 10x before exiting.

Fixes: #523

Signed-off-by: Douglas Landgraf <[email protected]>
@dougsland dougsland mentioned this pull request Sep 3, 2024
Closed
@dougsland dougsland added the jira label Sep 3, 2024
@dougsland
Copy link
Collaborator Author

The error now is different, I will dig this one tests/ffi/agent-flood/test.sh in a different ticket:

[ INFO  ] setup_test_containers_in_qm(): prepare quadlet files for bluechi-tester-1.container
[ INFO  ] setup_test_containers_in_qm(): updating AllowedNodeNames in /etc/bluechi/controller.conf
[ INFO  ] setup_test_containers_in_qm(): bluechi-controller reload & restart
[ INFO  ] setup_test_containers_in_qm(): prepare quadlet files for bluechi-tester-2.container
[ INFO  ] setup_test_containers_in_qm(): updating AllowedNodeNames in /etc/bluechi/controller.conf
[ INFO  ] setup_test_containers_in_qm(): bluechi-controller reload & restart
[ INFO  ] run_test_containers(): bluechi-tester-1 reload & restart
[ INFO  ] run_test_containers(): bluechi-tester-2 reload & restart
[ INFO  ] test() bluechi-tester-1 is not active

@dougsland
skip agent-flood test it's failing
a7d8c58 
lets dig it in a different report.

Signed-off-by: Douglas Landgraf <[email protected]>
@dougsland
main.fmf: add test macro
2dd3e4e 
Signed-off-by: Douglas Landgraf <[email protected]>
@dougsland
Copy link
Collaborator Author

CI/CD is green again, the agent-flood must be addressed in a different patch.

@dougsland dougsland merged commit 96ff300 into main Sep 3, 2024
11 checks passed
@dougsland dougsland mentioned this pull request Sep 3, 2024
Closed
@dougsland dougsland deleted the fixqmoomscore branch September 12, 2024 21:44
@dougsland dougsland mentioned this pull request Sep 15, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rhatdan rhatdan Awaiting requested review from rhatdan rhatdan is a code owner

@lsm5 lsm5 Awaiting requested review from lsm5 lsm5 is a code owner

@Yarboa Yarboa Awaiting requested review from Yarboa Yarboa is a code owner

@sandrobonazzola sandrobonazzola Awaiting requested review from sandrobonazzola sandrobonazzola is a code owner

Assignees
No one assigned
Labels
jira
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

tests/ffi/qm-oom-score-adj/test.sh is failing
1 participant
@dougsland